jolly submitted this change.

View Change

Approvals: pespin: Looks good to me, approved laforge: Looks good to me, but someone else must approve; Verified fixeria: Looks good to me, approved
VTY: Initialize fields of struct cmd_element in order of declaration

gpp requires designated initialisers to be initialized in the order
their fields were declared. With this patch it is possible to include
VTY's command.h within C++ code.

Change-Id: Ifaa31980f149f9c1f1a9b28f318a4850f6855472
---
M include/osmocom/vty/command.h
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/osmocom/vty/command.h b/include/osmocom/vty/command.h
index 61b5881..667e9ab 100644
--- a/include/osmocom/vty/command.h
+++ b/include/osmocom/vty/command.h
@@ -218,8 +218,8 @@
.string = cmdstr, \
.func = funcname, \
.doc = helpstr, \
- .attr = attrs, \
.daemon = dnum, \
+ .attr = attrs, \
};

/* global (non static) cmd_element */
@@ -229,8 +229,8 @@
.string = cmdstr, \
.func = funcname, \
.doc = helpstr, \
- .attr = attrs, \
.daemon = dnum, \
+ .attr = attrs, \
};

#define DEFUN_CMD_ELEMENT_ATTR_USRATTR(funcname, cmdname, cmdstr, helpstr, attrs, usrattrs) \

To view, visit change 40188. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ifaa31980f149f9c1f1a9b28f318a4850f6855472
Gerrit-Change-Number: 40188
Gerrit-PatchSet: 2
Gerrit-Owner: jolly <andreas@eversberg.eu>
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: jolly <andreas@eversberg.eu>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>