News:

--

Main Menu

Pointers in warnings from ADM_edRender.cpp

Started by eumagga0x2a, September 26, 2016, 02:43:55 PM

Previous topic - Next topic

eumagga0x2a

It is probably just a failure on my part to grasp the deeper purpose, but why do warnings in ADM_Composer::goToTimeVideo:193 and in ADM_Composer::switchToSegment:583 show pointers instead of segment numbers? Should it be rather this way?

diff --git a/avidemux/common/ADM_editor/src/ADM_edRender.cpp b/avidemux/common/ADM_editor/src/ADM_edRender.cpp
index 96e7dc5..0b29667 100644
--- a/avidemux/common/ADM_editor/src/ADM_edRender.cpp
+++ b/avidemux/common/ADM_editor/src/ADM_edRender.cpp
@@ -190,7 +190,7 @@ uint32_t seg;
     uint64_t to=segTime+s->_refStartTimeUs;
     if(false==seektoTime(s->_reference,to))
     {
-            ADM_warning("Cannot seek to beginning of segment %" PRIu32" at  %" PRIu64" ms\n",s,to/1000);
+            ADM_warning("Cannot seek to beginning of segment %" PRIu32" at  %" PRIu64" ms\n",seg,to/1000);
             return false;
     }
     _currentSegment=seg;
@@ -580,7 +580,7 @@ bool        ADM_Composer::switchToSegment(uint32_t s,bool dontdecode)
     
     if(false==seektoTime(seg->_reference,from,dontdecode))
     {
-            ADM_warning("Cannot seek to beginning of segment %" PRIu32" at  %" PRIu64" ms\n",s,from/1000);
+            ADM_warning("Cannot seek to beginning of segment %" PRIu32" at  %" PRIu64" ms\n",seg,from/1000);
             return false;
     }
     _currentSegment=s;