At Thu, 3 Mar 2016 12:15:13 +0100, Pavel Stehule <pavel.stehule@gmail.com> wrote in <CAFj8pRDB2PpESLxnwNDXmhvTS9VL0NMeAnUdv_hps9WzYwXLjw@mail.gmail.com> pavel.stehule> 2016-03-03 12:06 GMT+01:00 Kyotaro HORIGUCHI < > the requirement of space before is not good :( - It should be any different > than operator chars. Not only space. > > all other is perfect :)
Yeah, I fortunately agree with you:p
But the things are not so simple. readline can handle single prefix characters but cannot not handle prefix strings.
The new diff adds ':' to WORD_BREAKS and adjusts related codes. As far as I can see, colons are used only for variable prefix, type casts and named parameter assignments in function calls. This covers the first two and the last wouldn't be a matter of tab-completion. This works as the following.
It is good if we could limit the candidate types by knowing the operand type but it seems a bit complicated.
Some familiar type names won't come as candidates. They are "int", "float", "decimal", "dec", which are known only to gram.y but it does't seem to matter, too.
Thoughts? Opinions?
It works. I found only one issue with multi word named types
I have not any complete option for select 1::timestamp with
maybe it is another limit of readline
Regards
Pavel
regards,
-- Kyotaro Horiguchi NTT Open Source Software Center