I would like to point out that identifiers like "__LIBQMI_GLIB_H_INSIDE__" and "_LIBQMI_GLIB_QMI_MESSAGE_H_" do not fit to the expected naming convention of the C language standard.
Would you like to adjust your selection for unique names?
Created attachment 112808 [details] [review]
Developed the changes required in the attached patch, but I don't think I'm going to go for it. Keeping the patch for future reference.
I don't think that this is very important truth be told, given that the possible name clash with a reserved identifier is virtually impossible (we anyway add a libqmi-specific prefix after the underscores).
Plus, the patch cannot cover every possible underscore prefix usage without breaking API (e.g. the GObject struct type definitions start with an underscore), so I'm going to close this one as wontfix (for now). If we ever do a full API cleanup, I'll recover the changes.
The patch applies on top of 89c7b0fc54aa, btw.