> Yes we should, to match the parser behaviour you note further on.

One thing I forgot to mention is that it's the same commit mentioned above that 
also modified the parser not to allow leading whitespaces…

> Well the default value in Scintilla is 1 which matches the comment, but then 
> Geany sets it 0 which is correct (IMNAAHO).

You seem to have a lot against leading whitespaces here, far beyond whether we 
should or shouldn't use them in these specific files, why is that?
My personal *feeling* is that we shouldn't *use* them in our conffiles, but 
that we should support them in the file type; I don't see any reason why to 
limit this, is there really valid cases where it causes problem to support it?
@eht16 as the author of the mentioned commit, can you remember the reason ages 
later?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3428#issuecomment-1465693781
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/3428/[email protected]>

Reply via email to