[gstreamer-bugs] [Bug 337625] [patch] Streaming support for wavparse

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=337625
 GStreamer | gst-plugins-good | Ver: 0.10.2


Edward Hervey changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bilboed at bilboed.com




------- Comment #5 from Edward Hervey  2006-04-22 19:48 UTC -------
patch seems fine.

One comment though, this :
+ * Copyright (C) <2006> Nokia Corporation.

It would be more productive to put your actual name and email adress (possible
your nokia.com one) instead of just the company hiring you. Like:
+ * Copyright (C) <2006> Tommi Myohanen <ext-tommi.myohanen at nokia.com>

It's still puts your company there, but most importantly you, the person who
wrote that code and to which we can ask questions if bugs/issues about that
file are seen later on.


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