You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Roger Meier (JIRA)" <ji...@apache.org> on 2014/02/09 11:32:19 UTC

[jira] [Resolved] (THRIFT-2352) msvc failed to compile thrift tests

     [ https://issues.apache.org/jira/browse/THRIFT-2352?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Roger Meier resolved THRIFT-2352.
---------------------------------

    Resolution: Fixed
      Assignee: Roger Meier

committed including modification of lib/cpp/test/JSONProtoTest.cpp

;-r

> msvc failed to compile thrift tests
> -----------------------------------
>
>                 Key: THRIFT-2352
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2352
>             Project: Thrift
>          Issue Type: Bug
>          Components: C++ - Library
>    Affects Versions: 1.0
>         Environment: Windows7/Visual Studio 2012
>            Reporter: Sergei Nikulov
>            Assignee: Roger Meier
>              Labels: build, patch, test
>         Attachments: 0001-fixed-tests-compilation-on-windows.patch
>
>
> Patch related to THRIFT-2029 and fixes two test compilation issues with msvc.
> 1. Renamed small to tiny in test/DebugProtoTest.thrift
> Rationale: generated code contains bool small; which is conflicted with macro definition picked up from windows.h > rpcndr.h > "#define small char" (refer to http://connect.microsoft.com/VisualStudio/feedback/details/775845/poorly-picked-names-in-rpcndr-h-have-a-high-chance-of-collision and http://stackoverflow.com/questions/21165891/is-small-a-keyword-in-c and http://code.opencv.org/issues/1041)
> 2. Use THRIFT_GETTIMEOFDAY instead of gettimeofday in lib/cpp/test/Benchmark.cpp because of absence of latter on Windows



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)