[x264-devel] [PATCH] fix more illegal identifiers

[Guillaume POIRIER]

On 4/25/08, Diego Biurrun <diego at biurrun.de> wrote:
> On Thu, Apr 24, 2008 at 09:24:37PM +0200, Guillaume POIRIER wrote:
>  >
>  > On Thu, Apr 24, 2008 at 9:21 PM, Måns Rullgård <mans at mansr.com> wrote:
>  > > Diego Biurrun <diego at biurrun.de> writes:
>  > >
>  > >  > OK, here is the next patch to do $subject.
>  > >  >
>
> > >  > --- a/gtk/x264_gtk.c
>  > >  > +++ b/gtk/x264_gtk.c
>  > >  > @@ -330,7 +330,7 @@ x264_gtk_free (X264_Gtk *x264_gtk)
>  > >  >  /* Callbacks */
>  > >  >
>  > >  >  static void
>  > >  > -_dialog_run (GtkDialog       *dialog __UNUSED__,
>  > >  > +_dialog_run (GtkDialog       *dialog UNUSED,
>  > >
>  > >  _dialog_run and similar names are also bad, since all identifiers
>  > >  beginning with _ are reserved at the file scope.  _[a-z] is allowed
>  > >  for local variables and struct/union members.
>  >
>  > Yep, I noticed that too, I just figured that Diego had a separate
>  > patch for that issue.
>
>
> No, I haven't, so feel free to beat me to it :)

Done, see attached patch. It should be Ok since it compiles OK and
seems to run likewise. Please someone have a look at it, one never
knows, a bug may have got in.

Guillaume
-- 
I don't measure a man's success by how high he climbs but how high he
bounces when he hits bottom.
 -- George S. Patton
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Fix-define-of-illegal-function-identifiers-as-defin.patch
Type: application/octet-stream
Size: 39453 bytes
Desc: not available
Url : http://mailman.videolan.org/pipermail/x264-devel/attachments/20080425/b16b27e0/attachment-0001.obj