[coyotos-dev] BigNum.cxx buglet? - CAPIDL_U32

Jonathan S. Shapiro shap at eros-os.com
Mon Oct 22 09:14:18 EDT 2007


On Sun, 2007-10-21 at 19:50 -0400, Jeroen Visser wrote:
> Hello Dex,
> 
> I hacked around the problem for now and haven't really fixed the
> issue. The problem seems to be here:
> 
> "capidl/backend/o_c_hdr.cxx" line 2548 of 3317 --76%-- col 12:
>       else {
>         unsigned long sig = s->CodedName();
>         size_t indent = out.indent_for_macro();
>         out << "\n#define OC_"
>             << s->QualifiedName('_')
>             << " CAPIDL_U32(0x"
>             << std::hex << sig << std::dec
>             << ")\n";
>         out.indent(indent);

Very sorry. I forgot about this completely. The fix is simple: declare
'sig' as uint32_t.  I have just committed that, and also the
corresponding fixes for other uses of CodedName (which should have been
uint64_t).

The change is committed and pushed.


shap
-- 
Jonathan S. Shapiro, Ph.D.
Managing Director
The EROS Group, LLC
www.coyotos.org, www.eros-os.org



More information about the coyotos-dev mailing list