Hi,
2013/11/08 Fri 14:54:55 UTC+9 KF Leong wrote:
> On Friday, 8 November 2013 10:16:08 UTC+8, Bram Moolenaar wrote:
> > Patch 7.4.078
> >
> > Problem: MSVC 2013 is not supported.
> > Solution: Recognize and support MSVC 2013. (Ed Brown)
> > Files: src/Make_mvc.mak
> ...
> >
> > --- 488,494 ----
> > !endif
> >
> > # Static code analysis generally available starting with VS2012
> > ! !if ("$(ANALYZE)" == "yes") && ("$(MSVCVER)" == "11.0") && ("$(MSVCVER)"
> > == "12.0")
> > CFLAGS=$(CFLAGS) /analyze
> > !endif
> >
>
> The check should be Logical OR: (MSVCVER == "11.0" || MSVCVER == "12.0")
cl.exe Ver.16.00 shipped with Windows SDK 7.1 also supports /analyze option.
(The version number is same as VS2010, but VS2010 Express Edition doesn't
support /analyze. I don't know about VS2010Pro.)
I think it's better to change the line as:
!if ("$(ANALYZE)" == "yes") && (("$(MSVCVER)" == "10.0") || ("$(MSVCVER)" ==
"11.0") || ("$(MSVCVER)" == "12.0"))
or just remove the version checking:
!if "$(ANALYZE)" == "yes"
Regards,
Ken Takata
--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
---
You received this message because you are subscribed to the Google Groups
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
