patchThe GNU Hurd - Patches: patch #642, simple nfs cleanups

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #642: simple nfs cleanups

Submitter:  James A. Morrison <mrphython>
Submitted:  Sat 09 Nov 2002 11:35:03 PM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  None Privacy:  Public
Assigned to:  None Open/Closed:  Open
Planned Release:  None
Wiki-like text discussion box: 


Sat 09 Nov 2002 11:35:03 PM UTC, original submission:  

It would be nice if you could get rid of the assert instead just moving it.

Also, I would like to see a rationale (if not as a comment, then along with the patch) for each XXX you removed.  Somebody put the comment there because he had doubts.  From looking at the patch, I can not decide if you didn't have those doubts because you had a better or worse understanding of the code than the person putting in those XXX :)

Thanks,
Marcus

James A. Morrison <mrphython>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #4770:  patch_item_642.patch added by mrphython (6KiB - text/plain - patch item 642)

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

 

No changes have been made to this item

Back to the top

Powered by Savane 3.13-758e.
Corresponding source code