Re: svn commit: r1483292 - in /subversion/trunk/subversion: include/private/svn_string_private.h libsvn_subr/string.c libsvn_subr/types.c

Branko Čibej <brane_at_wandisco.com> writes:

> I always felt the strtol signature was massively broken because of
> that.

Probably a legacy of K&R C which didn't have function prototypes or
const. Very few people parse truly const memory so it is usually
legitimate to write to 'end'. In ISO C with both const and non-const
'end' there is going to be a cast in the user code somewhere. I guess
not breaking legacy code won.

In our code we could change the function signature from 'char **end' to
'int *count' and avoid the cast.

-- 
Certified & Supported Apache Subversion Downloads:
http://www.wandisco.com/subversion/download