Remove FIXME's
Issue #120
resolved
The word "FIXME" shows up ~20 times in the Amp code. (Run grep -r FIXME
from the amp directory to see them.) We should remove as many as possible before doing a release.
Comments (5)
-
-
reporter @akhorshi I just removed a bunch more. Only 3 to go, and they are marked with your initials. Can you take care of these?
-
Mine removed. @muammar had a few in his recent commit 9e96dfa.
-
I think I found an acceptable way of getting rid of the ones introduced in my commit mentioned by @akhorshi. You can check it here ebf69b85e3edbf1aece1fc498ce9f45cddcc0bda.
-
- changed status to resolved
Thanks @muammar . Then I close this issue.
- Log in to comment
Some of them removed in the commit 177e8d0.