Took function declaration out of a loop; removed a dubious 'for' loop
Took function declaration out of a loop; removed a dubious 'for' loop

* JSLint's error: "Don't make functions within a loop". It leads to unnecessary
  multiple copies of a function in memory.
* Converted a 'for' look with a var declaration in it to be a cleaner 'while'
* Removed unused parameter 'breadth' from a nodesByBreadth call
* JSLint hates ++ and --; converted those to plain expressions

nowthis/d3-plugin-captain-sankey