[PATCH] build: Fixup headers for C++ inclusion

Craig Gallek kraigatgoog at gmail.com
Fri Aug 12 14:35:06 PDT 2016


On Fri, Aug 12, 2016 at 5:03 PM, Thomas Haller <thaller at redhat.com> wrote:
> On Wed, 2016-08-10 at 11:29 -0400, Craig Gallek wrote:
>> On Fri, Jul 15, 2016 at 10:29 AM, Craig Gallek <kraigatgoog at gmail.com
>> > wrote:
>> >
>> > From: Craig Gallek <kraig at google.com>
>> >
>> > - Fix typos in extern "C" wrappers.
>> > - 'class' is a reserved word, change to cls.
>> > - int -> enum casts are not automatic.
>> > - Static strings must be of type const char*
>> > - Add a few implicit header dependencies explicitly.
>> >
>> > Signed-off-by: Craig Gallek <kraig at google.com>
>> Bump... Any takers for this patch?
>
>
> Hi Craig,
>
>
> sorry for the late reply (was on extended holidays).
>
>
> Merged: https://github.com/thom311/libnl/commit/625c8f13cf793ba61365c510970f4569a483f2ba
Thank you, enjoy the rest of your time off!

btw, I didn't find the github repository you linked here while trying
to figure out where to send this patch.  The project page
(https://www.infradead.org/~tgr/libnl/) lists a standard git
repository on infradead.org and a different github repository that
appears to be out of date.  What's the preferred patch mechanism for
the project now?



More information about the libnl mailing list