Code review comment for lp:~3v1n0/unity/lim-panel

Revision history for this message
Marco Trevisan (TreviƱo) (3v1n0) wrote :

> 8 +BuilderWrapper& BuilderWrapper::add(char const* name, long int value)
> 9 +{
> 10 + g_variant_builder_add(builder_, "{sv}", name,
> g_variant_new_int64(value));
>
> long is 32-bit on i386, if you really need 64bit value this should be changed
> to "long long".

Right. I've added wrappers for long long too, anyway I'm keeping the variant as a gint64 also for long int, to make sure it's working also on 64bit...

« Back to merge proposal