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
fix(_comp_command_offset): fix offset mismatching for words vs COMP_WORDS (use words) #860
Conversation
32f077e
to
25d5f8f
Compare
I have rebased it on top of the latest master and added adjustments to the existing interfaces in commit 25d5f8f. |
25d5f8f
to
3842c49
Compare
3842c49
to
440abbf
Compare
440abbf
to
f0b7d24
Compare
f0b7d24
to
717b98a
Compare
6251f46
to
e1ace64
Compare
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.
Thanks! Seeing the examples in the tests makes it clearer to me to understand what's going on.
Added some comment suggestions, pre-approved + feel free to merge when the conflicts are resolved.
750a244
to
bfc752b
Compare
I have applied the suggested changes to the code comments and squashed them into the corresponding commit. I added an additional adjustment as follows:
|
bfc752b
to
d9aed51
Compare
Co-authored-by: Ville Skyttä <ville.skytta@iki.fi>
d9aed51
to
06bfe23
Compare
Thanks. |
Fixes #722.
This fixes it the other way around as #802. Now
_comp_command_offset
receives the index inwords
instead of that inCOMP_WORDS
.