suspicious code in file.cpp

[ghostvoodooman]

There is suspicious code in ReadAll() function:

 http://svn.wxwidgets.org/viewvc/wx/wxWidgets/trunk/src/common/file.cpp?view=markup

RealAll() function:

    for ( ;; )
    {
        static const unsigned READSIZE = 4096;

        ssize_t nread = Read(p, length > READSIZE ? READSIZE : length);
        if ( nread == wxInvalidOffset )
            return false;

        p += nread;
    }

    *p = 0;

    wxString strTmp(buf, conv);
    str->swap(strTmp);

    return true;

compiler warns about line 314 saying non-reachable code

*p = 0;

the code is not reachable, since there is infinite loop [namely "for(;;)" construct], and inside of it there is only conditional "return" statement, but no "break" statement. It is assumed the loop either loops infinitely or returns "false".

[vadz]

This is worse than suspicious, it's totally wrong :-( I forgot to finish writing this function before checking it in somehow. Fixing now, thanks for noticing this.

[VZ]

(In [72614]) Fix fatal bug in the recently added wxFile::ReadAll().

Make sure we exit the loop when reading the file in chunks in
wxFile::ReadAll() and add a unit test for it to ensure that it's really
correct.

Closes #14725.