Re: converting Ada to handle USE_MAPPED_LOCATION

2005-03-23 Thread Geert Bosch
On Mar 22, 2005, at 22:09, Per Bothner wrote: Of course that's in the eye of the beholder. I think a local translation is cleaner and more robust/safer than a global opaque type/call-back. OK, let's go with that approach then. -Geert

Re: converting Ada to handle USE_MAPPED_LOCATION

2005-03-22 Thread Per Bothner
Geert Bosch wrote: Of the three proposals: [...] The ideal solution I think is for Ada to use line-map's source_location for Sloc in its lexer. [...] translate Sloc integers to source_location when we translate the Ada intermal format to Gcc trees. [...] the location_t in the shared Gcc should be

Re: converting Ada to handle USE_MAPPED_LOCATION

2005-03-20 Thread Arnaud Charlet
> IIRC, current default max line length for Ada programs is 255, and 32767 > for configuration pragmas. But it's always better to be prepared > for the worse :). Actually that's no longer the same. You can have line length much bigger now with GNAT, assuming you use the proper compiler switch. As

Re: converting Ada to handle USE_MAPPED_LOCATION

2005-03-19 Thread Geert Bosch
Hi Per, Of the three proposals: [...] The ideal solution I think is for Ada to use line-map's source_location for Sloc in its lexer. [...] translate Sloc integers to source_location when we translate the Ada intermal format to Gcc trees. [...] the location_t in the shared Gcc should be a language-

Re: converting Ada to handle USE_MAPPED_LOCATION

2005-03-19 Thread Laurent GUERBY
On Sat, 2005-03-19 at 11:52 -0800, Per Bothner wrote: > && line not excessively big; > The latter is a heuristic to avoid using up line number too greedily; > I suspect it may not be an issue. IIRC, current default max line length for Ada programs is 255, and 32767 for configuration p

converting Ada to handle USE_MAPPED_LOCATION

2005-03-19 Thread Per Bothner
I fixed the regressions USE_MAPPED_LOCATION for C and C++, and we've now branched, so it's is probably time to get Ada working. (There is a Java problem, for which I have a patch, but it may hurt debugging. Until that is fixed, leave Java out of --enable-languages, or apply this patch: http://gcc.