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

Simon South updated THRIFT-2685:
--------------------------------
    Attachment: thrift-2685-3-c_glib-add-integration-test-suite-server.patch
                thrift-2685-2-c_glib-improve-integration-test-suite-client.patch

I've attached two more patches.

The first makes minor improvements to the client driver by

- Adding a missing "LL" suffix to a 64-bit literal (in the insanity-test code),

- Adding a missing call to {{g_type_init}} for compatibility with older 
versions of GLib, and

- Reformatting the code to better match [the GNOME (i.e. GLib) coding 
guidelines|https://help.gnome.org/users/programming-guidelines/stable/c-coding-style.html.en].

The second patch adds and integrates the server driver.

> c_glib: Include in integration test suite
> -----------------------------------------
>
>                 Key: THRIFT-2685
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2685
>             Project: Thrift
>          Issue Type: Test
>          Components: C glib - Library, Test Suite
>    Affects Versions: 0.9.1
>         Environment: Fedora 20 64-bit on x86_64
>            Reporter: Simon South
>            Priority: Minor
>              Labels: c_glib, integration-test
>         Attachments: 
> thrift-2685-2-c_glib-improve-integration-test-suite-client.patch, 
> thrift-2685-3-c_glib-add-integration-test-suite-server.patch, 
> thrift-2685-c_glib-add-integration-test-suite-client.patch
>
>
> The c_glib implementation should be included in the integration test suite.



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

Reply via email to