Dict value cannot be in a new line about rapydscript HOT 6 CLOSED

I'm looking into this issue, for now the following workaround should work:

a = {"ke":\
    "looooooong string"
}

from rapydscript.

This is fundamentally unfixable with the current design of the compiler. The problem is that the the ':' character is used both to denote the start of a new block and to delineate keys from values. Indentation checking currently happens in the tokenizer and not the parser. The tokenizer has no reliable way to know if ':' mean a new block or end of key. This is the case because object literals can have function as values.

To fix this would mean moving the whitespace checking into the parser. But that is a pretty invasive change.

from rapydscript.

Actually, I figured this can be done fairly simply by having the parser modify the internal state of the tokenizer. This is not a clean solution, but it works, and is, as far as I can see the ebst that can be done with the current compiler design.

kovidgoyal/rapydscript-ng@880e38f

from rapydscript.

If I remember correctly, I already used a similar hack elsewhere - it was also to treat the state within a dictionary differently than within a regular block (although this may have been in original version of RapydScript). Also, you may have already addressed this (I haven't had a chance to look at your code), but beware of functions nested within the dictionary:

def():
    # normal indentation rules apply
    a = {
        # special indention rules
        "b":
            def():
                # normal indention rules again
                ...
        # special indentation rules
    }

from rapydscript.

Yeah functions as dictionary values are the reason the hack becomes necessary in the first place :)

from rapydscript.

This is now partially fixed, only the first key now throws this error. Don't remember which commit addressed it, but the code is now using a similar trick to Kovid's version - so I'm confused why it doesn't work when the split key appears in hash first.

from rapydscript.