Reply To: MWEdit

Yeah, that’s pretty normal. When there’s an error, oftentimes the compilers complain about surrounding lines as well even though there isn’t an error in them. Pretty annoying as it can sometimes be hard to fix a lot of issues all at once but I think it has something to do with how the parsers are designed.

Getting close to being finished with the File/ directory. Once that’s done, I’ll be able to start working on the IL/ directory. Depending on how extensive the changes are, we may be able to simply delete most of it in favor of the stock version of DevIL, which will make getting a build together much easier, so I’ll keep an eye out while I’m reformatting things.

May take a break soonish. Haven’t really been getting some good entertainment in due to being hyper-focused on MWEdit 😛

Edit:

It’s possible that File/dxffile.h and File/dxffile.cpp can be removed. They don’t do anything but print the data that’s sent to the function and nothing else. The cpp file is actually empty, in fact.

Edit 2:

A bit early but I did some looking and getting a cross-build setup on my end running will take a bit. PopOS doesn’t have all of the packages I’m used to such as the configuration files for various build systems and I’d need to build the library packages manually for Windows so there will be a bit of manual setup (they do have MinGW-W64 stack so that helps). I’ll worry about it when we get to that point, though. VS should still work, though, as the project is already set up for that.

Edit 3:

Okay, finished up reformatting the whitespace in the File/ directory and pushed it and File/XML/ to the repo

Edit 4:

Still looking at the modifications to DevIL and, so far, it looks like it’s simply been changed to force the use of the Windows macros and the DLL support instead of letting that be handled at build time by choosing the options. If that holds, it looks like the modified version can be removed and handled by the stock versions of RevIL or DevIL with the options chosen at build time as designed.

Edit 5:

IL/ is done and pushed. After looking it over, I’m pretty sure the analysis in Edit 4 holds but I’ll take another look when I’m not so tired