widl [3/4]: Remove printf format strings that aren't really format strings.

Michael Stefaniuc mstefani at redhat.com
Thu Oct 18 04:12:23 CDT 2007


Dan Hipschman wrote:
> This is just a minor cleanup patch.  Even if we're definitely sure a
> string doesn't contain any % signs, it's still better not to use it as a
> printf format string.
fputs() would be a faster alternative to that. Though i doubt that
matters in this case.

bye
	michael

> ---
>  tools/widl/typegen.c |    4 ++--
>  tools/widl/widl.c    |    6 +++---
>  2 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/tools/widl/typegen.c b/tools/widl/typegen.c
> index bf24d3f..53ec78c 100644
> --- a/tools/widl/typegen.c
> +++ b/tools/widl/typegen.c
> @@ -2501,7 +2501,7 @@ void print_phase_basetype(FILE *file, int indent, enum remoting_phase phase,
>              fprintf(file, " *)_StubMsg.Buffer = *");
>          else
>              fprintf(file, " *)_StubMsg.Buffer = ");
> -        fprintf(file, varname);
> +        fprintf(file, "%s", varname);
>          fprintf(file, ";\n");
>      }
>      else if (phase == PHASE_UNMARSHAL)
> @@ -2510,7 +2510,7 @@ void print_phase_basetype(FILE *file, int indent, enum remoting_phase phase,
>              print_file(file, indent, "");
>          else
>              print_file(file, indent, "*");
> -        fprintf(file, varname);
> +        fprintf(file, "%s", varname);
>          if (pass == PASS_IN && is_ptr(type))
>              fprintf(file, " = (");
>          else
> diff --git a/tools/widl/widl.c b/tools/widl/widl.c
> index 53c9e29..37f1a5e 100644
> --- a/tools/widl/widl.c
> +++ b/tools/widl/widl.c
> @@ -422,13 +422,13 @@ int main(int argc,char *argv[])
>        idfile_name = xstrdup(optarg);
>        break;
>      case 'V':
> -      printf(version_string);
> +      printf("%s", version_string);
>        return 0;
>      case 'W':
>        pedantic = 1;
>        break;
>      default:
> -      fprintf(stderr, usage);
> +      fprintf(stderr, "%s", usage);
>        return 1;
>      }
>    }
> @@ -458,7 +458,7 @@ int main(int argc,char *argv[])
>        input_name = xstrdup(argv[optind]);
>    }
>    else {
> -    fprintf(stderr, usage);
> +    fprintf(stderr, "%s", usage);
>      return 1;
>    }
>  
> 
> 


-- 
Michael Stefaniuc               Tel.: +49-711-96437-199
Sr. Network Engineer            Fax.: +49-711-96437-111
--------------------------------------------------------------------
Reg. Adresse: Red Hat GmbH, Hauptstätter Strasse 58, 70178 Stuttgart
Handelsregister: Amtsgericht Stuttgart HRB 153243
Geschäftsführer: Brendan Lane, Charlie Peters, Michael Cunningham,
                 Werner Knoblich



More information about the wine-devel mailing list