Le 4 nov. 08 à 21:21, Joel E. Denny a écrit :

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

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.

Yep, definitely.

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.

NEWS should warn about this now.

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

AFAIC, you can do it now.

Reply via email to