[ 
https://issues.apache.org/jira/browse/THRIFT-3288?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14651753#comment-14651753
 ] 

Simon South commented on THRIFT-3288:
-------------------------------------

Note the unit tests actually embed certain files from the library, so the 
library needs to build without warnings first.

> c_glib: Build unit tests with all warnings enabled, no warnings generated
> -------------------------------------------------------------------------
>
>                 Key: THRIFT-3288
>                 URL: https://issues.apache.org/jira/browse/THRIFT-3288
>             Project: Thrift
>          Issue Type: Improvement
>          Components: C glib - Library
>    Affects Versions: 0.9.3
>         Environment: Cloud9 (Ubuntu 14.04.2 LTS; gcc 4.8.2)
>            Reporter: Simon South
>            Assignee: Simon South
>            Priority: Minor
>              Labels: c_glib, unit-test
>
> For correctness and to help with tracking down bugs, the C (GLib) unit tests 
> (in {{lib/c_glib/test}}) should build with the same compiler-warning options 
> enabled for the C++ library's tests ({{-Wall -Wextra -pedantic}}) and without 
> any warnings generated.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to