jql icon indicating copy to clipboard operation
jql copied to clipboard

Remove useless outer parens

Open impredicative opened this issue 6 years ago • 4 comments

Consider removing the redundant outer parens since all your examples have it. Why can't it just be imputed? Why do I have to remember to type two extra characters that convey zero extra information?

impredicative avatar Jan 07 '20 18:01 impredicative

You mean trailing parentheses of the whole query? That's actually a great idea! I'll think about how to include this into the grammar.

cube2222 avatar Jan 07 '20 18:01 cube2222

Yes, the leading and trailing paren of the whole query. I would make it optional, such that the query continues to work either way, with or without the outer parens, assuming this makes sense.

impredicative avatar Jan 07 '20 18:01 impredicative

There will only ever be a single leading one, but trailing ones are definitely not useful, and were annoying to me too.

Thanks for the issue!

cube2222 avatar Jan 07 '20 18:01 cube2222

I would remove the requirement of the single leading paren, making it optional. I figure that the program should be able to impute it since all the examples have it. If something is always present, it conveys no information.

impredicative avatar Jan 07 '20 18:01 impredicative