[PATCH:libX11 1/2] Expand GetSizeOfFile() macro at the one place it's called

Julien Cristau jcristau at debian.org
Wed Feb 2 11:29:36 PST 2011


On Wed, Feb  2, 2011 at 10:18:43 -0800, Alan Coopersmith wrote:

> Removes XrmI.h header that only contained this single macro
> 
> Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
> ---
>  src/Makefile.am |    1 -
>  src/Xrm.c       |    9 +++++++--
>  src/XrmI.h      |   46 ----------------------------------------------
>  3 files changed, 7 insertions(+), 49 deletions(-)
>  delete mode 100644 src/XrmI.h
> 
On Wed, Feb  2, 2011 at 10:18:44 -0800, Alan Coopersmith wrote:

> We can simplify the fstat failure case now that the GetFileSize macro
> has been expanded inline.
> 
> Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
> ---
>  src/Xrm.c |   13 ++++---------
>  1 files changed, 4 insertions(+), 9 deletions(-)
> 
For both:
Reviewed-by: Julien Cristau <jcristau at debian.org>

Cheers,
Julien


More information about the xorg-devel mailing list