devosoft/avida — pull request #10 by dgodfrey95 (2014-05-20 21:07:50+00:00)
Title: Removed unnecessary dynamic allocation with cTextWindow, used stack allo...

...cation instead; used std::vector instead of raw C-style arrays.

I removed the check for 0 in the destructor, as according to this (http://dietmar-kuehl.de/mirror/c++-faq/freestore-mgmt.html#faq-16.8) it is not needed. Deleting 0 is well-defined.

Also, inside cAnalyzeView::Confirm, you have an unchecked case, the part where cur_char == NULL. I think you should have a default case where you change the result and finished variables.