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