Crash when using with search + complete about grumble

[skaldesh]

I was being stupid. This bug has nothing to do with backsearch. The problem is that grumble can not handle the case that somebody types a command, say daemon, then moves the cursor back to the start of the word and hits Tab. This crashes as well. Good thing you discovered this @h4ng3r

[skaldesh]

Hi,

I could reproduce your bug by doing the following:

• go run sample/simple/main.go
• run the daemon command once
• then press Ctrl+r for back-search and enter for example daem
• hit Tab
• crash

To me, it seems like it is critical that the command that will be completed by the back-search is also in the command history of grumble. I could not reproduce the bug with a fresh start and directly doing the back-search. I had to run the command in question at least once. Can you confirm that @h4ng3r?

@r0l1 and I will take a look at your suggested changes, thank you for submitting and reporting the issue, very much appreciated.

[h4ng3r]

Hi @skaldesh,

Yep that's true you need a match in the back search, you need to hit TAB on a match otherwise it won't crash. Nice catch.

Thanks,

[skaldesh]

@h4ng3r I played around with your proposed fix, but I think the len(line) > pos prevents the auto completion for me to work generally. Typing dae and hitting Tab then does no longer auto complete, since the line is dae, pos is 2 (the end) and the expression evaluates to false then.

In addition, when using the backsearch, which then autocompletes daemon, position is still 0, which causes grumble to append a dash - in front of daemon. This happens, since the auto completer is presented with the line daemon and therefore wants to start suggesting flags... I will propose some solution for this soon

[h4ng3r]

@skaldesh yep, that's true I realized yesterday that my solution was disabling auto completion. Still not sure how to fix it properly, sorry.

[skaldesh]

No problem, will play around a little bit, thank you for the quick answers :D

[skaldesh]

@h4ng3r I created a PR. Can you, @h4ng3r , check it out and see, if it fixes your problem and hopefully does not cause any further bugs? :)

[h4ng3r]

I tested the PR, the issue is fixed and the autocomplete is working perfectly. Looks good to me.

[skaldesh]

Thank you for the help again. The issue has been fixed in master