Skip to content
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

make VS2010 happy #165

Closed
wants to merge 8 commits into from
Closed

make VS2010 happy #165

wants to merge 8 commits into from

Conversation

ssrlive
Copy link

@ssrlive ssrlive commented May 8, 2018

make VS2010 happy

@msftclas
Copy link

msftclas commented May 8, 2018

CLA assistant check
All CLA requirements met.

@clzls
Copy link
Contributor

clzls commented May 9, 2018

It may be the same idea of #158 . Either we remove all 'for' loop initial declarations, etc. to fit ISO C90/GNU C90 or lower versions, or we choose to adopt C99 standard and not to satisfy VS2010.

@ssrlive
Copy link
Author

ssrlive commented May 9, 2018

I think my commits fit ISO C90.

@clzls
Copy link
Contributor

clzls commented May 9, 2018

@ssrlive , I mean that we should decide whether to fit C90 or send it to the trash and adopt C99.
In C99, for(int i = 1;... is accepted while in C90 not. If we adopt C99, we do not need these commits (maybe the whole PR) any more.

And I may call @craigwi to help us.

@ssrlive
Copy link
Author

ssrlive commented May 9, 2018

I think it's easy to make the decision. because the code base will not change many and supporting more compilers is a good idea.

@craigwi
Copy link
Contributor

craigwi commented May 9, 2018

This PR is also similar to #59. The current thinking is that the master branch is compiled using one of two toolsets: VS2017 and mingw.

Right now, it would be easy to make VS2010 work, but the question is whether going forward we support VS2010 indefinitely.

I agree with a comment @leeter made in #9 and elsewhere: let's stick with modern compilers to get the best code generation, optimization and security features.

Other comments?

@clzls
Copy link
Contributor

clzls commented May 9, 2018

I second the motion. We should send C90, MinGW 3.x, VS2010s into the trash.
(But I cannot compile it in my VS2017 Community. I bumped into "internal errors" in BagOValues.h)

@ssrlive ssrlive closed this May 9, 2018
@ssrlive ssrlive deleted the patch-1 branch May 9, 2018 11:50
@leeter
Copy link

leeter commented May 9, 2018

@craigwi I wasn't planning on commenting on this but if you're going to go this route you should consider removing the dependency on undocumented apis for formatting and copying volumes and switch to WMI to manage that depending on what OSs you still want to support. The storage management classes are the official way to do these things in windows 8 and beyond.

@ssrlive
Copy link
Author

ssrlive commented May 9, 2018

I have finished the work for Windows XP and VS2010. https://github.com/ssrlive/winfile/tree/winxp
Bye bye.

@clzls
Copy link
Contributor

clzls commented May 9, 2018

@ssrlive , WinXP support is in retro brunch. Thank you anyway.
Maybe you should read #88 before you start working. See you next time.

@malxau malxau mentioned this pull request Mar 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants