First, realize that this ONLY matters for calculating the difference
to target balance.  It never effects the ledger file itself.  The
assumption is made for display only in ledger-commodity-to-string.

You make a very good point about using split-commodity in
read-commodity-string.  I will work on that.

On Mon, Apr 1, 2013 at 10:48 AM, Rémi Vanicat <[email protected]> wrote:
> Craig Earls <[email protected]> writes:
>
>> It assumes single character commodity go in front.
>
> If it was the case, it would be a problem because this assumption is
> false in some country (France is an example). But I can't see where in
> ledger-read-commodity-string the code assume that single character
> commodity go in front.
>
> I'm speaking about the code one can find there:
> https://github.com/ledger/ledger/blob/next/lisp/ldg-commodities.el#L129
>
> This code assume a lot of thing (no commodity with " or space for
> example, and a space between the commodity and the am mount) and seem
> not needed as it could just call ledger-split-commodity-string to split
> the commodity string, and profit from the complex code there.
>
>>
>> On Sun, Mar 31, 2013 at 7:39 AM, Rémi Vanicat <[email protected]> wrote:
>>> Craig Earls <[email protected]> writes:
>>>
>>>> Have you set ledger-reconcile-default-commodity?
>>>
>>> Thanks, it solve this problem. But if I type $320, ledger mode won't
>>> understand that I mean 320 $.
>>>
> [...]
> --
> Rémi Vanicat
>
> --
>
> ---
> 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/groups/opt_out.
>
>



-- 
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/groups/opt_out.


Reply via email to