gonzalrd/RoyalSocietyRG — pull request #1 by harmonbc (2012-09-25 17:58:49+00:00)
Title: Code Review

Here is a summary of the changes I made:
Moved files to appropriate folders (src/resources) - All new files were created in the default vc10
Removed if statements from add() - moved the assignment of next and prev pointer of the sentinel to the constructor
Foxed Border Detection - If checking should have been looking for greater than values not for exact x and y coordinates. Also made the shifting uniform.
Changed help screen to display on start up and toggle with the ? button.
Reversal logic fixed - Although reversal is fixed it will not rearrange because the list is recreated every time update is run.

Issues:
Potential issues include the use of new in the update method with no corresponding delete. This creates a new object in memory every time update is run.