On Wed, Oct 20, 2010 at 1:11 PM, Julien Cristau <[email protected]> wrote:
> +  static xcb_key_symbols_t *keysyms;
> +
> +  if (!keysyms)
> +         keysyms = xcb_key_symbols_alloc(HostX.conn);

I realize static variables are assigned from zero'ed areas, but isn't
it nicer to explicitly say "static xcb_key_symbols_t *keysyms =
NULL;"? Just a nit, I guess.
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to