/[Apache-SVN]
ViewVC logotype

Revision 672907


Jump to revision: Previous Next
Author: dreiss
Date: Mon Jun 30 20:45:47 2008 UTC (16 years, 4 months ago)
Changed paths: 4
Log Message:
Partial handlling of 64-bit integer constants.

- Parse integer constants in Thrift files as 64-bit ints.
- Die if an overflow occurs.
- Warn if an enum value cannot fit in 32 bits.
- Add a simple test for the above.

I ran all of the generators over BrokenConstants.thrift before adding the
overflow, and they appeared to work.  However, the code generated was not
always valid (for example, the 64-bit constant must have an LL suffix in C++).


Changed paths

Path Details
Directoryincubator/thrift/trunk/compiler/cpp/src/parse/t_const_value.h modified , text changed
Directoryincubator/thrift/trunk/compiler/cpp/src/thriftl.ll modified , text changed
Directoryincubator/thrift/trunk/compiler/cpp/src/thrifty.yy modified , text changed
Directoryincubator/thrift/trunk/test/BrokenConstants.thrift added

infrastructure at apache.org
ViewVC Help
Powered by ViewVC 1.1.26