[ https://issues.apache.org/jira/browse/THRIFT-5790?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Carel closed THRIFT-5790. ------------------------- Resolution: Invalid I see this was already fixed We are still using Thrift 0.17 and this was fixed in 0.19 https://github.com/apache/thrift/commit/4fcd0725b837dc65842638b9d2f5167bedd15399 > Thrift headers raise compiler warnings on MSVC > ---------------------------------------------- > > Key: THRIFT-5790 > URL: https://issues.apache.org/jira/browse/THRIFT-5790 > Project: Thrift > Issue Type: Bug > Environment: Windows > MSVC compiler > Reporter: Carel > Assignee: Carel > Priority: Major > > When building a C++ application using MSVC (2019) the following warnings are > raised: > [c4706|https://learn.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-4-c4706?view=msvc-160] > {code} > <...>\include\thrift\protocol\TBinaryProtocol.tcc(453) : warning C4706: > assignment within conditional expression > {code} > This makes it impossible to build code using the warnings as errors option. > One option could have been to mark the headers as external as discussed > [here|https://learn.microsoft.com/en-us/cpp/build/reference/external-external-headers-diagnostics?view=msvc-160] > but one should see on that page under Limitations: > bq. Some warnings emitted by the compiler's back-end code generation aren't > affected by the /external options. These warnings usually start with C47XX, > though not all C47XX warnings are back-end warnings. You can still disable > these warnings individually by using /wd47XX. Code analysis warnings are also > unaffected, since they don't have warning levels. > And as one would guess these all fall under that limitation. > Disabling the warning globally does work, but is not ideal for hopefully > obvious reasons. -- This message was sent by Atlassian Jira (v8.20.10#820010)