[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Enhancement: Internal ledger lines. (issue1855056)

From: hanwenn
Subject: Enhancement: Internal ledger lines. (issue1855056)
Date: Wed, 04 Aug 2010 21:44:58 +0000

what's the plan for this patch/functionality ?

the code looks a bit out of style.

would it be possible to group all the internal-ledger code together?
Without wanting to trample on feet, this looks like a feature that could
easily bitrot due to lack of use, so it's good to make it easy to remove
File lily/ (right):
lily/ Internal_ledgers () : halfspace_ (1)
we usually do

  = 1

in the body.
lily/ *
is this a new coding style? - if not follow the rest of the code?
lily/ return ledger_extent_.contains (pos /
? should be * rather than /
lily/ Real &left_shorten);
IIRC, we use pointers rather than refs for modifying arguments.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]