camerabin: code cleanups

Downgrade a WARNING to INFO. Use a bit more compact code. Add a fixme comment.
This commit is contained in:
Stefan Kost 2009-06-26 18:35:06 +03:00
parent 59bfe55110
commit 3d6d1c2ed2

View file

@ -500,8 +500,7 @@ camerabin_setup_src_elements (GstCameraBin * camera)
if (camera->fps_n > 0 && camera->fps_d > 0) { if (camera->fps_n > 0 && camera->fps_d > 0) {
if (camera->night_mode) { if (camera->night_mode) {
GST_WARNING_OBJECT (camera, GST_INFO_OBJECT (camera, "night mode, lowest allowed fps will be forced");
"night mode, lowest allowed fps will be forced");
camera->pre_night_fps_n = camera->fps_n; camera->pre_night_fps_n = camera->fps_n;
camera->pre_night_fps_d = camera->fps_d; camera->pre_night_fps_d = camera->fps_d;
detect_framerate = TRUE; detect_framerate = TRUE;
@ -863,6 +862,8 @@ camerabin_destroy_elements (GstCameraBin * camera)
static void static void
camerabin_dispose_elements (GstCameraBin * camera) camerabin_dispose_elements (GstCameraBin * camera)
{ {
GST_INFO ("cleaning");
if (camera->capture_mutex) { if (camera->capture_mutex) {
g_mutex_free (camera->capture_mutex); g_mutex_free (camera->capture_mutex);
camera->capture_mutex = NULL; camera->capture_mutex = NULL;
@ -885,24 +886,18 @@ camerabin_dispose_elements (GstCameraBin * camera)
camera->user_vid_src = NULL; camera->user_vid_src = NULL;
} }
/* Free caps */
if (camera->image_capture_caps) { if (camera->image_capture_caps) {
gst_caps_unref (camera->image_capture_caps); gst_caps_replace (&camera->image_capture_caps, NULL);
camera->image_capture_caps = NULL;
} }
if (camera->view_finder_caps) { if (camera->view_finder_caps) {
gst_caps_unref (camera->view_finder_caps); gst_caps_replace (&camera->view_finder_caps, NULL);
camera->view_finder_caps = NULL;
} }
if (camera->allowed_caps) { if (camera->allowed_caps) {
gst_caps_unref (camera->allowed_caps); gst_caps_replace (&camera->allowed_caps, NULL);
camera->allowed_caps = NULL;
} }
if (camera->preview_caps) { if (camera->preview_caps) {
gst_caps_unref (camera->preview_caps); gst_caps_replace (&camera->preview_caps, NULL);
camera->preview_caps = NULL;
} }
if (camera->event_tags) { if (camera->event_tags) {
@ -2753,7 +2748,6 @@ gst_camerabin_dispose (GObject * object)
gst_camerabin_preview_destroy_pipeline (camera); gst_camerabin_preview_destroy_pipeline (camera);
camerabin_destroy_elements (camera); camerabin_destroy_elements (camera);
camerabin_dispose_elements (camera); camerabin_dispose_elements (camera);
G_OBJECT_CLASS (parent_class)->dispose (object); G_OBJECT_CLASS (parent_class)->dispose (object);
@ -3152,6 +3146,8 @@ gst_camerabin_user_start (GstCameraBin * camera)
if (camera->capturing) { if (camera->capturing) {
GST_WARNING_OBJECT (camera, "capturing \"%s\" ongoing, set new filename", GST_WARNING_OBJECT (camera, "capturing \"%s\" ongoing, set new filename",
camera->filename->str); camera->filename->str);
/* FIXME: we need to send something more to the app, so that it does not for
* for img-done */
g_mutex_unlock (camera->capture_mutex); g_mutex_unlock (camera->capture_mutex);
return; return;
} }