Rename class files to match their class declarations in JS_Mode#145
Conversation
|
@wmiler if you're happy with this one, please just merge it. I don't think it will affect work-in-process too much because these files aren't worked on too often. But it's best to not leave changes like this hanging just in case. |
|
Oh, and yes, Xcode clang-format removed some whitespace from the doxy hooks in DecodedText |
wmiler
left a comment
There was a problem hiding this comment.
Why are both files in tools linking to QT5core? Should they not be linking to qt6?
Also, nitpick: Why are neither of the two tool files or varicode capitalized like every thing else?
|
No, those two diagnostic tools use Qt5. They are standalone tools used to tweak the kalman filter and frequency tracker, not members of any class. They can be named anything they want. Varicode is that way because that's the way Jordan originally wrote it. I"m working on JS8_Mode right now. Haven't gotten to JS8_Main yet. I'll fix it when I get there. I'm trying to do this in smaller steps. But the UI is coming out of mainwindow next. That one is going to be big. |
|
Push it then |
This is a continued cleanup of the codebase. It renames four class files to match their class declarations. When code documentation is referred to, this helps with maintainability of the code by making it easier to identify the files that contain the class.
Trying to be a little less invasive now with each step of the code cleanup process.