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

Support compilation on Windows using MS Visual Studio compiler ("nmake") #4

Closed
wants to merge 2 commits into from

Conversation

fancycode
Copy link

The pull requests supports compiling the library and the "qrtest" sample on Windows using the MS compiler.

@dlbeer
Copy link
Owner

dlbeer commented Jan 8, 2013

On Tue, Jan 08, 2013 at 06:18:58AM -0800, Joachim Bauch wrote:

  • Support compilation on Windows using MS Visual Studio compiler ("nmake").
  • Remove local debug path.

Hi Joachim,

I think it would be good to add VC++ compatibility, but if you add
old-style initializers for data tables in the source, then the new-style
(designated initializers) are completely redundant -- if I modified the
latter, I'd have to modify the former anyway.

So, if I was going to do this, I'd remove the designated initializers
completely. However, I think that's a pretty big sacrifice in
maintainability to make for the sake of a single compiler which doesn't
support C99 and apparently never will.

MinGW is available as a compiler for Windows, and being based on GCC, it
does support C99. Quirc has quite a small API -- would it be possible
instead to add rules (and possibly some export-related definitions) to
build a library using MinGW that could be linked against by a VC++
project?

This would be a much smaller change and wouldn't require me to give up
the use of C99 features.

Regards,
Daniel

Daniel Beer dlbeer@gmail.com www.dlbeer.co.nz
IRC: inittab (Freenode) PGP key: 2048D/160A553B

@sagartesla
Copy link

I compiled this project on my VS 2010 with some modifications. But, now facing issue like How to run it for simple image_load and decode example? I am not able to load qr image.

@kaworu
Copy link
Collaborator

kaworu commented Jul 2, 2019

Closing as it is too old to merge as-is. If there is interest for VS support please create a new PR.

@kaworu kaworu closed this Jul 2, 2019
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.

4 participants