On Tue, 4 Nov 2008, Akim Demaille wrote:

> > Anyway, I pushed the following to branch-2.4.1 to fix it.  I'll push to
> > master soon.
> The problem with this fix is that it is very "C/C++/Java" based.  That is why
> is was obsoleted and I believed that Paul had agreed to get rid of it.  I'm
> obviously wrong.
> The day we add support for languages that are not using ; as these languages
> do, we'll have problems.  If possible, I'd like to add a warning so that code
> that lacks this ';' is reported (I don't know yet how feasible this warning
> is).

I thought about the possibility of a warning too.  It seems like 
/[};]\s*$/ is the right pattern, but I have this feeling I'm forgetting 
some special case that will annoy users with correct grammars.  Actually, 
there could be trailing comments or cpp directives.

Alternatively, we could just formally deprecate it in NEWS now and then 
drop support as soon as it gets in the way of other languages.

I'll wait until we've settled this before pushing this fix to master.

Reply via email to