Hacker News new | ask | show | jobs
by burstmode 3469 days ago
Results of code review of false.c:

* Use of undocumented magic numbers (1)

* No use of getter/setter pattern

* Inflexible design (datatype of result is fixed, no template pattern implemented)

* Manual memory management (no garbage collection used)

* No infrastructure for automated testing included

* No unit tests available

* Code has not changed for Years (code smell!); Probable stale code, to be removed in next release.