On Wed, 31 Mar 2010 at  4:39:46 +0200, Tamas TEVESZ wrote:
> Subject: [PATCH] Style stuff up

> diff --git a/util/fontconv.c b/util/fontconv.c
> index e470998..e3263b4 100644
> --- a/util/fontconv.c
> +++ b/util/fontconv.c

[...]

>  typedef struct str {
>       char *str;
> -     int len;
> +     size_t len;
>  } str;

I've just noticed that the change above introduces these warnings:

fontconv.c: In function ‘mapWeightToName’:
fontconv.c:114: warning: field precision should have type ‘int’, but argument 4 
has type ‘size_t’
fontconv.c:114: warning: field precision should have type ‘int’, but argument 4 
has type ‘size_t’
fontconv.c: In function ‘xlfdToFc’:
fontconv.c:157: warning: field precision should have type ‘int’, but argument 4 
has type ‘size_t’
fontconv.c:157: warning: field precision should have type ‘int’, but argument 4 
has type ‘size_t’

I'll revert this back to 'int' unless there is a strong reason not to. 
Comments anyone?


-- 
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.
  • style stuff Tamas TEVESZ
    • New gcc warning Carlos R. Mafra

Reply via email to