Author: rinrab
Date: Tue May 27 23:10:12 2025
New Revision: 1925891
URL: http://svn.apache.org/viewvc?rev=1925891&view=rev
Log:
On the 'utf8-cmdline-prototype' branch: svnlook: use svn_revnum_parse() instead
of relying on the custom parsing code.
* subversion/svnlook/svnlook.c
(sub_main::-r): ditto.
Let's not overcomplicate things 😁
Unrelated to the 'utf8-cmdline-prototype' branch.
Modified:
subversion/branches/utf8-cmdline-prototype/subversion/svnlook/svnlook.c
Modified:
subversion/branches/utf8-cmdline-prototype/subversion/svnlook/svnlook.c
URL:
http://svn.apache.org/viewvc/subversion/branches/utf8-cmdline-prototype/subversion/svnlook/svnlook.c?rev=1925891&r1=1925890&r2=1925891&view=diff
==============================================================================
--- subversion/branches/utf8-cmdline-prototype/subversion/svnlook/svnlook.c
(original)
+++ subversion/branches/utf8-cmdline-prototype/subversion/svnlook/svnlook.c Tue
May 27 23:10:12 2025
@@ -2529,15 +2529,7 @@ sub_main(int *exit_code,
switch (opt_id)
{
case 'r':
- {
- char *digits_end = NULL;
- opt_state.rev = strtol(opt_arg, &digits_end, 10);
- if ((! SVN_IS_VALID_REVNUM(opt_state.rev))
- || (! digits_end)
- || *digits_end)
- return svn_error_create(SVN_ERR_CL_ARG_PARSING_ERROR, NULL,
- _("Invalid revision number supplied"));
- }
+ SVN_ERR(svn_revnum_parse(&opt_state.rev, opt_arg, NULL));
break;
case 't':
