diff options
author | Kristian Høgsberg <krh@redhat.com> | 2008-03-11 13:11:04 -0400 |
---|---|---|
committer | Kristian Høgsberg <krh@redhat.com> | 2008-03-11 13:18:01 -0400 |
commit | c7536f4b87e089a7e7c43026b189922fec70c565 (patch) | |
tree | 6efd8c1276ddc0c010f376fa8af9129176003ec7 /include | |
parent | cc05255191413b3f376edbc600122ff085f45f7b (diff) |
Silence REGION_INIT() warning.
Evaluating the address of a BoxRec as a boolean gives this warning:
i830_driver.c:2317: warning: the address of 'ScreenBox' will always
evaluate as 'true'
which is pretty annoying. This patch compares the address to NULL to
avoid the pointer->bool conversion and gets rid of the warning. Seems
like a lame hack, but the warning is worse.
Diffstat (limited to 'include')
-rw-r--r-- | include/regionstr.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/regionstr.h b/include/regionstr.h index f44cab7b0..5a79b1228 100644 --- a/include/regionstr.h +++ b/include/regionstr.h @@ -158,7 +158,7 @@ extern RegDataRec miBrokenData; #define REGION_INIT(_pScreen, _pReg, _rect, _size) \ { \ - if (_rect) \ + if ((_rect) != NULL) \ { \ (_pReg)->extents = *(_rect); \ (_pReg)->data = (RegDataPtr)NULL; \ |