Road to pixman 0.16.0

Edward O'Callaghan eocallaghan at auroraux.org
Tue Jul 14 03:29:34 PDT 2009


Hi,

Just wanted to give me input on the choice of code style. That is, I
believe you should use if (..) {  instead, for two reason right off
the top of my head.

A.) Does not pointless waste screen real estate; Allowing someone to
see more of the code block at once, instead of scrolling all the time.
B.) Its clear that there is indeed a '{' following that if statement
and its indeed clear that this '{' is part of that if statement.

Any way, you may or may not agree however this is my two cent.

Thanks for listening,
Edward.

2009/7/14 Soeren Sandmann <sandmann at daimi.au.dk>:
> Hi,
>
> Pixman 0.15.16 contains the last bits of refactoring, so from now on,
> there will be a coding style enforced. The coding style is similar to
> cairo's, with the main difference being that braces go on their own
> line always. Ie.,
>
>        if (...)
>        {
>        }
>
> and not
>
>        if (...) {
>        }
>
> The cairo coding style document is here:
>
>        http://cgit.freedesktop.org/cairo/tree/CODING_STYLE
>
> Here is a rough road map for the remaining things before 0.16.0:
>
> * Disable region self-check
>
> * MSVC fixes.
>
>   - The Makefile.win32 broke as Rainman Lee pointed out.
>
>   - There is this bug:
>
>       https://bugs.freedesktop.org/show_bug.cgi?id=22390
>
>     with fixes for sse2 on x64. The mmx file there seems sane to me,
>     but it would be good if someone with access to Windows can verify
>     that it work.
>
>  Someone with access to this compiler, please help out if you can.
>
> * ARM fixes
>
>  - The current code is not releasable.
>
>    Unless I hear otherwise, I'll just apply Siarhei's fixes after
>    they are updated for code style issues.
>
>  - Build fixes for older CPUs:
>
>        https://bugs.freedesktop.org/show_bug.cgi?id=22692
>
>    Siarhei says it's reasonable, so unless I hear otherwise, I'll
>    apply that as well.
>
> * Merge the blitter-test code.
>
>  - Some changes are needed; see other mail.
>
> * Fix bugs reported by the blitter-test
>
>  - The discrepancies between the architecture specific code and the
>    general C code should be tracked down and fixed.
>
>  - It should be tracked down why enabling component-alpha causes
>    crashes.
>
> * Fix performance regressions.
>
>  There have been reports of performance regressions. The ones I am
>  aware of are:
>
>  - Image allocation may have become slower
>
>  - _pixman_run_fast_path() may have too much overhead
>
>  - Delegates may have too much overhead
>
>  - Carl reported that the swfdec-fill-rate-2xaa perf-trace had a 25%
>    performance regression between 0.15.1 and 0.15.13
>
> Let me know if I missed anything.
>
> There will be at least one, and probably two, development releases
> before 0.16.0. I'll be spending a bit less time on pixman in the
> coming weeks, so help is appreciated with the above items.
>
>
> Thanks,
> Soren
> _______________________________________________
> xorg-devel mailing list
> xorg-devel at lists.x.org
> http://lists.x.org/mailman/listinfo/xorg-devel
>



-- 
-- 
Edward O'Callaghan
http://www.auroraux.org/
eocallaghan at auroraux dot org


More information about the xorg-devel mailing list