...
// Now we filter out an 
List retval = new LinkedList();
...

Shit. Ok, so it’s not too much code after that to see what the filtering is doing, but the comment still left me hanging. :)

2 Responses to “Comment of the Day: We filter what?”


  1. Mike McCune

    I saw the use of new LinkedList() and realized it was probably me since I’m the only one on the team that likes seems to init empty lists with a LinkedList()

    git annotate proved me right… :)


  2. Jay

    Wow, that’s impressive to sense the code smell from one line.

    If I remember right, it was in Kickstart stuff which kinda suggested a few people it likely would have been, but I wasn’t going to dig to find out. :)