[gstreamer-bugs] [Bug 613371] gst_element_class_set_details => gst_element_class_set_details_simple

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=613371
  GStreamer | gst-plugins-gl | git

--- Comment #3 from Filippo Argiolas <fargiolas at gnome.org> 2010-03-24 13:09:50 UTC ---
(In reply to comment #2)
> > Yep, we'll fix it sooner or later...
> > The indent script is not super perfect anyway, look what it does with the
> > fragment source in gstgldeinterlace.c e.g.
> > I never managed to completely replicate what it does with emacs... IIRC
> > correctly I even found an indent bug while trying to debug it [1].
> 
> For what it's worth, you can put markers like these into the code:
> 
> /* *INDENT-OFF* */
> 
> /* *INDENT-ON* */
> 
> to make indent skip it.

Oh nice, didn't know about it, we could use it at least for the fragment
sources.

> Benjamin: committed my own version of the patch, which is more or less the same
> as yours, sorry about that (I thought I had looked at yours and was under the
> impression that it contained indent problems, but it doesn't...)

Nope, the patch was ok, the indent problems are everywhere else :)
Hence I asked him to resubmit it in the git way to give him the proper credit.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.