[orbitcpp-list] Re: [orbitcpp-commits] orbitcpp philipd

[Andreas K. <ak...@ix...>]

Hi Phil,

> - Added compiler helper function to detect whether type is variable length 
> (and added the function to IDLElement (I ripped these off of the
> orbit-idl code - hope this is okay)

Uh, hope you don't mind me bickering, but I'd rather have 
"isVariableLength()" as a member func of IDLType. It's much more natural,
you don't have to rip off dumb hacks from orbit-idl code, polymorphism
makes your day without lengthy case statements. also mind the fact that
not all IDLTypes are IDLElements (such as short, string).

> - Added struct declaration support for fixed and variable length structs

cool.

phil, i guess i'll be kinda busy till next thursday. don't expect too much
work done by me till that damn calculus exam is all over...

cu
andy