search

infobloxopen / seal

Apache License 2.0
16 stars 11 forks source link

#111 single quoted strings #112

Closed abynenkov-ib closed 4 years ago

abynenkov-ib commented 4 years ago

plus null pointer exception fix

rchowinfoblox commented 4 years ago

Not sure if we're supposed to allow single-quoted strings in SEAL, I'll let others comment on that.

wplu commented 4 years ago

we discussed this and do not see a need for it. every feature requires maintenance. it is harder to remove something once we support it - we only want to implement this when needed.

rchowinfoblox commented 4 years ago

If we don't support single-quoted strings, we should still fix the coredump and instead return an error when a single-quoted string is attempted as indicated in issue #111

wplu commented 4 years ago

i agree - @abynenkov-ib please return appropriate error