Posts: 4
Threads: 1
Joined: May 2023
Thanks: 0
Given 1 thank(s) in 1 post(s)
Hello all,
I have recently been working on an issue on GitHub and have also been advised that the code could do with some further changes.
After investigating how to reactor the code (and to what extent) I have some questions.
As I am new here, what do I ask to begin with?
Thanks,
MrSach
Posts: 4
Threads: 1
Joined: May 2023
Thanks: 0
Given 1 thank(s) in 1 post(s)
Thank you for letting me know this.
Working on issue #5495, I have been working on StringCompression.h and StringCompression.cpp.
It took some time, but I managed to find a way to refactor the code to simplify some data types.
I am, however, still unsure about one member method function that I have amended on my local system as it may risk undefined behaviour.
Would you like me to post the refactored code here in this thread?
Posts: 4,628
Threads: 115
Joined: Dec 2011
Thanks: 693
Given 494 thank(s) in 423 post(s)
It would be easier to review and easier to put into the code base if you create a pull request on Github.
Posts: 4
Threads: 1
Joined: May 2023
Thanks: 0
Given 1 thank(s) in 1 post(s)
Hello,
Pull request #5503
I refreshed my forked branch, but git is flagging an issue with commit #1eb0c8b.
It appears that appveyor may need to be re-run?
Also, feedback would be welcome for the PR.