apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joe Orton <jor...@redhat.com>
Subject Re: svn commit: r424900 - /apr/apr/trunk/test/testnames.c
Date Mon, 24 Jul 2006 10:18:11 GMT
On Mon, Jul 24, 2006 at 04:27:33AM -0000, William Rowe wrote:
> Author: wrowe
> Date: Sun Jul 23 21:27:27 2006
> New Revision: 424900
> 
> URL: http://svn.apache.org/viewvc?rev=424900&view=rev
> Log:
> 
>   Revert my last commit.  I've determined that data becomes data, while
>   data/ remains data/.  ../../.. should observe the same behavior, so this
>   is effectively a bug, even if it has no negative impact.

The Unix implementation will always append a trailing slash in this 
case, it seems.  The APR_FILEPATH_TRUENAME documentation appears to 
suggest it should too; is this a bug?

> 
> Modified:
>     apr/apr/trunk/test/testnames.c
> 
> Modified: apr/apr/trunk/test/testnames.c
> URL: http://svn.apache.org/viewvc/apr/apr/trunk/test/testnames.c?rev=424900&r1=424899&r2=424900&view=diff
> ==============================================================================
> --- apr/apr/trunk/test/testnames.c (original)
> +++ apr/apr/trunk/test/testnames.c Sun Jul 23 21:27:27 2006
> @@ -100,7 +100,7 @@
>                              "../../..", APR_FILEPATH_TRUENAME, p);
>      ABTS_PTR_NOTNULL(tc, dstpath);
>      ABTS_INT_EQUAL(tc, APR_SUCCESS, rv);
> -    ABTS_STR_EQUAL(tc, "../../../", dstpath);
> +    ABTS_STR_EQUAL(tc, "../../..", dstpath);
>  
>      rv = apr_filepath_merge(&dstpath, "", 
>                              "../../../", APR_FILEPATH_TRUENAME, p);
> 

Mime
View raw message