[prev in list] [next in list] [prev in thread] [next in thread] 

List:       intermezzo-devel
Subject:    Re: clear_all_fsets (presto.c)
From:       "Peter J. Braam" <braam () clusterfilesystem ! com>
Date:       2001-11-13 16:26:32
[Download RAW message or body]

It was something like:

 while ( next fset ) {		
   error = cleanup_fset();
 }

Then it became after you + Adriano:

 error = 0;
 while ( next fset ) { 
    cleanup_fset();
 }

I'm proposing:

 while ( next fset ) { 
    if ( !error ) 
      error = cleanup_fset()
    else 
      cleanup_fset();
 }

So that at least we get the error; at present we don't.  Now that just
reminds me of rmdir :) 

- Peter -



On Tue, Nov 13, 2001 at 11:06:00AM -0500, Phil Schwan wrote:
> On Nov 12, Peter J. Braam wrote:
> > 
> > It appears that the return of -EINVAL is related to some changes you
> > made. I wonder if we should exit the function the moment we find any
> > error? 
> 
> We're talking about presto.c:clear_all_fsetroots(), right?  Which
> changes did I make, or were those changes in another function?
> 
> Did Adriano's patch not fix that, or are we talking about something else?
> 
> -Phil

-- 

_______________________________________________
intermezzo-devel mailing list
intermezzo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/intermezzo-devel

[prev in list] [next in list] [prev in thread] [next in thread]