Re: remove bar-size and replace its usage by bar-extent (issue4025044)

[n . puttock]

Hi Pál,

LGTM apart from the change in behaviour for dashed barlines: they're
overshooting the outer lines.  I couldn't see this from the regtest
comparison for bar-line-dashed.ly, but it's clearly visible on paper.

Cheers,
Neil



http://codereview.appspot.com/4025044/diff/20001/lily/bar-line.cc
File lily/bar-line.cc (left):

http://codereview.appspot.com/4025044/diff/20001/lily/bar-line.cc#oldcode135
lily/bar-line.cc:135: {
remove { }

http://codereview.appspot.com/4025044/diff/20001/lily/bar-line.cc#oldcode140
lily/bar-line.cc:140: {
remove { }

http://codereview.appspot.com/4025044/diff/20001/lily/bar-line.cc
File lily/bar-line.cc (right):

http://codereview.appspot.com/4025044/diff/20001/lily/bar-line.cc#newcode37
lily/bar-line.cc:37: {
remove { }

http://codereview.appspot.com/4025044/diff/20001/lily/bar-line.cc#newcode72
lily/bar-line.cc:72: if (ex.length() > 0)
length ()

http://codereview.appspot.com/4025044/diff/20001/lily/bar-line.cc#newcode113
lily/bar-line.cc:113: Real const h = extent.length();
length ()

http://codereview.appspot.com/4025044/diff/20001/lily/bar-line.cc#newcode325
lily/bar-line.cc:325: Real const h = extent.length();
length ()

http://codereview.appspot.com/4025044/diff/20001/lily/span-bar.cc
File lily/span-bar.cc (right):

http://codereview.appspot.com/4025044/diff/20001/lily/span-bar.cc#newcode80
lily/span-bar.cc:80: {
remove { }

http://codereview.appspot.com/4025044/