|
From: | hanwenn |
Subject: | Re: Sketch for broken beams with consistent slopes (issue 4961041) |
Date: | Mon, 03 Oct 2011 04:53:48 +0000 |
i know it's annoying, but could you separate out the cosmetics (adding _ ) to members from the rest of this change? The cosmetic changes make it difficult to see the essence of what you are trying to do. http://codereview.appspot.com/4961041/diff/39001/lily/include/beam-scoring-problem.hh File lily/include/beam-scoring-problem.hh (right): http://codereview.appspot.com/4961041/diff/39001/lily/include/beam-scoring-problem.hh#newcode151 lily/include/beam-scoring-problem.hh:151: vector<Real> stem_ypositions_; organize so it's clear to what members the comment pertains. http://codereview.appspot.com/4961041/
[Prev in Thread] | Current Thread | [Next in Thread] |