Re: [PATCH] Fix PR pascal/2283

[Jonas Maebe <jonas at freepascal dot org>]

On 26 Sep 2007, at 15:49, Eli Zaretskii wrote:

> > From: "Pierre Muller" <muller@ics.u-strasbg.fr>
> > Cc: "'Jonas Maebe'" <jonas@freepascal.org>
> > Date: Wed, 26 Sep 2007 14:25:38 +0200
> >
> > -              || ((current_language->la_language == language_m2)
> > +              || ((current_language->la_language == language_pascal)
>
> Why is it TRT to remove Modula-2 here?

Because the that code resides in pascal_val_print() in p-valprint.c,  
and not in m2_val_print() in m2-valprint.c (the former was based on  
the latter, and the leftover language_m2 was simply a copy/paste  
error afaics).

In fact, I have no idea why that code (in both the Pascal and  
Modula-2 cases) even checks the current language, as those routines  
are obviously language-specific. Maybe the Modula-2 code originally  
came from a generic routine, and the check was never removed (and  
then copied over to the Pascal version when that one was written)?


Jonas