Re: [mico-devel] Marshall exception

[Karel G. <kg...@ob...>]

Hi Peter,

this looks like an issue solved by this patch on MICO HEAD, isn't it?

Sun Jan  3 18:59:34 CET 2010  Karel Gardas <kg...@ob...>
   * fix "miscompilation" of typecode.cc by GNU C++ >= 4.4
   This patch fixes typecode.cc to not use TK_RECURSIVE definition since
   this makes GNU C++ >= 4.4 miscompiling several of typecode.cc's switch
   statements where it simply omits case blocks where TK_RECURSIVE is
   used, since TK_RECURSIVE provides value which is not defined in the
   TCKind enum type. Whole behaviour is completed by emitted warnings:
     typecode.cc:<line>: warning: case label value is less than minimum
     value for type
   Unfortunatelly GCC community members claim the compiler is right here
   so I enhanced tckind.h by a provided bootstrap patch to include
   tk_recursive value inside the TCKind enum definition which makes GNU
   C++ happy and resulting MICO working again. For more information about
   this issue see GCC PR#42576 
(http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42576)


Thanks!
Karel

On 10/31/12 05:54 PM, Gregory, Peter wrote:
> Hi
>
> I have tracked this issue down and can confirm it is another problem
> caused by the upgrade to gcc version 4.4.
>
>
>
> A different problem with the same cause was reported in the issue
> "typecode assertion failed on gcc 4.4
> <https://sourceforge.net/mailarchive/message.php?msg_id=29183633>  ".
>
>
>
> The issue is caused by a change in behaviour of the switch statement.
>
> With this version of gcc the switch statement in CORBA::TypeCode::decode
> (DataDecoder&dc, MapPosTC *_omap, ULong level),
>
> now enters the default block instead of the "case TK_RECURSIVE:" block.
> This is because the value of TK_RECURSIVE is outside the range of the
> TCKind enum.
>
>
>
> I made the relevant changes to typecode.cc and tckind.h, and the issue
> is resolved.
>
>
>
> Peter
>
>
>
>
>
> ------------------------------------------------------------------------------
> Everyone hates slow websites. So do we.
> Make your web apps faster with AppDynamics
> Download AppDynamics Lite for free today:
> http://p.sf.net/sfu/appdyn_sfd2d_oct
>
>
>
> _______________________________________________
> Mico-devel mailing list
> Mic...@li...
> https://lists.sourceforge.net/lists/listinfo/mico-devel


-- 
Karel Gardas                  kg...@ob...
ObjectSecurity Ltd.           http://www.objectsecurity.com