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

Roger Meier commented on THRIFT-1003:
-------------------------------------

my env on Debian Lenny
{code}
ltmain.sh (GNU libtool) 1.5.26 Debian 1.5.26-4+lenny1 (1.1220.2.493 2008/02/01 
16:58:18)
automake (GNU automake) 1.10.1
autoconf (GNU Autoconf) 2.65
{code}

same behavior with new libtool(2.2.6b) from backports {{sudo apt-get -t 
lenny-backports install libtool}}

> Polishing c_glib code
> ---------------------
>
>                 Key: THRIFT-1003
>                 URL: https://issues.apache.org/jira/browse/THRIFT-1003
>             Project: Thrift
>          Issue Type: Improvement
>            Reporter: Anatol Pomozov
>         Attachments: c_glib_fixes.diff
>
>
> attached patch contains following changes:
> * Added Apache headers to c/h files
> * Use gtester for running tests. We don't need -wrapper script anymore
> * Use one-line macros G_DEFINE_TYPE instead of 15-line class definition
> * Keep formatting closer to glib-like style (one line class definition 
> macroses/remove trailing spaces)
> Given changes are mostly fixing low hanging fruits. It does not change any 
> logic/api.
> There are more chages needed, such as 
>  * using CLASS_TYPE_new functions instead of g_object_new(CLASS_TYPE) 
>  * stop using _set_property (aka reflection) in constructors
>  * check more careful about _ref and _unref handling 
> but this requires more careful refactoring so it will be later in a separate 
> patch.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to