From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============9077204308919166679==" MIME-Version: 1.0 From: Magnus Fromreide Subject: Re: [Powertop] [PATCH 1/2] Make it possible to include parse-events.h before libc headers. Date: Wed, 25 Jul 2012 21:26:10 +0200 Message-ID: <1343244370.6440.4.camel@sara> In-Reply-To: 50103074.5020703@linux.intel.com To: powertop@lists.01.org List-ID: --===============9077204308919166679== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable 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 names= pace > > so it should be renamed. > > Additionally the __ununsed macro is used from just one implementation f= ile so > > the macro is moved there. > > > > Signed-off-by: Magnus Fromreide > > = > > = > 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 --===============9077204308919166679==--