Re: [Powertop] [PATCH 1/2] Make it possible to include parse-events.h before libc headers.
Magnus Fromreide <magfr at lysator.liu.se> Wed, 25 Jul 2012 21:26:10 +0200
| Newsgroups | dev.linux.lists.powertop |
|---|---|
| Message-ID | <1343244370.6440.4.camel@sara> |
On Wed, 2012-07-25 at 10:44 -0700, Chris Ferron wrote: > On 07/23/2012 11:03 PM, Magnus Fromreide wrote: > > The __unused macro interferes with the glibc and is in a reserved namespace > > so it should be renamed. > > Additionally the __ununsed macro is used from just one implementation file so > > the macro is moved there. > > > > Signed-off-by: Magnus Fromreide <magfr(a)lysator.liu.se> > > > > > I will forward this on to Steve Rostedt. pevent code is not actually > strictly part of the powertop src, it is just temporarily being built as > part of the project so powertop can take advantage of it early. Thanks. I found no contact information for pevent but it feels like an imported library. > That said, it may not be a good fit for the grand scheme. > Which reminds me I need to push a readme explaining this. I am looking forward to it. Patch 2/2 depends on this, should I prepare an alternate version that doesn't trigger the bug in the pevent headers? /MF