Thanks, I ran a quick check and the single digit amount is getting by the filter. It turns out that it doesn't like pure integer amounts so it isn't as far in the corner as I would like. I will get it fixed.
On Wed, Sep 10, 2014 at 6:54 AM, Johann Klähn <[email protected]> wrote: > I just stripped this down, I noticed it in my own journal where I use normal > account names but happened to have a zero in some auto-generated postings. I > agree that this is a corner case and very low priority, as I can just modify > my journal. Just wanted to let you know. > > On Wed, Sep 10, 2014 at 3:51 PM, Craig Earls <[email protected]> wrote: >> >> I will look into this, but I think single character account names and >> single digit amounts are not realistic. >> >> On Wed, Sep 10, 2014 at 6:48 AM, Johann Klähn <[email protected]> wrote: >> > Nice work Craig! >> > One thing I noticed, though: In the following transaction the account >> > name >> > in the second line (B) is highlighted using >> > `ledger-font-posting-amount-face`. >> > And the 0 in the first posting is highlighted as if it was part of the >> > account name. >> > >> > 2014/03/11 Whatever >> > A 0 >> > B 6,75€ >> > C -6,75€ >> > >> > This seems to happen for every posting that follows one ending with a >> > zero >> > (i.e. for permutations of the example above). >> > >> > On Tue, Sep 9, 2014 at 6:43 AM, Craig Earls <[email protected]> wrote: >> >> >> >> Indentation is totally independent of font-lock. There is a bug noted >> >> already. I just haven't gotten to it. >> >> >> >> >> >> On Monday, September 8, 2014, Andrew Schwartzmeyer >> >> <[email protected]> wrote: >> >>> >> >>> >> >>> On Sep 8, 2014, at 8:49 PM, Craig Earls <[email protected]> wrote: >> >>> >> >>> > I have spent the last month overhauling ledger fontification. I >> >>> > gave >> >>> > up on using the standard REGEX based methods that most font lockers >> >>> > use. Instead I wrote a few functions that actually understand the >> >>> > semantics of a ledger xact then had jit-lock and font-lock call >> >>> > those. >> >>> > Overall there is much more flexibility and it seems to be plenty >> >>> > fast. The regexes I do use are much simpler. I tested with a file >> >>> > that had 2,613,072 postings and it was lighting fast. Comments >> >>> > after >> >>> > amounts are now properly recognized and you can fontify an entire >> >>> > xact >> >>> > based on its overall status (my favorite part, that font-lock just >> >>> > really couldn't do). >> >>> >> >>> Awesome! Thanks so much for your work man. >> >>> >> >>> > For those of you who use lots of different directives there are now >> >>> > separate faces possible for all valid directives (or just use one >> >>> > face >> >>> > for all directives) >> >>> > >> >>> > Please check it out and report any bugs you find. I have banged on >> >>> > it >> >>> > for a while and it has been stable and well behaved. Of course you >> >>> > will find problems. >> >>> >> >>> Gotta say for the first thirty seconds I though it was totally broken >> >>> when everything was white, made sense once I scrolled down (white for >> >>> cleared transactions, green for budget, normal for uncleared, at least >> >>> with >> >>> my theme). >> >>> >> >>> So far it's recognizing (highlighting properly): assertions, lot value >> >>> / >> >>> posting cost / amount expressions, and posting notes. Sweet! >> >>> >> >>> However, it unfortunately doesn't smartly indent any of the above >> >>> except >> >>> plain currency values. What would go into fixing that? I can take a >> >>> look. >> >>> >> >>> > -- >> >>> > Craig, Corona De Tucson, AZ >> >>> > enderw88.wordpress.com >> >>> > >> >>> > -- >> >>> > >> >>> > --- >> >>> > You received this message because you are subscribed to the Google >> >>> > Groups "Ledger" group. >> >>> > To unsubscribe from this group and stop receiving emails from it, >> >>> > send >> >>> > an email to [email protected]. >> >>> > For more options, visit https://groups.google.com/d/optout. >> >>> >> >>> -- >> >>> >> >>> --- >> >>> You received this message because you are subscribed to the Google >> >>> Groups >> >>> "Ledger" group. >> >>> To unsubscribe from this group and stop receiving emails from it, send >> >>> an >> >>> email to [email protected]. >> >>> For more options, visit https://groups.google.com/d/optout. >> >> >> >> >> >> >> >> -- >> >> Craig, Corona De Tucson, AZ >> >> enderw88.wordpress.com >> >> >> >> -- >> >> >> >> --- >> >> You received this message because you are subscribed to the Google >> >> Groups >> >> "Ledger" group. >> >> To unsubscribe from this group and stop receiving emails from it, send >> >> an >> >> email to [email protected]. >> >> For more options, visit https://groups.google.com/d/optout. >> > >> > >> > -- >> > >> > --- >> > You received this message because you are subscribed to the Google >> > Groups >> > "Ledger" group. >> > To unsubscribe from this group and stop receiving emails from it, send >> > an >> > email to [email protected]. >> > For more options, visit https://groups.google.com/d/optout. >> >> >> >> -- >> Craig, Corona De Tucson, AZ >> enderw88.wordpress.com >> >> -- >> >> --- >> You received this message because you are subscribed to the Google Groups >> "Ledger" group. >> To unsubscribe from this group and stop receiving emails from it, send an >> email to [email protected]. >> For more options, visit https://groups.google.com/d/optout. > > > -- > > --- > You received this message because you are subscribed to the Google Groups > "Ledger" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > For more options, visit https://groups.google.com/d/optout. -- Craig, Corona De Tucson, AZ enderw88.wordpress.com -- --- You received this message because you are subscribed to the Google Groups "Ledger" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
