Cleanup of 'NativeModule.require' calls in lib/module.js

All calls to NativeModule.require called with string literal argument ('fs' and 'path') are unnecessarily explicitly referring to NativeModule and can and should be changed to plain require call. This matches with the way how any other native module perform these requires (module.js is in no way special in this).
Only call which should retain the explicit NativeModule.require is the one inside Module._load which calls with a variable argument id, not with a literal argument.

[ry]

please provide a patch

https://gist.github.com/1340039
tested, works (only a couple of tests fail, as is typical in freebsd)

Hello. Was the patch somehow inadequate or my answer only slipped somehow? @ry

Hello! I sent the patch as you requested. Is any other problem with this issue?

[bnoordhuis]

@Herby: Can you submit it as a pull request? Thanks.

Is it possible to reopen this? I can then issue a PR that fixes it.

Ping @isaacs.