sfos/ledger
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix bug 935, very long account names can get stomped on.

Browse source 
This works, but hammers performance
This commit is contained in:
Craig Earls 2013-03-27 15:37:52 -07:00
parent ad07d28427
commit 4ca0e8916b
1 changed files with 8 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
lisp/ldg-post.el
View file

@ -136,17 +136,16 @@ point at beginning of the commodity."
(match-end 3)) (point))))
(defvar ledger-post-account-regex
(concat "\\(^[ \t]+\\)"
"\\([\\[(*!;a-zA-Z0-9]\\)"))
"\\(^[ \t]+\\)\\([\\[(;!*A-Za-z0-9]\\)\\(.+?\\)\\( \\|\n\\)")
(defsubst ledger-next-account (&optional end)
(defun ledger-next-account (&optional end)
"Move point to the beginning of the next account, or status marker (!*), as long as it is not past END.
Return the column of the beginning of the account and leave point
at beginning of account"
(if (> end (point))
(when (re-search-forward ledger-post-account-regex end t)
(when (re-search-forward ledger-post-account-regex (1+ end) t)
(goto-char (match-beginning 2))
(current-column))))
(list (current-column) (length (match-string-no-properties 3))))))
(defsubst ledger-post-adjust (adjust-by)
(if (> adjust-by 0)
@ -185,10 +184,12 @@ region align the posting on the current line."
lines-left)
(< (point) end-region))
(when acc-col
(if (/= (setq acc-adjust (- ledger-post-account-alignment-column acc-col)) 0)
(if (/= (setq acc-adjust (- ledger-post-account-alignment-column (car acc-col))) 0)
(ledger-post-adjust acc-adjust))
(when (setq amt-offset (ledger-next-amount (line-end-position)))
(when (and
(> ledger-post-amount-alignment-column (+ ledger-post-account-alignment-column (cadr acc-col)))
(setq amt-offset (ledger-next-amount (line-end-position))))
(let* ((amt-adjust (- ledger-post-amount-alignment-column
amt-offset
(current-column))))