On 10/ 1/11 10:12 AM, zhigang gong wrote:
Alan,

According to your previous comments, I fixed those indentation and
coding style problems
except the camelCase function naming rule. Will do that soon after
this patch get reviewed
and committed.

As the patch is too big to attach in this mail, I created a new branch
to track it. Please check
it at: git://people.freedesktop.org/~gongzg/xserver-glamor glamor-style-fixes.
commit cb1aa7d3...

Thanks - at a quick glance that at least seems more consistent with itself.
(The X server has so many coding styles already we don't enforce one true style,
but we do ask that each file have a consistent style, and not do things like
using different amounts of space per indent level in different parts of the 
file.)

--
        -Alan Coopersmith-        [email protected]
         Oracle Solaris Platform Engineering: X Window System

_______________________________________________
[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