c4milo / c4milo/node-inotify

Review code style - brought up by @juancate

Open
#54 0 comments 1 reaction 0 assignees View on GitHub
Dominant language
C++
Stars
242
Forks
39
PR merge metrics
No merged PRs in 30d

Description

Originally, node-inotify was written following Node.JS core C++ conventions. But, there are a couple of things we could improve in terms of readability:
- Explicit name spacing is better: http://stackoverflow.com/questions/4872373/why-is-including-using-namespace-into-a-header-file-a-bad-idea-in-c
- Using C++11 may be safer and cleaner.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.