[pal-developer] VisualStudio Solution

Maxence Laurent maxence.laurent at gmail.com
Tue Dec 6 03:40:25 EST 2011


Hi Alexey,

Thanks for your patch. I just apply it and commit changes.

As far as I am not a VisualStudio user, I cannot asset the correctness
of your #ifdef statement.

Regards,
Maxence Laurent

On 17/11/11 15:36, Alexei Kolganov wrote:
> Greetings, Developers.
>
> Recently I have started integrating libPAL in my project and found
> that it does not include any MS Visual Studio solution (or any
> other convenient way) to build the library under Windows. Also, when I
> have started building the library I have faced some issues (all of
> them were minor, but still). Therefore I would like to submit a little
> patch which would allow new users of libPAL build the library easily.
>
> The patch is minor, it contains only the features required to build
> the library in my environment.
>
> It consists of:
> - Solution files
> - Changes to source files:
>   - Minor: eliminated trailing spaces in files that I have modified.
>   - Minor: eliminated few compiler warnings.
>   - Minor: fixed compilation issue with M_PI.
>
> One thing that, however, I am not sure is correct:
>   #ifdef _MSC_VER
>   #define _HAVE_WINDOWS_H_
>   #endif // _MSC_VER
> This code assumes that Windows SDK is always installed in MSVC, which,
> frankly, is not.
>
> The patch is submitted to Trac as Ticket #14 (
> http://geosysin.iict.ch/pal-trac/ticket/14 )
> If you have any suggestions or comments please don't hesitate to
> contact me.
>
> Best regards, Alexey Kolganov
>
>
> _______________________________________________
> pal-developer mailing list
> pal-developer at lists.osgeo.org
> http://lists.osgeo.org/mailman/listinfo/pal-developer

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.osgeo.org/pipermail/pal-developer/attachments/20111206/e4cd4d64/attachment.html


More information about the pal-developer mailing list