Code review comment for lp:~karl-qdh/ubuntu/maverick/libindicator/absfilename-ibus-bug564034

Revision history for this message
David Barth (dbarth) wrote :

THe general logic looks right.
Style issue: g_return_if_fail shouldn't be used since it would catch a runtime error (as opposed to a programming error).

« Back to merge proposal