On 2013/11/06 21:20:50, jeff.pihach wrote: ...
https://codereview.appspot.com/22500043/diff/1/app/views/viewlets/inspector-header.js#newcode59 > app/views/viewlets/inspector-header.js:59: var name = > pojoModel.displayName.match(/^\(([^)]*)\)$/)[1]; > I'd love it if there was a quick comment describing what this regex looks for to > make the code easier to scan.
You want comments? I'm your man! Done. :-)
Thanks for the review!
https://codereview.appspot.com/22500043/
« Back to merge proposal
On 2013/11/06 21:20:50, jeff.pihach wrote:
...
https:/ /codereview. appspot. com/22500043/ diff/1/ app/views/ viewlets/ inspector- header. js#newcode59 viewlets/ inspector- header. js:59: var name = displayName. match(/ ^\(([^) ]*)\)$/ )[1];
> app/views/
> pojoModel.
> I'd love it if there was a quick comment describing what this regex
looks for to
> make the code easier to scan.
You want comments? I'm your man! Done. :-)
Thanks for the review!
https:/ /codereview. appspot. com/22500043/