Warning fixes and new build system (Qbs) #36
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
DetachThread can't be static. InitializeDll.h doesn't exist. Returning const values makes no sense. Some missing standard headers.
Include <algorithm> for std::min and std::max Define NOMINMAX so this can work Missing header Tiny bit of _MSC_VER fixing for MinGW-w64 plans
Remove isinff(x) replacing it with !isfinite(x)
Added build script for Qt-Project/Digia's Qbs. Seems nice so far, though there's sure to be some problems as this is my first time using Qbs. Tested with Qt-Creator 2.8.1 Qbs Plugin on Linux (GCC), OSX (GCC), Windows (MSVC) and also with the stand-alone Qbs command-line program on Windows (v1.0.1, roughly): http://mingw-and-ndk.googlecode.com/files/qbs-1.0.1-win32-130901.7z
|
The Qbs file is a bit WIP as there's a specific thing I wanted to do involving capturing stdout from flex to redirect it to the output file which didn't work. I tried a few ways and also asked the Digia engineers who work on Qbs about it .. I'd quite like the commented out stuff to be retained until the issue is gotten to the bottom of, but do apologise for the untidiness of it in the meantime! |
Generated sources are put in Generated/ folder as this seems to work better. clang support for linux added (needs pthread explicitly added).
.. as sometimes requires that you remove the old build folder.
.. where appropriate.
|
Closing as I'm going to move it to another branch and tidy it up some. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.