Bug 27846 - Return value of apr_uri_parse wrongly documented.
Summary: Return value of apr_uri_parse wrongly documented.
Status: RESOLVED FIXED
Alias: None
Product: APR
Classification: Unclassified
Component: APR-util (show other bugs)
Version: HEAD
Hardware: All other
: P3 minor (vote)
Target Milestone: ---
Assignee: Apache Portable Runtime bugs mailinglist
URL:
Keywords: PatchAvailable
Depends on:
Blocks:
 
Reported: 2004-03-22 15:39 UTC by Arno Bakker
Modified: 2005-07-19 04:37 UTC (History)
0 users


Attachments
Fixes Docs and Return Types (2.16 KB, patch)
2004-05-18 03:00 UTC, Paul Querna 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Arno Bakker 2004-03-22 15:40:00 UTC 
The include file says it's "An HTTP status code" whereas the source returns
APR_* values. So return value should be apr_status_t.
Comment 1 Paul Querna 2004-05-18 03:00:52 UTC 
Created attachment 11583 [details]
Fixes Docs and Return Types
Comment 2 Paul Querna 2004-05-18 03:02:44 UTC 
Patch Against APR-Util HEAD.

Please also look the seperate bug under #28453.  The patches supplied there need
to be updated with the correct return codes.
Comment 3 Garrett Rooney 2004-11-18 17:02:53 UTC 
I'm pretty sure this patch can be committed...

At the very least the documentation update should go in, since the current
doxygen is just wrong, and we can probably get away with changing the return
value to apr_status_t for clarity, since apr_status_t is typedefed to int anyway.
Comment 4 Paul Querna 2005-07-19 12:37:49 UTC 
Ya, this has been fixed since I got commit bits.