Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

includes aufräumen #27

Open
IceFreez3r opened this issue Jan 14, 2020 · 0 comments
Open

includes aufräumen #27

IceFreez3r opened this issue Jan 14, 2020 · 0 comments
Assignees
Labels
documentation Improvements or additions to documentation small change Pull request or issue with only a small change

Comments

@IceFreez3r
Copy link
Owner

Gelegentlich fügt man für einen kurzen Test ein #include <> hinzu und vergisst es dann wieder zu entfernen, wenn man es nicht mehr braucht.
Deshalb gilt bei includes in der Zukunft, dass daneben ein kurzer Kommentar gehört, wofür der include gebraucht wird. Dann lässt sich schneller schauen, ob includes nicht eigentlich überflüssig sind.
Außerdem sollte jemand einmal alle bisherigen includes durchgehen, unnötige löschen und bei benötigten einen Kommentar wie oben machen.

#include math // "math.sqrt()"
@IceFreez3r IceFreez3r added documentation Improvements or additions to documentation small change Pull request or issue with only a small change labels Jan 14, 2020
@IceFreez3r IceFreez3r self-assigned this Jan 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation small change Pull request or issue with only a small change
Projects
None yet
Development

No branches or pull requests

1 participant