Message-Id: <199904100751.RAA04041@silk.apana.org.au>
From: "Brian Havard" <brianh@kheldar.apana.org.au>
To: "new-httpd@apache.org" <new-httpd@apache.org>
Date: Sat, 10 Apr 1999 17:51:30 +1000 (EST)
In-Reply-To: <19990409143705.27603.qmail@hyperreal.org>
MIME-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
Subject: Re: cvs commit: apache-apr/include apr_file_io.h
Sender: new-httpd-owner@apache.org
Precedence: bulk
Reply-To: new-httpd@apache.org

On 9 Apr 1999 14:37:05 -0000, rbb@hyperreal.org wrote:

>rbb         99/04/09 07:37:05
>
>  Modified:    apr/file_io/unix Makefile readwrite.c
>               apr/test testfile.c
>               docs     fileio.txt impl.txt
>               include  apr_file_io.h
>  Added:       apr/file_io/unix dir.c
>  Log:
>  Added support for directory commands in apr.  I also fixed a few
>  warnings in the code.  I updated the fileio.txt and impl.txt docs to
>  reflect the current state of the fileio code.  And lastly, I updated the
>  test suite to include a few more tests.

I can't say I really like what you've done for the directory API. It's way
too unix oriented in that it's an almost no-op wrapper for opendir() and
friends. Sure, it'll be fast but it's not too portable.

The way I see it there shouldn't be any platform specific stuff in
apr_file_io.h so putting stuff like:

    DIR *dirstruct;
typedef struct dirent     apr_dirent_t;

in it isn't a good idea. We need to define and use our own structures and
hide these platform specific details in the library code.

--
 ______________________________________________________________________________
 |  Brian Havard                 |  "He is not the messiah!                   |
 |  brianh@kheldar.apana.org.au  |  He's a very naughty boy!" - Life of Brian |
 ------------------------------------------------------------------------------