-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Pocket Sax improvements #5622
Pocket Sax improvements #5622
Conversation
Why 2 commits, and with the same title? |
Thanks, not sure how that happened. I updated a branch in the past without having to squash. Should be fixed now. I assume this will only be merged post-3.4? |
@Jojo-Schmitz are you happy with the changes (since the original/previous Pocket Sax was your PR) and ready to be merged? |
Fine by me, except maybe the "squashed" in the commit title |
Agreed, " squashed" squashed. Ready for merge. |
?? It's still in the commit title, forgot to |
Jup that was it. What would I do without you? |
Once we're in 3.5 mode... |
I was thinking the exact thing paraphrased; "once the 3.4.x dust settles" |
Resolves: Pocket Sax spelling and abbreviations improvements
(forum discussion, no issue)
These are the commands I used to edit and the two edited files
I also swapped around Pocket Sax/Xaphoon to Xaphoonin/Pocket Sax in the description field to prevent ambiguous expansion of the "/" to Pocket Sax / Pocket Xaphoon
perl -p -i -e 's:Pocket-Sax/Xaphoon:Xaphoon/Pocket Sax:' share/instruments/instruments{.xml,xml.h}
perl -p -i -e 's/pocket-sax/Pocket Sax in/i' share/instruments/instruments{.xml,xml.h}
perl -p -i -e 's/P.-Sx. /Pkt. Sax /' share/instruments/instruments{.xml,xml.h}
egrep -i 'P.-Sx.|Pocket-Sax|Pkt. Sax|Pocket Sax' share/instruments/instruments{.xml,xml.h}