diff options
author | Rob Kendrick (humdrum) <rob.kendrick@codethink.co.uk> | 2013-04-25 15:13:36 +0100 |
---|---|---|
committer | Rob Kendrick (humdrum) <rob.kendrick@codethink.co.uk> | 2013-04-25 15:13:36 +0100 |
commit | fb5c3f3169d3a8d1e85626914b55825733b5376c (patch) | |
tree | fcbd30f7b9c369b0c1dbf1930d87eb8e6029843e /src/plot/16bpp.c | |
parent | 9aeab2e5176285583f8d669ba79ce6ff45b81a00 (diff) | |
download | libnsfb-rjek/clang-warnings.tar.gz libnsfb-rjek/clang-warnings.tar.bz2 |
Rationalise UNUSED macros. This changeset allows libnsfb to berjek/clang-warnings
built warning-free both with Clang 3.3 (from trunk) and GCC 4.7.
Remove the use of GCC-specific __attribute__ ((unused)) as this
only works on declarations, meaning there is no "fallback" to use
on compilers that are not GCC or clang. The (void)foo; approach
works with both clang and GCC, and hopefully other compilers
people might be using.
The alternative would to have been two UNUSED macros, one for use
in definitions, and one for use in function bodies.
Diffstat (limited to 'src/plot/16bpp.c')
-rw-r--r-- | src/plot/16bpp.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/plot/16bpp.c b/src/plot/16bpp.c index d629944..9aa7ab1 100644 --- a/src/plot/16bpp.c +++ b/src/plot/16bpp.c @@ -18,23 +18,23 @@ #include "nsfb.h" #include "plot.h" -#define UNUSED __attribute__((unused)) - static inline uint16_t *get_xy_loc(nsfb_t *nsfb, int x, int y) { return (void *)(nsfb->ptr + (y * nsfb->linelen) + (x << 1)); } -static inline nsfb_colour_t pixel_to_colour(UNUSED nsfb_t *nsfb, uint16_t pixel) +static inline nsfb_colour_t pixel_to_colour(nsfb_t *nsfb, uint16_t pixel) { + UNUSED(nsfb); return ((pixel & 0x1F) << 19) | ((pixel & 0x7E0) << 5) | ((pixel & 0xF800) >> 8); } /* convert a colour value to a 16bpp pixel value ready for screen output */ -static inline uint16_t colour_to_pixel(UNUSED nsfb_t *nsfb, nsfb_colour_t c) +static inline uint16_t colour_to_pixel(nsfb_t *nsfb, nsfb_colour_t c) { + UNUSED(nsfb); return ((c & 0xF8) << 8) | ((c & 0xFC00 ) >> 5) | ((c & 0xF80000) >> 19); } |