On 08/25/2014 04:05 PM, Jason Uh wrote:
On 08/25/2014 03:41 PM, Sean Mullan wrote:
Just a couple of comments:
1. I think the check on lines 106-110 can be done prior to the for loop
on line 94
I think that check is actually where it should be because the for loop
on line 94 loops over each component (while the inner loop on line 115
checks the characters of that component). The check for the first and
last characters is done just once per component.
Right, never mind. There is some extra indentation on line 95 where it
isn't aligned properly - can you remove the extra spaces?
--Sean