- Joined
- Apr 5, 2008
- Messages
- 368
- Reaction score
- 0
- Points
- 0
While studying C++ and Visual Studio 2010 I loaded up the source files for [ame="http://www.orbithangar.com/searchid.php?ID=4781"]LolaMFD for Orbiter 2010[/ame] to use as a test platform. While studying the code I noticed one oapiDebugString line uncommented. I commented it out, recompiled the code, ran a scenario, and found that (to no one's surprise but my own, I am sure) no longer did I have that annoying debug string in white at the lower left of the screen.
Is this minor change enough to justify another version posted to Orbit Hanger?
Pros: 1) It's the first thing I've ever done that's worked in C++ and VS2010.
2) That debug string is really annoying, and I've seen other posters mention that too, so my change is useful. (Orb was nice enough to share his mod that totally eliminated the debug string, but unfortunately that also blocks *all* debug strings, and that space is used by several other addons to provide user instructions.)
3) As I understand it, Lola is open sourced, and anyone can contribute as long as full credit is given and the source files are included.
4) I am sure the last developers to work on it meant to remove the debugs, since all other instances of oapiDebugString are indeed commented out. They just missed one.
Cons: 1) It's the first thing I've ever done that's worked in C++ and VS2010.
2) It's a really, really simple fix for anyone who knows even the basics of coding, so it might be regarded as too trivial to post.
3) Even though Lola is still useful, it is also still incomplete and somewhat buggy. I'd love to be able to fix it, or write my own autopilot from scratch, but that is going to be a bit into the future, as I am still learning basics.
So what do you guys think? If the consensus is no, well, another in my pile of self tweaked mods that only exist on my computer.
And that's just fine too....
Is this minor change enough to justify another version posted to Orbit Hanger?
Pros: 1) It's the first thing I've ever done that's worked in C++ and VS2010.
2) That debug string is really annoying, and I've seen other posters mention that too, so my change is useful. (Orb was nice enough to share his mod that totally eliminated the debug string, but unfortunately that also blocks *all* debug strings, and that space is used by several other addons to provide user instructions.)
3) As I understand it, Lola is open sourced, and anyone can contribute as long as full credit is given and the source files are included.
4) I am sure the last developers to work on it meant to remove the debugs, since all other instances of oapiDebugString are indeed commented out. They just missed one.
Cons: 1) It's the first thing I've ever done that's worked in C++ and VS2010.
2) It's a really, really simple fix for anyone who knows even the basics of coding, so it might be regarded as too trivial to post.
3) Even though Lola is still useful, it is also still incomplete and somewhat buggy. I'd love to be able to fix it, or write my own autopilot from scratch, but that is going to be a bit into the future, as I am still learning basics.
So what do you guys think? If the consensus is no, well, another in my pile of self tweaked mods that only exist on my computer.
And that's just fine too....