Re: fix for #3219

[Eric POUECH]

why not test for '?' and '.' which are the forms we know off, instead of letting the door ajar ?
A+
> Message du 19/10/05 18:27
> De : "Vijay Kiran Kamuju" 
> A : "Hans Leidekker" , [e-mail]
> Copie à : 
> Objet : Re: fix for #3219
>
> I have kept it there incase of regressions
> We can just easily uncomment and test.
>
> Thanks,
> Vijay
>
> On 10/19/05, Hans Leidekker wrote:
> > On Wednesday 19 October 2005 09:19, Vijay Kiran Kamuju wrote:
> > > - /* MS mangled names always begin with '?' */
> > > - if (*sym->current != '?') return FALSE;
> > > - /* MS mangled names always begin with '?'
> > > - if (*sym->current != '?') return FALSE; */
> >
> > It's better to remove code then disabling it
> > this way. We can always refer to CVS history.
> >
> > -Hans

fix for #3219

[Vijay Kiran Kamuju]

The patch has been commited.
If you want i will send another one.
bye,
Vijay

On 10/20/05, Eric POUECH  wrote:
> why not test for '?' and '.' which are the forms we know off, instead of
> letting the door ajar ?
>
> A+
> > Message du 19/10/05 18:27
> > De : "Vijay Kiran Kamuju"
> > A : "Hans Leidekker" , [e-mail]
> > Copie à :
> > Objet : Re: fix for #3219
> >
> > I have kept it there incase of regressions
> > We can just easily uncomment and test.
> >
> > Thanks,
> > Vijay
> >
> > On 10/19/05, Hans Leidekker wrote:
> > > On Wednesday 19 October 2005 09:19, Vijay Kiran Kamuju wrote:
> > > > - /* MS mangled names always begin with '?' */
> > > > - if (*sym->current != '?') return FALSE;
> > > > - /* MS mangled names always begin with '?'
> > > > - if (*sym->current != '?') return FALSE; */
> > >
> > > It's better to remove code then disabling it
> > > this way. We can always refer to CVS history.
> > >
> > > -Hans

fix for #3219

[Eric Pouech]

Vijay Kiran Kamuju wrote:
> The patch has been commited.
> If you want i will send another one.
> bye,
> Vijay

I'll send a correct fix for this.
A+