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
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.
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.