Require semi-colons

[Adam Cameron]

G'day:

I could have sworn I saw a setting in Railo (and, by association, Lucee) to the effect of "require semi-colons". Am I imagining things?

-- 

Adam

[Peter Boughton]

I don't remember one, but can't promise that isn't a local memory fault.

[Adam Cameron]

On Tuesday, 26 May 2015 15:40:24 UTC+1, Peter Boughton wrote:

I don't remember one, but can't promise that isn't a local memory fault.

This is unlikely. I've not known you to be wrong in the past, and I don't quite know how to react to that as a possibility.

[Christian Meis]

I; don't; think; there's; such; a; setting;

At least I've not come across one ever...

Christian 

[Nando Breiter]

https://issues.jboss.org/browse/RAILO-1510

Maybe there was discussion on this at some point to make writing code compatible with ACF a little easier ...

Aria Media Sagl
Via Rompada 40
6987 Caslano
Switzerland

+41 (0)91 600 9601
+41 (0)76 303 4477 cell
skype: ariamedia

--
You received this message because you are subscribed to the Google Groups "Lucee" group.
To unsubscribe from this group and stop receiving emails from it, send an email to lucee+un...@googlegroups.com.
To post to this group, send email to lu...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/lucee/CABCrx0UzYFOt7%2BTe8h%2BYZ5zXttbTp4QCvoKGnvT0V2msutA72A%40mail.gmail.com.

For more options, visit https://groups.google.com/d/optout.

[Harry Klein]

Hi Adam,

 

I don’t see a settings check in the checkSemiColonLineFeed() method:

 

lucee\source\java\core\src\lucee\transformer\cfml\script\AbstrCFMLScriptTransformer.java

 

                private final boolean checkSemiColonLineFeed(ExprData data,boolean throwError, boolean checkNLBefore,boolean allowEmptyCurlyBracked) throws TemplateException {

                               comments(data);

                               if(!data.srcCode.forwardIfCurrent(';')){

                                              

                                               // curly brackets?

                                               if(allowEmptyCurlyBracked) {

                                                               int pos = data.srcCode.getPos();

                                                               if(data.srcCode.forwardIfCurrent('{')) {

                                                                              comments(data);

                                                                              if(data.srcCode.forwardIfCurrent('}')) return true;

                                                                              data.srcCode.setPos(pos);

                                                               }

                                               }

                                              

                                              

                                               if((!checkNLBefore || !data.srcCode.hasNLBefore()) && !isFinish(data) && !data.srcCode.isCurrent('}')){

                                                               if(!throwError) return false;

                                                               throw new TemplateException(data.srcCode,"Missing [;] or [line feed] after expression");

                                               }

                               }

                               return true;

                }

 

-Harry

--

You received this message because you are subscribed to the Google Groups "Lucee" group.
To unsubscribe from this group and stop receiving emails from it, send an email to lucee+un...@googlegroups.com.
To post to this group, send email to lu...@googlegroups.com.

To view this discussion on the web visit https://groups.google.com/d/msgid/lucee/8ea5a0c4-ffa1-476f-b21d-271aceae7853%40googlegroups.com.

[Adam Cameron]

On Tuesday, 26 May 2015 16:01:50 UTC+1, Nando Breiter wrote:

https://issues.jboss.org/browse/RAILO-1510

Maybe there was discussion on this at some point to make writing code compatible with ACF a little easier ...

Ha: I'd already voted for that one, so it was probably what i was (mis-)remembering though.

I've copied it to the new system as https://luceeserver.atlassian.net/browse/LDEV-369

Cheers Nando.

--
Adam 

[Michael Offner]

nope, never was ...

Micha

--

You received this message because you are subscribed to the Google Groups "Lucee" group.
To unsubscribe from this group and stop receiving emails from it, send an email to lucee+un...@googlegroups.com.
To post to this group, send email to lu...@googlegroups.com.

To view this discussion on the web visit https://groups.google.com/d/msgid/lucee/0baa816b-a321-4fbf-92e4-3585cea0d278%40googlegroups.com.

[Adam Cameron]

On Tue, May 26, 2015 at 4:06 PM, Adam Cameron <camero...@gmail.com> wrote:

G'day:

I could have sworn I saw a setting in Railo (and, by association, Lucee) to the effect of "require semi-colons". Am I imagining things?

On Wednesday, 27 May 2015 08:26:04 UTC+1, Micha wrote:

nope, never was ...

Cool, thanks for confirmation.

-- 

Adam