| commit | d61d97c097947e9aabbb5edc3a2ac5cd92cddae3 | |
| author | (no author) <(no author)@41a61cd8-c433-0410-bb1c-e256eeef9e11> | |
| Tue, 18 Dec 2007 02:57:30 +0000 (18 02:57 +0000) | ||
| committer | (no author) <(no author)@41a61cd8-c433-0410-bb1c-e256eeef9e11> | |
| Tue, 18 Dec 2007 02:57:30 +0000 (18 02:57 +0000) | ||
| tree | 1ad490c3d61c464ecb8dc016af1a3b8463f58013 | treesnapshot (tar.gz zip) |
| parent | 5cc8c2c5317b67a3ddb06cbe5cb1db6343ce8a1b | commitdiff |
r1415@opsdev009 (orig r74655): dreiss | 2007-12-17 18:54:06 -0800
Thrift: Make default values for base types work in Java.
Summary:
The Java generator wasn't setting default values for base types
in the zero argument constructor. This check was probably brought over
from the C++ generator, where base types are given their default values
in the member variable initializer list (or whatever that thing is called).
Blame Rev: Somewhere in the mysterious past. I tried to find it, but failed.
Reviewed By: mcslee
Test Plan: Recompiled Thrift and thrifted a file that showed the bug.
Revert Plan: ok
Other Notes:
Bug reported by Jake Luciani.
git-svn-id: http://svn.facebook.com/svnroot/thrift/trunk@726 41a61cd8-c433-0410-bb1c-e256eeef9e11
Thrift: Make default values for base types work in Java.
Summary:
The Java generator wasn't setting default values for base types
in the zero argument constructor. This check was probably brought over
from the C++ generator, where base types are given their default values
in the member variable initializer list (or whatever that thing is called).
Blame Rev: Somewhere in the mysterious past. I tried to find it, but failed.
Reviewed By: mcslee
Test Plan: Recompiled Thrift and thrifted a file that showed the bug.
Revert Plan: ok
Other Notes:
Bug reported by Jake Luciani.
git-svn-id: http://svn.facebook.com/svnroot/thrift/trunk@726 41a61cd8-c433-0410-bb1c-e256eeef9e11
| compiler/cpp/src/generate/t_java_generator.cc | diffblobblamehistory |