I think I found an API documentation bug.
svn_types.h specifies for svn_dirent_t.size:

  /** length of file text, or 0 for directories */
  svn_filesize_t size;

However, (almost?) all implementations set it to
SVN_INVALID_FILESIZE for directories. This is also
what we do for svn_io_dirent_t.size.

I discovered this as r1818578 broke our Perl tests.
The same test makes it clear that no API user could
have successfully used the API as documented.

Therefore, I suggest to change the API doc to match
the implementation, check the code for consistency
and add an entry to our errata list.

-- Stefan^2.

Reply via email to