RE: svn commit: r884250 - /subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c
From: Bert Huijben <bert_at_qqmail.nl>
Date: Mon, 21 Dec 2009 17:38:28 +0100
> -----Original Message-----
One problem with this is that we convert all assertions to SVN_ERR_ASSERTION_FAIL, so we can never convert them to anything more detailed without processing the file and line number. Which you can't expect any ordinary user will do.
So... a buffer index out of range or buffer load assertion (libsvn_subr) assertion would have to have the exact same error handling as a missing header assertion (serf is full of these). Specifying an error code provides more options to our (library) users.
The only thing we can say for an assertion is: send a mail to users_at_subversion.a.o /.t.o. including this piece of information you (as user) don't understand. And probably no one at users@ understands either.
Currently I ask our users to mail our support list, but I'm happy to hardcode the users@ list address for this class of errors if we decide to show this unresolvable error in more cases in 1.7...
Please add useful error messages on all places that a user might encounter, likely or unlikely...
Only when an error can't be reached by a normal user an assertion could be the correct solution.
Bert
|
This is an archived mail posted to the Subversion Dev mailing list.
This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.