diff options
| author | David Disseldorp <ddiss@samba.org> | 2014-01-06 06:20:08 +0100 |
|---|---|---|
| committer | Günther Deschner <gd@samba.org> | 2014-01-16 16:22:53 +0100 |
| commit | e891652339bbfa192f88c9d22d78c78ed0fc8511 (patch) | |
| tree | 387323fef0ea35fcfe2cd26c02171433da93f0df /python/modules.h | |
| parent | 1a0eddbe6c3e89097ff35d5c38e411e09e3f4d30 (diff) | |
| download | samba-e891652339bbfa192f88c9d22d78c78ed0fc8511.tar.gz samba-e891652339bbfa192f88c9d22d78c78ed0fc8511.tar.xz samba-e891652339bbfa192f88c9d22d78c78ed0fc8511.zip | |
dcerpc.idl: use logical dcerpc_object ordering
This IDL code:
typedef [nodiscriminant] union {
[default] dcerpc_empty empty;
[case(LIBNDR_FLAG_OBJECT_PRESENT)] GUID object;
} dcerpc_object;
Compiles into the following default-before-case marshalling code:
switch (level) {
default: {
NDR_CHECK(ndr_push_dcerpc_empty(ndr, NDR_SCALARS, &r->empty));
break; }
case LIBNDR_FLAG_OBJECT_PRESENT: {
NDR_CHECK(ndr_push_GUID(ndr, NDR_SCALARS, &r->object));
break; }
}
The default entry before case does not change the flow of execution but
is more logical when present at the end of the switch statement.
Signed-off-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Diffstat (limited to 'python/modules.h')
0 files changed, 0 insertions, 0 deletions
