-
Notifications
You must be signed in to change notification settings - Fork 10
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
Cleanup - Remove unused function and do not use removed function #236
Conversation
922f6f4
to
98ea2c7
Compare
98ea2c7
to
0991333
Compare
@rswgnu I learned today how easy it is to remove a file from a PR and then it was easy to repurpose this PR to just a small cleanup. PTAL. |
Let's work on the merge of this when we are together as well. |
36fd5db
to
e10ae65
Compare
@rswgnu With the fix of the CI this PR now builds so can be merged if approved. Was that the reason for merging this together? It has 9 commits but all boils down to two small changes. Replace of one function and removal of another. So just look at the combined result and not each commit. I'll squash so the commits becomes one. |
What
Do not use obsolete functions.
Why
We can use the new function directly since they are supported from 27.2.
Note
Also fixes a duplication related to an obsolete function. Maybe I missed this or made a bad rebase!?