[Gnash-dev] Error compiling trunk on Ubuntu Hardy

[John Gilmore]

gst/MediaParserGst.cpp: In member function `bool 
gnash::media::gst::MediaParserGst::pushGstBuffer()':
gst/MediaParserGst.cpp:204: error: ISO C++ says that these are ambiguous, even 
though the worst conversion for the first is better than the worst conversion 
for the second:
/usr/include/c++/4.2/bits/postypes.h:175: note: candidate 1: std::fpos<_StateT> 
std::fpos<_StateT>::operator-(std::streamoff) const [with _StateT = __mbstate_t]
gst/MediaParserGst.cpp:204: note: candidate 2: operator-(std::streamoff, 
size_t) <built-in>

It looks like an example of why you should never write code in C++.
Straightforward looking code like this:

  _stream->seek(_stream->tell()-ret);

has gone all bizarro.  Did somebody overload subtraction or something?
I can't figure out what to do with it.  I'm using g++ 4.2.4 (Ubuntu
4.2.4-1ubuntu3).

There's also a logic bug in the same file, caught by the compiler:

gst/MediaParserGst.cpp: In member function `bool 
gnash::media::gst::MediaParserGst::probingConditionsMet(const 
gnash::media::gst::SimpleTimer&)':
gst/MediaParserGst.cpp:259: warning: comparison of unsigned expression >= 0 is 
always true
make[2]: *** [libgnashmedia_la-MediaParserGst.lo] Error 1

This is from:

   getBytesLoaded() >= MIN_PROBE_SIZE

where MIN_PROBE_SIZE is 0.  It's defined in the same file, with a commented
out definition as (PUSHBUF_SIZE * 3).  ???

        John