This is the mail archive of the cygwin-xfree@cygwin.com mailing list for the Cygwin XFree86 project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: [patch] cursor


Jehan,

> A static would do the job but, by personnally, I don't like static in a
> function. It's easy to miss the "static" keyword when reading the code
> which then make it hard to understand (or at least, I missed it. Several
> times already I spent a few hours to try to understand why the code was
> working just because I missed it :p).

Okay, I have implemented a new convention: static variable names are now
prefixed with 's_', just like globals are prefixed with 'g_'.

I've updated all files in the hw/xwin directory and I'll submit that patch
to XFree86.

Harold


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]