[Pixman] [PATCH] pixman.c: Coding style
Pekka Paalanen
ppaalanen at gmail.com
Fri Apr 10 07:07:24 PDT 2015
On Thu, 9 Apr 2015 12:07:20 +0300
Pekka Paalanen <ppaalanen at gmail.com> wrote:
> On Wed, 8 Apr 2015 14:20:30 +0100
> Ben Avison <bavison at riscosopen.org> wrote:
>
> > A few violations of coding style were identified in code copied from here
> > into affine-bench.
> > ---
> > pixman/pixman.c | 18 ++++++++++--------
> > 1 files changed, 10 insertions(+), 8 deletions(-)
> >
> > diff --git a/pixman/pixman.c b/pixman/pixman.c
> > index 9555cea..a07c577 100644
> > --- a/pixman/pixman.c
> > +++ b/pixman/pixman.c
> > @@ -325,18 +325,20 @@ _pixman_compute_composite_region32 (pixman_region32_t * region,
> > return TRUE;
> > }
> >
> > -typedef struct
> > +typedef struct box_48_16 box_48_16_t;
> > +
> > +struct box_48_16
> > {
> > - pixman_fixed_48_16_t x1;
> > - pixman_fixed_48_16_t y1;
> > - pixman_fixed_48_16_t x2;
> > - pixman_fixed_48_16_t y2;
> > -} box_48_16_t;
> > + pixman_fixed_48_16_t x1;
> > + pixman_fixed_48_16_t y1;
> > + pixman_fixed_48_16_t x2;
> > + pixman_fixed_48_16_t y2;
> > +};
> >
> > static pixman_bool_t
> > -compute_transformed_extents (pixman_transform_t *transform,
> > +compute_transformed_extents (pixman_transform_t *transform,
> > const pixman_box32_t *extents,
> > - box_48_16_t *transformed)
> > + box_48_16_t *transformed)
> > {
> > pixman_fixed_48_16_t tx1, ty1, tx2, ty2;
> > pixman_fixed_t x1, y1, x2, y2;
>
> Reviewed-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
>
> I'll push this on Friday unless anyone objects. I heard I should have
> push access now. This is a nice small patch to test my pushing.
Pushed,
978dd9f..9bc025f master -> master
Thanks,
pq
More information about the Pixman
mailing list