On Sat, Feb 09, 2002 at 11:07:09PM -0000, Chris January wrote: >The attached patch (against cygwin-1.3.9-1/winsup/cygwin) adds support for a >/proc virtual filesystem and a read-only version of /proc/registry. I've >read http://cygwin.com/assign.txt but need to sort out the legalese before >doing anything. >Partically fufills Cygwin TODO item: >2001-11-08 /proc filesystem Nicos
Wow. From a casual inspection, this looks pretty nice. However, I didn't look at it too closely. I don't want to be tainted if you are unable to send in an assignment. However, the selection of methods, etc. seems fine. The ChangeLog looked good, AFAICT. The code formatting is not GNU standard. That will have to be rectified. And, if/when the code is incorporated into the cygwin DLL, the copyright info will have to be changed to confirm to the rest of cygwin's source modules. I wish I could get people to look at sending in an assigment *before* they start doing work. It would save a lot of time, I think. Instead we'll now shortly be hearing from the "usual" people asking why this isn't in cygwin already. cgf
