Affects Version/s: None
Fix Version/s: None
Component/s: D - Library
Patch Info:Patch Available
The attached patches fix handling of structs with non-nullable (as in assignability of null to their D representation), required fields, and add a simple test case to prevent similar stupid mistakes in the future.
An actual test case for the functionality should be added to the standalone test site at some point (I was absolutely surprised that this is not covered yet), but I can't spare the time to do so right now, and fixing the issue is critical, as it e.g. breaks the HBase Thrift interface for D.
|Transition||Time In Source Status||Execution Times||Last Executer||Last Execution Date|
|2d 1h 31m||1||Roger Meier||24/Jun/12 20:21|
|32d 20h 43m||1||Jake Farrell||27/Jul/12 17:05|
|Status||Resolved [ 5 ]||Closed [ 6 ]|
|Status||Open [ 1 ]||Resolved [ 5 ]|
|Assignee||David Nadlinger [ klickverbot ]|
|Resolution||Fixed [ 1 ]|
|Field||Original Value||New Value|
|Attachment||0001-Fixed-D-codegen-for-required-non-nullable-fields.patch [ 12533077 ]|
|Attachment||0002-Added-struct-codegen-sanity-tests-directly-to-thrift.patch [ 12533078 ]|