mirror of
https://gitlab.freedesktop.org/gstreamer/gstreamer.git
synced 2024-11-27 04:01:08 +00:00
e40c284572
Original commit message from CVS: README and the beginning of the first code-review. Here's the README: Code reviews: ============= Files are to be named by file or subsystem, and CVS revision number or date: gstbin.c-1.41 editor-20001216 A file should look something like the following: ---------------------------------------------------------------------- Code Review =========== File: gst/gstbin.c Revision: 1.41 Date: Dec 16, 2000 Reviewer: Erik Walthinsen <omega@cse.ogi.edu> ----- Line 20: #define GST_DEBUG_ENABLED Shouldn't be here, DEBUG should be enabled globally. May leave until scheduling changes are done. ----- Line 117: (gst_bin_class_init) gstelement_class->elementfactory = gst_elementfactory_find("bin"); Not sure this is such a great idea. I thought the GstElement code did this kind of stuff? ---------------------------------------------------------------------- The format will evolve as we do more stuff, such as putting in fields for recommended actions, comments regarding any later changes made and when, etc. |
||
---|---|---|
.. | ||
gstbin.c-1.41 | ||
README |
Code reviews: ============= Files are to be named by file or subsystem, and CVS revision number or date: gstbin.c-1.41 editor-20001216 A file should look something like the following: ---------------------------------------------------------------------- Code Review =========== File: gst/gstbin.c Revision: 1.41 Date: Dec 16, 2000 Reviewer: Erik Walthinsen <omega@cse.ogi.edu> ----- Line 20: #define GST_DEBUG_ENABLED Shouldn't be here, DEBUG should be enabled globally. May leave until scheduling changes are done. ----- Line 117: (gst_bin_class_init) gstelement_class->elementfactory = gst_elementfactory_find("bin"); Not sure this is such a great idea. I thought the GstElement code did this kind of stuff? ---------------------------------------------------------------------- The format will evolve as we do more stuff, such as putting in fields for recommended actions, comments regarding any later changes made and when, etc.