-
-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add autocomplete for wxf
#4872
Add autocomplete for wxf
#4872
Conversation
Probably I'm not understand what you want to achieve with this patch :( why would you want to autocomplete the configuration variables when you use |
Maybe I misunderstood then or edited the wrong file. Latter most likely. Trying to add wxf to the list of commands when typing |
Ok don't worry :) If you just want to add 'wxf' to the list of commands that are autocompleted, the first line of your patch is enough. The rest should be to autocomplete the arguments of a command. Seems like you copied the autocompletion of the |
@@ -446,7 +446,7 @@ static const char *radare_argv[] = { | |||
"b", "bf", "b?", | |||
"/", "//", "/a", "/c", "/m", "/x", "/v", "/v2", "/v4", "/v8", "/r" | |||
"y", "yy", "y?", | |||
"wx", "ww", "w?", | |||
"wx", "ww", "w?", "wxf", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this line
Ok thanks. I tried just the first line and had no luck. Might have been something else though. I'll double check tomorrow. |
Oh that's weird. I tried that to and when i pressed "w" i could see "wxf" on the list of possible completions :( |
I must have goofed somewhere. Remove the config var block and squashed commits |
Thanks a lot! |
for #4751