Added support for method(\t syntax for operators #1
Conversation
| rl_line_buffer[tokenstart] == '!')) { | ||
| tokenstart--; | ||
| // check for special operators | ||
| if (strchr("\\><=|&+-*/%^~!", rl_line_buffer[tokenstart])){ |
There was a problem hiding this comment.
Should ! be in this list? It seems like this would make push! give completions for !.
There was a problem hiding this comment.
No, it must be handled separatly. Should this logic be written in julia repl_methods instead, so that iJulia and REPL.jl can just copy the result?
There was a problem hiding this comment.
Is there any problem with sending the whole buffer there?
There was a problem hiding this comment.
We can send the whole buffer, and eventually I think we do want one canonical token extraction function written in Julia, although I'm not sure if that needs to be part of this PR. There's already something in REPLCompletions.jl, although it might have the same problems with operators.
| tokenstart--; | ||
| } | ||
| tokenstart++; | ||
| // ! can't be the first character of a function, unless is's the only |
There was a problem hiding this comment.
Sorry to nitpick, but there's still a small typo here (is's -> it's)
Added support for method(\t syntax for operators
Operators like (><=|&+-*/%^~!) was not supported in the original version. This patch adds that capability.