diff options
author | patacongo <patacongo@42af7a65-404d-4744-a932-0658087f49c3> | 2012-05-10 16:25:56 +0000 |
---|---|---|
committer | patacongo <patacongo@42af7a65-404d-4744-a932-0658087f49c3> | 2012-05-10 16:25:56 +0000 |
commit | 68f3044b5099572850a9e81a1c63161c20809547 (patch) | |
tree | 1160b821f035a4e90a5ed6472800e6d7e03468b8 /nuttx/graphics | |
parent | 2bfda8161a877d7e618a37353f806981c0ab4a4b (diff) | |
download | px4-nuttx-68f3044b5099572850a9e81a1c63161c20809547.tar.gz px4-nuttx-68f3044b5099572850a9e81a1c63161c20809547.tar.bz2 px4-nuttx-68f3044b5099572850a9e81a1c63161c20809547.zip |
NX trapezoid fixes; Various NxWM fixes
git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@4720 42af7a65-404d-4744-a932-0658087f49c3
Diffstat (limited to 'nuttx/graphics')
-rw-r--r-- | nuttx/graphics/nxbe/nxbe_filltrapezoid.c | 8 | ||||
-rw-r--r-- | nuttx/graphics/nxglib/fb/nxglib_filltrapezoid.c | 16 | ||||
-rw-r--r-- | nuttx/graphics/nxglib/nxglib_bitblit.h | 36 | ||||
-rw-r--r-- | nuttx/graphics/nxglib/nxglib_nullrect.c | 4 | ||||
-rw-r--r-- | nuttx/graphics/nxmu/nx_filltrapezoid.c | 19 |
5 files changed, 48 insertions, 35 deletions
diff --git a/nuttx/graphics/nxbe/nxbe_filltrapezoid.c b/nuttx/graphics/nxbe/nxbe_filltrapezoid.c index c3c95d22b..68dad838c 100644 --- a/nuttx/graphics/nxbe/nxbe_filltrapezoid.c +++ b/nuttx/graphics/nxbe/nxbe_filltrapezoid.c @@ -1,8 +1,8 @@ /**************************************************************************** * graphics/nxbe/nxbe_filltrapezoid.c * - * Copyright (C) 2008-2009 Gregory Nutt. All rights reserved. - * Author: Gregory Nutt <spudmonkey@racsa.co.cr> + * Copyright (C) 2008-2009, 2012 Gregory Nutt. All rights reserved. + * Author: Gregory Nutt <gnutt@nuttx.org> * * Redistribution and use in source and binary forms, with or without * modification, are permitted provided that the following conditions @@ -140,7 +140,7 @@ void nxbe_filltrapezoid(FAR struct nxbe_window_s *wnd, /* Clip to any user specified clipping window */ - if (clip && !nxgl_nullrect(clip)) + if (clip) { struct nxgl_rect_s tmp; nxgl_rectoffset(&tmp, clip, wnd->bounds.pt1.x, wnd->bounds.pt1.y); @@ -157,8 +157,6 @@ void nxbe_filltrapezoid(FAR struct nxbe_window_s *wnd, info.cops.visible = nxbe_clipfilltrapezoid; info.cops.obscured = nxbe_clipnull; - nxgl_trapcopy(&info.trap, trap); - /* Then process each color plane */ #if CONFIG_NX_NPLANES > 1 diff --git a/nuttx/graphics/nxglib/fb/nxglib_filltrapezoid.c b/nuttx/graphics/nxglib/fb/nxglib_filltrapezoid.c index ddaa0f9bf..64554fae1 100644 --- a/nuttx/graphics/nxglib/fb/nxglib_filltrapezoid.c +++ b/nuttx/graphics/nxglib/fb/nxglib_filltrapezoid.c @@ -93,6 +93,7 @@ void NXGL_FUNCNAME(nxgl_filltrapezoid,NXGLIB_SUFFIX)( { unsigned int stride; unsigned int width; + FAR uint8_t *dest; FAR uint8_t *line; int nrows; b16_t x1; @@ -103,7 +104,6 @@ void NXGL_FUNCNAME(nxgl_filltrapezoid,NXGLIB_SUFFIX)( b16_t dx2dy; #if NXGLIB_BITSPERPIXEL < 8 - FAR uint8_t *dest; uint8_t mpixel = NXGL_MULTIPIXEL(color); uint8_t mask; int lnlen; @@ -191,11 +191,10 @@ void NXGL_FUNCNAME(nxgl_filltrapezoid,NXGLIB_SUFFIX)( ngl_swap(dx1dy, dx2dy, tmp); } - /* Convert the positions to integer and get the run width */ + /* Convert the positions to integer */ - ix1 = b16toi(x1); - ix2 = b16toi(x2); - width = ix2 - ix1 + 1; + ix1 = b16toi(x1); + ix2 = b16toi(x2); /* Handle some corner cases where we draw nothing. Otherwise, we will * always draw at least one pixel. @@ -211,6 +210,10 @@ void NXGL_FUNCNAME(nxgl_filltrapezoid,NXGLIB_SUFFIX)( ix1 = ngl_clipl(ix1, bounds->pt1.x); ix2 = ngl_clipr(ix2, bounds->pt2.x); + /* Get the run length for the clipped row */ + + width = ix2 - ix1 + 1; + #if NXGLIB_BITSPERPIXEL < 8 /* Handle masking of the fractional initial byte */ @@ -253,7 +256,8 @@ void NXGL_FUNCNAME(nxgl_filltrapezoid,NXGLIB_SUFFIX)( #else /* Then draw the run from (line + ix1) to (line + ix2) */ - NXGL_MEMSET(line + NXGL_SCALEX(ix1), (NXGL_PIXEL_T)color, width); + dest = line + NXGL_SCALEX(ix1); + NXGL_MEMSET(dest, (NXGL_PIXEL_T)color, width); #endif } diff --git a/nuttx/graphics/nxglib/nxglib_bitblit.h b/nuttx/graphics/nxglib/nxglib_bitblit.h index a272b284c..a737a0647 100644 --- a/nuttx/graphics/nxglib/nxglib_bitblit.h +++ b/nuttx/graphics/nxglib/nxglib_bitblit.h @@ -111,17 +111,17 @@ # define NXGL_MEMSET(dest,value,width) \ { \ - FAR uint8_t *_ptr = (FAR uint8_t*)dest; \ + FAR uint8_t *_ptr = (FAR uint8_t*)(dest); \ int _nby = NXGL_SCALEX(width); \ while (_nby--) \ { \ - *_ptr++ = value; \ + *_ptr++ = (value); \ } \ } # define NXGL_MEMCPY(dest,src,width) \ { \ - FAR uint8_t *_dptr = (FAR uint8_t*)dest; \ - FAR uint8_t *_sptr = (FAR uint8_t*)src; \ + FAR uint8_t *_dptr = (FAR uint8_t*)(dest); \ + FAR uint8_t *_sptr = (FAR uint8_t*)(src); \ int _nby = NXGL_SCALEX(width); \ while (_nby--) \ { \ @@ -132,20 +132,20 @@ #elif NXGLIB_BITSPERPIXEL == 24 # define NXGL_MEMSET(dest,value,width) \ { \ - FAR uint8_t *_ptr = (FAR uint8_t*)dest; \ - nxgl_coord_t _npix = width; \ + FAR uint8_t *_ptr = (FAR uint8_t*)(dest); \ + nxgl_coord_t _npix = (width); \ while (_npix--) \ { \ - *_ptr++ = value; \ - *_ptr++ = value >> 8; \ - *_ptr++ = value >> 16; \ + *_ptr++ = (value); \ + *_ptr++ = (value) >> 8; \ + *_ptr++ = (value) >> 16; \ } \ } # define NXGL_MEMCPY(dest,src,width) \ { \ - FAR uint8_t *_dptr = (FAR uint8_t*)dest; \ - FAR uint8_t *_sptr = (FAR uint8_t*)src; \ - nxgl_coord_t _npix = width; \ + FAR uint8_t *_dptr = (FAR uint8_t*)(dest); \ + FAR uint8_t *_sptr = (FAR uint8_t*)(src); \ + nxgl_coord_t _npix = (width); \ while (_npix--) \ { \ *_dptr++ = *_sptr++; \ @@ -156,18 +156,18 @@ #else # define NXGL_MEMSET(dest,value,width) \ { \ - FAR NXGL_PIXEL_T *_ptr = (FAR NXGL_PIXEL_T*)dest; \ - nxgl_coord_t _npix = width; \ + FAR NXGL_PIXEL_T *_ptr = (FAR NXGL_PIXEL_T*)(dest); \ + nxgl_coord_t _npix = (width); \ while (_npix--) \ { \ - *_ptr++ = value; \ + *_ptr++ = (value); \ } \ } # define NXGL_MEMCPY(dest,src,width) \ { \ - FAR NXGL_PIXEL_T *_dptr = (FAR NXGL_PIXEL_T*)dest; \ - FAR NXGL_PIXEL_T *_sptr = (FAR NXGL_PIXEL_T*)src; \ - nxgl_coord_t _npix = width; \ + FAR NXGL_PIXEL_T *_dptr = (FAR NXGL_PIXEL_T*)(dest); \ + FAR NXGL_PIXEL_T *_sptr = (FAR NXGL_PIXEL_T*)(src); \ + nxgl_coord_t _npix = (width); \ while (_npix--) \ { \ *_dptr++ = *_sptr++; \ diff --git a/nuttx/graphics/nxglib/nxglib_nullrect.c b/nuttx/graphics/nxglib/nxglib_nullrect.c index 3e3355a45..26fc9e8d7 100644 --- a/nuttx/graphics/nxglib/nxglib_nullrect.c +++ b/nuttx/graphics/nxglib/nxglib_nullrect.c @@ -1,8 +1,8 @@ /**************************************************************************** * graphics/nxglib/nxsglib_nullrect.c * - * Copyright (C) 2008-2011 Gregory Nutt. All rights reserved. - * Author: Gregory Nutt <spudmonkey@racsa.co.cr> + * Copyright (C) 2008-2012 Gregory Nutt. All rights reserved. + * Author: Gregory Nutt <gnutt@nuttx.org> * * Redistribution and use in source and binary forms, with or without * modification, are permitted provided that the following conditions diff --git a/nuttx/graphics/nxmu/nx_filltrapezoid.c b/nuttx/graphics/nxmu/nx_filltrapezoid.c index 8708225d3..f0a55fc62 100644 --- a/nuttx/graphics/nxmu/nx_filltrapezoid.c +++ b/nuttx/graphics/nxmu/nx_filltrapezoid.c @@ -1,8 +1,8 @@ /**************************************************************************** * graphics/nxmu/nx_filltrapezoid.c * - * Copyright (C) 2008-2009, 2011 Gregory Nutt. All rights reserved. - * Author: Gregory Nutt <spudmonkey@racsa.co.cr> + * Copyright (C) 2008-2009, 2011-2012 Gregory Nutt. All rights reserved. + * Author: Gregory Nutt <gnutt@nuttx.org> * * Redistribution and use in source and binary forms, with or without * modification, are permitted provided that the following conditions @@ -98,6 +98,8 @@ int nx_filltrapezoid(NXWINDOW hwnd, FAR const struct nxgl_rect_s *clip, int ret; int i; + /* Some debug-only sanity checks */ + #ifdef CONFIG_DEBUG if (!wnd || !wnd->conn || !trap || !color) { @@ -111,22 +113,31 @@ int nx_filltrapezoid(NXWINDOW hwnd, FAR const struct nxgl_rect_s *clip, outmsg.msgid = NX_SVRMSG_FILLTRAP; outmsg.wnd = wnd; + /* If no clipping window was provided, then use the size of the entire window */ + if (clip) { nxgl_rectcopy(&outmsg.clip, clip); } else { - memset(&outmsg.clip, 0, sizeof(struct nxgl_rect_s)); + nxgl_rectcopy(&outmsg.clip, &wnd->bounds); } + + /* Copy the trapezod and the color into the message */ + nxgl_trapcopy(&outmsg.trap, trap); +#if CONFIG_NX_NPLANES > 1 for (i = 0; i < CONFIG_NX_NPLANES; i++) +#else + i = 0; +#endif { outmsg.color[i] = color[i]; } - /* Forward the fill command to the server */ + /* Forward the trapezoid fill command to the server */ ret = mq_send(wnd->conn->cwrmq, &outmsg, sizeof(struct nxsvrmsg_filltrapezoid_s), NX_SVRMSG_PRIO); |