From: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | Peter Geoghegan <peter(at)2ndquadrant(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgadmin-hackers] Re: [pgadmin-hackers] Re: [pgadmin-hackers] error: expected initializer before ‘!’ token |
Date: | 2011-03-14 20:22:35 |
Message-ID: | 4D7E790B.3060306@lelarge.info |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Le 13/03/2011 09:46, Dave Page a écrit :
> On Sun, Mar 13, 2011 at 1:52 AM, Peter Geoghegan <peter(at)2ndquadrant(dot)com> wrote:
>> On 12 March 2011 19:13, Guillaume Lelarge <guillaume(at)lelarge(dot)info> wrote:
>>> Maybe a stupid question but what is the ! in the variable name
>>> (ex_hash_setop_except_png!_data)?
>>
>> There's no good reason. It isn't a valid identifier name.
>>
>> Whatever preprocessor generates ex_hash_setop_except.pngc and other
>> files like it is broken.
>>
>> Take a look at this code, at line 205 of same, that clearly illustrates this:
>>
>> static wxIcon *ex_hash_setop_except_png!_ico()
>
> Line 205 here reads:
>
> static wxIcon *ex_hash_setop_except_png_ico()
>
> On Mac and Windows (and Linux worked last time I tried it). Can you
> see what's causing the issue for you in the preprorocessor?
>
So I worked a bit on this issue tonight. I think you didn't allocate
enough memory for filename. See the patch attached. With this patch, it
works OK for me. I'm wondering if the patch should also do +1 on the
other calloc calls. Any ideas on this?
Can you check that it still works on Windows and Mac OS X with this patch?
--
Guillaume
http://www.postgresql.fr
http://dalibo.com
Attachment | Content-Type | Size |
---|---|---|
pngc.patch | text/x-diff | 383 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Geoghegan | 2011-03-14 20:37:16 | Re: [pgadmin-hackers] Re: [pgadmin-hackers] Re: [pgadmin-hackers] error: expected initializer before ‘!’ token |
Previous Message | Dave Page | 2011-03-14 09:00:02 | pgAgent commit: USe exit status macros to get the Unix exit code, and r |