openni2src: Fix unitialized variable compiler warning

gstopenni2src.cpp:721:14: error: variable 'oni_ts' is used uninitialized whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized]
  } else if (src->color->isValid () && src->sourcetype == SOURCETYPE_COLOR) {
             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
gstopenni2src.cpp:752:26: note: uninitialized use occurs here
  GST_BUFFER_PTS (buf) = oni_ts - src->oni_start_ts;
                         ^~~~~~
gstopenni2src.cpp:721:10: note: remove the 'if' if its condition is always true
  } else if (src->color->isValid () && src->sourcetype == SOURCETYPE_COLOR) {
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
gstopenni2src.cpp:721:14: error: variable 'oni_ts' is used uninitialized whenever '&&' condition is false [-Werror,-Wsometimes-uninitialized]
  } else if (src->color->isValid () && src->sourcetype == SOURCETYPE_COLOR) {
             ^~~~~~~~~~~~~~~~~~~~~~
gstopenni2src.cpp:752:26: note: uninitialized use occurs here
  GST_BUFFER_PTS (buf) = oni_ts - src->oni_start_ts;
                         ^~~~~~
gstopenni2src.cpp:721:14: note: remove the '&&' if its condition is always true
  } else if (src->color->isValid () && src->sourcetype == SOURCETYPE_COLOR) {
             ^~~~~~~~~~~~~~~~~~~~~~~~~
gstopenni2src.cpp:642:18: note: initialize the variable 'oni_ts' to silence this warning
  uint64_t oni_ts;
                 ^
                  = 0
This commit is contained in:
Sebastian Dröge 2014-07-21 10:14:17 +02:00
parent 2f28fcb862
commit c42595cc69

View file

@ -744,6 +744,9 @@ openni2_read_gstbuffer (GstOpenni2Src * src, GstBuffer * buf)
src->colorFrame->getWidth (), src->colorFrame->getWidth (),
src->colorFrame->getHeight (), src->colorFrame->getHeight (),
src->colorFrame->getDataSize ()); src->colorFrame->getDataSize ());
} else {
g_return_val_if_reached (GST_FLOW_ERROR);
return GST_FLOW_ERROR;
} }
if (G_UNLIKELY (src->oni_start_ts == GST_CLOCK_TIME_NONE)) if (G_UNLIKELY (src->oni_start_ts == GST_CLOCK_TIME_NONE))