-
Notifications
You must be signed in to change notification settings - Fork 370
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes to issue #25 #26
Conversation
fobo66
commented
Mar 21, 2016
- Updated project files to VS2015
- Little conditions fixes according to static analyser report at issue Checking with PVS-Studio static analyser #25
- Only obvious bugs fixed.
Project files are meant to be usable in VS2013 and VS2015, there is no need in updating them this way, because as you open the solution in VS2015 all the projects can be automatically updated in a few seconds. |
I've seen it just happen transparently with solution files as old as VS2010 when opened with VS2015 I believe. |
Alright, can .vcxproj files be converted back to VS2013 when I open the solution in current state with this version of VS? |
@fobo66 yes. You can open project with any version starting from 2010 SP1 (without service pack it didn't work). |
So, what's the problem? |
Those people that uses VS2013 will no longer be able to compile the projects if they are updated to VS2015 projects. From my VS2013: There was no such button to "upgrade" from 2015 to 2013. Searches on Google so far suggest that I have to manually edit the XML to downgrade to 2013. |
@yamgent what? On project settings you can change platform toolset to v130. There is no need to manually editing xml project files. |
Either way, VS2013 users will have to manually change project settings in order to build the projects. Why should we complicate things like that? Let's leave the old platform toolset version in the projects, so both VS2013 and VS2015 users can build the solution easily. |
Okay, close this PR. I'll send another one later, without .vcxproj modification. |
Some 64bit improvements, lots of 64bit TODOs/STUBBED()