[gstreamer-bugs] [Bug 487360] motorola contribution for gstreamer framework performance optimization

[GStreamer (bugzilla.gnome.org)]

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=487360

  GStreamer | gstreamer (core) | Ver: 0.10.13

Edward Hervey changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bilboed at bilboed.com
             Status|UNCONFIRMED                 |NEEDINFO




------- Comment #3 from Edward Hervey  2007-10-17 08:54 UTC -------
Hi, nice seeing patches coming from big corporations from Motorola !

The problem is that your patch is REALLY big and contains a lot of things.
Could you split up that patch into smaller parts explaining for each:
* what the patch does
* how it improves performance
* what it changes in current behaviour

Small comments already :
* We don't put change logs in the files themselves, but in the global ChangeLog
specifying what files/functions were changed (have a look at it for examples of
how it's used).
* We don't need patches for generated files like Makefile.in or configure, only
supply the modifications to Makefile.am or configure.ac.
* If you're adding configure options (like "OPTIMIZE"), document what it does


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=487360.