Account Options

  1. Sign in
The old Google Groups will be going away soon, but your browser is incompatible with the new version.
Google Groups Home
« Groups Home
reserved keyword "class" in xdo.h
There are currently too many topics in this group that display first. To make this topic appear first, remove this option from another topic.
There was an error processing your request. Please try again.
flag
  5 messages - Collapse all  -  Translate all to Translated (View all originals)
The group you are posting to is a Usenet group. Messages posted to this group will make your email address visible to anyone on the Internet.
Your reply message has not been sent.
Your post was successful
 
From:
To:
Cc:
Followup To:
Add Cc | Add Followup-to | Edit Subject
Subject:
Validation:
For verification purposes please type the characters you see in the picture below or the numbers you hear by clicking the accessibility icon. Listen and type the numbers you hear
 
Daniel Kahn Gillmor  
View profile  
 More options Mar 8 2012, 9:58 am
From: Daniel Kahn Gillmor <d...@fifthhorseman.net>
Date: Thu, 08 Mar 2012 09:58:08 -0500
Local: Thurs, Mar 8 2012 9:58 am
Subject: reserved keyword "class" in xdo.h

Hi Jordan--

I just got this report (http://bugs.debian.org/663007) from a debian user:

> xdo.h includes the name "class" which is a reserved keyword in c++.
> This prevents including it in a C++ program:
> extern "C" {
> #include <xdo.h>
> }

> g++ file.cpp
> In file included from test.cpp:2:0:
> /usr/include/xdo.h:530:37: error: expected ',' or '...' before 'class'

> please rename or remove the parameter.

I note that xdo.c already has the parameter re-named "_class" instead of
"class", and it doesn't look to me like shifting the name of the
parameter in xdo.h would change either the API or the ABI.

What do you think of the attached patch?

Regards,

        --dkg

  663077.patch
< 1K Download

 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Jordan Sissel  
View profile  
 More options Mar 8 2012, 10:01 am
From: Jordan Sissel <j...@semicomplete.com>
Date: Thu, 8 Mar 2012 07:01:35 -0800
Local: Thurs, Mar 8 2012 10:01 am
Subject: Re: [xdotool-users] reserved keyword "class" in xdo.h

On Thu, Mar 8, 2012 at 6:58 AM, Daniel Kahn Gillmor
<d...@fifthhorseman.net>wrote:

I swear I've fixed this like 30 times by now, my mind must be going! hah :)

Yeah shoot me a pull request or patch - otherwise I can hand patch it.
FYI, in case I forgot to announce - the source repo lives here now:
http://github.com/jordansissel/xdotool

-Jordan


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Daniel Kahn Gillmor  
View profile  
 More options Mar 8 2012, 10:45 am
From: Daniel Kahn Gillmor <d...@fifthhorseman.net>
Date: Thu, 08 Mar 2012 10:45:00 -0500
Subject: Re: [xdotool-users] reserved keyword "class" in xdo.h

On 03/08/2012 10:01 AM, Jordan Sissel wrote:

> On Thu, Mar 8, 2012 at 6:58 AM, Daniel Kahn Gillmor <d...@fifthhorseman.net>wrote:
>> What do you think of the attached patch?

> Yeah shoot me a pull request or patch

Hmm, i attached a patch to my original e-mail.  Maybe it got stripped by
your MUA somehow?  I'm trying again in this message.

> - otherwise I can hand patch it.
> FYI, in case I forgot to announce - the source repo lives here now:
> http://github.com/jordansissel/xdotool

Yep, the patch is based on that source.

Regards,

        --dkg

  663077.patch
< 1K Download

 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Jordan Sissel  
View profile  
 More options Mar 8 2012, 11:00 am
From: Jordan Sissel <j...@semicomplete.com>
Date: Thu, 8 Mar 2012 08:00:58 -0800
Local: Thurs, Mar 8 2012 11:00 am
Subject: Re: [xdotool-users] reserved keyword "class" in xdo.h

On Thu, Mar 8, 2012 at 7:45 AM, Daniel Kahn Gillmor
<d...@fifthhorseman.net>wrote:

Oh, I'm a nut. I see the patch now. :)

Patch applied in master. I'll try and get a release out in the next week.
Thanks!

-Jordan


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Daniel Kahn Gillmor  
View profile  
 More options Mar 8 2012, 11:04 am
From: Daniel Kahn Gillmor <d...@fifthhorseman.net>
Date: Thu, 08 Mar 2012 11:04:28 -0500
Local: Thurs, Mar 8 2012 11:04 am
Subject: Re: [xdotool-users] reserved keyword "class" in xdo.h
On 03/08/2012 11:00 AM, Jordan Sissel wrote:

> Patch applied in master. I'll try and get a release out in the next week.

Thanks, Jordan!

        --dkg


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
End of messages
« Back to Discussions « Newer topic     Older topic »