On Thu, Nov 02, 2006 at 07:59:50AM +0100, Martin Kersten wrote:
dank fabian.
Fabian Groffen wrote:
Hi,
Please find attached a (compressed) patch which loop lifts all loops in the pathfinder sources such that variable declarations go at the top of the scope the loop is in, or a higher scope if more appropriate.
Rationale: I don't feel like appending stdc99 flags in a (possible) Gentoo ebuild for the next release.
In other words, this patch removes the C99 requirement, which I think more people will benefit from, such that MonetDB/* cleanly compiles with --disable-strict.
Just for the records: there is more C99 specific (i.e., beyond C89) code in Pathfinder than for-loop-inlined variable declarations; gcc w/o --stdc99 might only complain about the for-loop-inlined variable declarations; other compiles might/will complain about more (macro's with variable argument lists, "//" comments, name enum initializations ...); hence, even with/after this patch, Pathfinder code still is/requires C99. Stefan
Please consider applying this patch to the current CVS sources.
My apologies for the large attachment on this list; there are simply lots of loops to "lift".
-- | Dr. Stefan Manegold | mailto:Stefan.Manegold@cwi.nl | | CWI, P.O.Box 94079 | http://www.cwi.nl/~manegold/ | | 1090 GB Amsterdam | Tel.: +31 (20) 592-4212 | | The Netherlands | Fax : +31 (20) 592-4312 |