Re: diskfs_make_node

2002-05-11 Thread Marcus Brinkmann
On Sat, Nov 17, 2001 at 12:15:00AM +0100, Neal H Walfield wrote: > > Those void functions you changed to error_t can never fail, so I don't see > > the point. The actual diskfs_make_node changes seem ok. > > They can never fail today, however, I thought it made the

Re: patch: diskfs_make_node returns an error_t

2002-04-02 Thread James Morrison
--- Roland McGrath <[EMAIL PROTECTED]> wrote: > I don't see anything wrong with the current diskfs_make_node signature. > It can only return failure for a malloc failure. > The advantages I see of using a signature that returns an error are: 1. Any users of make_node,

Re: patch: diskfs_make_node returns an error_t

2002-04-02 Thread Roland McGrath
I don't see anything wrong with the current diskfs_make_node signature. It can only return failure for a malloc failure. ___ Bug-hurd mailing list [EMAIL PROTECTED] http://mail.gnu.org/mailman/listinfo/bug-hurd

patch: diskfs_make_node returns an error_t

2002-04-02 Thread James A Morrison
I changed the signature of diskfs_make_node to return an error_t like diskfs_make_peropen or diskfs_make_protid. I'm holding back on the libnetfs stuff until I know how make_protid should act wrt it iouser struct. 2002-04-02 James A. Morrison <[EMAIL PROTECTED]> ext2fs:

Re: diskfs_make_node

2001-11-16 Thread Neal H Walfield
> Those void functions you changed to error_t can never fail, so I don't see > the point. The actual diskfs_make_node changes seem ok. They can never fail today, however, I thought it made the interface a bit more consistent. I guess, however, you are right, there is no need. Here

Re: diskfs_make_node

2001-11-16 Thread Roland McGrath
Those void functions you changed to error_t can never fail, so I don't see the point. The actual diskfs_make_node changes seem ok. ___ Bug-hurd mailing list [EMAIL PROTECTED] http://mail.gnu.org/mailman/listinfo/bug-hurd

diskfs_make_node

2001-11-16 Thread Neal H Walfield
The way we use diskfs_make_node is currently unsafe: we almost never check the return value, however, we really should. The following patches fix this. However, rather than simply correcting the problems, I opted to fix the error at its root: the interface. Now, we return an error code and