[DVDStreamInfo] Discuss: 'CDVDStreamInfo::operator=' should return 'CDVDStreamInfo...

diff --git a/xbmc/cores/dvdplayer/DVDStreamInfo.h b/xbmc/cores/dvdplayer/DVDStreamInfo.h
index 3070094..0b33d94 100644 (file)
--- a/xbmc/cores/dvdplayer/DVDStreamInfo.h
+++ b/xbmc/cores/dvdplayer/DVDStreamInfo.h
@@ -96,6 +96,9 @@ public:
 
   bool operator==(const CDVDStreamInfo& right)      { return Equal(right, true);}
   bool operator!=(const CDVDStreamInfo& right)      { return !Equal(right, true);}
+
+  //'CDVDStreamInfo::operator=' should return 'CDVDStreamInfo &'.
+  // Is this intended? It looks more like a copy,cf. extradata, what happens if info = info?
   void operator=(const CDVDStreamInfo& right)       { Assign(right, true); }
 
   bool operator==(const CDemuxStream& right)      { return Equal( CDVDStreamInfo(right, true), true);}