[Geany-devel] Geanydebug big endian elf patch

[Cameron Hart]

I've attached an updated patch file which works against the current
trunk version of geanygdb.

I thought I had followed the coding style in the existing plugin,
although it looks like it has changed a little between the tar version I
originally created the patch against and the current SVN version. If
there's any stylistic things you'd like me to change please let me know
more specifically.

Regards,
Cameron.

Enrico Tröger wrote:
> On Fri, 26 Dec 2008 23:37:20 +1300, Cameron Hart
> <cam at bitshifter.net.nz> wrote:
>
> Hi,
>
>
>   
>> I've modified geanydebug 0.0.1 so that it can open big endian
>> elf files. I've tested my changes on x86_64 and powerpc (PlayStation3)
>> running Ubuntu 8.10. All that needed to be changed we reading the elf
>> type correctly for the different endians. Once that was fixed so far
>> everything else has just worked.
>>     
>
> cool, I think we can accept these changes.
> But for some reason the patch doesn't apply against latest SVN.
> Cameron, could you please create the patch against the SVN version?
> Please note that the plugin has been renamed recently.
>
> Furthermore, I'd like to ask you to follow the existing coding style in
> the plugin so the whole code doesn't clutter too much and it stays
> easier to read.
> Thanks.
>
>
> Regards,
> Enrico
>
>   
> ------------------------------------------------------------------------
>
> _______________________________________________
> Geany-devel mailing list
> Geany-devel at uvena.de
> http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
>   
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gdb-ui-main.c.big-endian-elf.patch
Type: text/x-patch
Size: 3323 bytes
Desc: not available
URL: <http://lists.geany.org/pipermail/devel/attachments/20090104/8b8bc6a4/attachment.bin>