wmc: Fix symbol was not declared and using plain integer as NULL pointer sparse warnings.

Jacek Caban jacek at codeweavers.com
Thu Jan 21 06:13:48 CST 2010


Hi Rob,

On 1/21/10 12:57 PM, Rob Shearman wrote:
> ---
>   tools/wmc/wmc.c |    8 +++++---
>   tools/wmc/wmc.h |    2 +-
>   2 files changed, 6 insertions(+), 4 deletions(-)
>    
--- a/tools/wmc/wmc.h
+++ b/tools/wmc/wmc.h
@@ -65,7 +65,7 @@ extern node_t *nodehead;
  extern lan_blk_t *lanblockhead;

  int mcy_lex(void);
-FILE *yyin;
+extern FILE *yyin;


It looks like it worked before your patch only because it's used only in 
wmc.c, which means that it probably should be static.

Thanks,
     Jacek



More information about the wine-devel mailing list