On Tue, Jun 7, 2011 at 08:14, <rhuij...@apache.org> wrote: > Author: rhuijben > Date: Tue Jun 7 12:14:14 2011 > New Revision: 1132968 > > URL: http://svn.apache.org/viewvc?rev=1132968&view=rev > Log: > Following up on r1132965, just move the type. This matches how we handled the > problem for svn_error_t. > > * subversion/include/svn_types.h > (svn_version_t): Add full definition here. > > * subversion/include/svn_version.h > (svn_version_t): And remove it here.
I've been thinking more on this change and absolutely hate it. We have a header DEDICATED to this structure and its concepts. The structure should be in that header file. It makes no sense to have a dedicated header, yet to move its key structure somewhere else. Please revert this change. -g
