Frederic Leroy's update to Dan Widyono's patch to compile
cleanly on 64
Derek Atkins
warlord at MIT.EDU
Tue Aug 16 21:57:20 EDT 2005
FWIW, I would prefer a patch that has as few casts as possible but still
compiled without warnings. Casts can hide potential issues. Much better to
have a configure test and use some format macro instead of casting.
-derek
Quoting Frederic Leroy <fredo at starox.org>:
> On Tue, 16 Aug 2005 19:01:28 -0400
> Dan Widyono <dan at widyono.net> wrote:
>
>> Someone else might wish to give the patches a once-over with a
>> fine-toothed comb with regard to why not changing the other lld's,
>> but for mine, I just compiled, found errors, fixed the errors ONLY,
>> recompiled, found errors, fixed those errors ONLY, etc. This prevented
>> me from fixing anything except what I said I was fixing in the bug
>> report.
> I just took your patch and test(successfully).
> I think it is better to fix all llds to make the code clean and
> coherent.
> When I compile it, there is a bunch of warning, and some are related to
> cast size mismatch.
> I'll try to fix it and propose a patch tomorrow.
>
> --
> Frederic Leroy
> Lost in France
> _______________________________________________
> gnucash-devel mailing list
> gnucash-devel at gnucash.org
> https://lists.gnucash.org/mailman/listinfo/gnucash-devel
>
--
Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory
Member, MIT Student Information Processing Board (SIPB)
URL: http://web.mit.edu/warlord/ PP-ASEL-IA N1NWH
warlord at MIT.EDU PGP key available
More information about the gnucash-devel
mailing list