Review Request: Fix standard include dirs and macros under Windows

[Andreas Holzammer]

> On March 5, 2011, 5:47 p.m., Milian Wolff wrote:
> > languages/cpp/msvcdefinehelper.cpp, line 26
> > <http://git.reviewboard.kde.org/r/100801/diff/1/?file=10583#file10583line26>
> >
> >     a comment explaining why this file is required and what it does (basically what you wrote in the review description) would be good

yes i have added it in the new diff.


> On March 5, 2011, 5:47 p.m., Milian Wolff wrote:
> > languages/cpp/setuphelpers.cpp, line 36
> > <http://git.reviewboard.kde.org/r/100801/diff/1/?file=10584#file10584line36>
> >
> >     isn't this missing the extern keyword?
> >     
> >     and I'd personally use the #if MSC_VER here already

no there is no extern needed, because its in the same library. the extern keyword is only needed if it is in diffrent libs. Then i put this in ifdefs...


> On March 5, 2011, 5:47 p.m., Milian Wolff wrote:
> > languages/cpp/setuphelpers.cpp, line 43
> > <http://git.reviewboard.kde.org/r/100801/diff/1/?file=10584#file10584line43>
> >
> >     make this a ///TODO: ...

done in new diff


- Andreas


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100801/#review1779
-----------------------------------------------------------


On March 6, 2011, 10:25 a.m., Andreas Holzammer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100801/
> -----------------------------------------------------------
> 
> (Updated March 6, 2011, 10:25 a.m.)
> 
> 
> Review request for KDevelop, kdewin, Milian Wolff, and Patrick Spendrin.
> 
> 
> Summary
> -------
> 
> This patch enables Kdevelop to find the include directories and standard macros under Windows.
> 
> There is no "gcc -dM -E -" under Windows. So you need to do a hack to find the standard defined macros. Therefore a empty cpp file is compiled with a special commandline option "/Bx" to replace the compiler against a program which interpretes the output of the preprocessor.
> 
> For include directories Windows uses enviroment variables, so this patch processes them.
> 
> This patch has been written by Patrick Spenderin, thank you for that.
> 
> 
> Diffs
> -----
> 
>   languages/cpp/CMakeLists.txt e7c3c0c07ac3ac0ad5b49c60b0d0c279b77e8e75 
>   languages/cpp/msvcdefinehelper.cpp PRE-CREATION 
>   languages/cpp/parser/rpp/chartools.cpp 7fecf1ef2df8ce7e52f8afd2f847dfc746ecd2fc 
>   languages/cpp/setuphelpers.cpp b35ec8386a2c04a7caa421be9bf8e0d268f1213a 
>   languages/cpp/setuphelpers_gcc.cpp PRE-CREATION 
>   languages/cpp/setuphelpers_msvc.cpp PRE-CREATION 
>   languages/cpp/tests/CMakeLists.txt e3b7add33e9f89b96ae8f86da3cc240647cd99a4 
>   projectmanagers/cmake/cmakedocumentation.cpp b4679051fb3ad3e9a4a664f79a1844058f4d8fb5 
> 
> Diff: http://git.reviewboard.kde.org/r/100801/diff
> 
> 
> Testing
> -------
> 
> tested it with Windows 7 Ultimate x64 with KDE Windows trunk compiled with MSVC 2010. Works so far.
> 
> 
> Thanks,
> 
> Andreas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20110306/e93a9007/attachment.html>