Boost logo

Boost :

From: Arkadiy Vertleyb (vertleyb_at_[hidden])
Date: 2004-09-10 14:25:20

"Daniel James" <daniel_at_[hidden]> wrote

> When running your tests, the intel linux compiler gives this warning:
> main.cpp(143): warning #858: type qualifier on return type is meaningless
> const int cf();
> which means that BOOST_TYPEOF_PRESERVE_LVALUE(cf()) gives int, and your
> test fails, so you might want to remove that test.

OK. Is it the only problem with Intel? I did not realize things are so
good -- I've been only testing with VC7.1 and GCC 3.3.

> Also neither encode_signed nor encode_unsigned seem to work for zero
> values. I think encode_signed is missing a check for zero, and I guess
> encode_unsigned should add 1 to value when encoding, and then subtract 1
> when decoding - or have a special case like encode_signed does.

I see, both are buggy... I guess I'll have to use the second integer for
unsigned to properly handle UINT_MAX (somehow I think this value is likely
to be used).

> Other than those small points the library is looking good.

Thanks for your comments. I am glad you liked the library after all...
I'll make the fixes and post the modified version sometime soon.


Boost list run by bdawes at, gregod at, cpdaniel at, john at