This compatibility hack was accidently broken by commit f072cbbb

memcmp() can overflow the buffer.

Regression by commit 738583e3

The state ERROR can only be entered during decoder initialization, and
Seek() may only be called after initialization has been finished.

Some optimized implementations of memcmp() may not start from the
beginning of the string, and may thus segfault.

If the decoder was just started, it already seeks to the desired
position.

If this gets called while seeking, do the right thing.  This moves the
elapsed_time setter from method Run(), which is now redundant.

Allows removing a few special cases in the player thread.

It is futile to check for decoder errors before the decoder has
finished startup.  At this time, it's unlikely that the decoder has
already failed.

.. and fix its API documentation.