[PATCH v3 3/4] emacs: rename notmuch-decimal-separator to notmuch-thousands-separator
Dmitry Kurochkin
dmitry.kurochkin at gmail.com
Thu Dec 15 16:59:22 PST 2011
On Tue, 13 Dec 2011 18:32:11 +0100, Thomas Jost <schnouki at schnouki.net> wrote:
> In 123,456.78, "." is the decimal separator, but "," is the thousands separator.
>
> This commit also mentions the space being used as thousands separator in several
> European countries.
> ---
What do perople think about making the thousands separator a space by
default?
> emacs/notmuch-hello.el | 8 ++++----
> 1 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/emacs/notmuch-hello.el b/emacs/notmuch-hello.el
> index 0582cae..0fe9c1d 100644
> --- a/emacs/notmuch-hello.el
> +++ b/emacs/notmuch-hello.el
> @@ -131,10 +131,10 @@ So:
> (integer :tag "Number of characters")
> (float :tag "Fraction of window")))
>
> -(defcustom notmuch-decimal-separator ","
> - "The string used as a decimal separator.
> +(defcustom notmuch-thousands-separator ","
> + "The string used as a thousands separator.
I suggest renaming this to notmuch-hello-thousands-separator.
Looks good otherwise.
Regards,
Dmitry
>
> -Typically \",\" in the US and UK and \".\" in Europe."
> +Typically \",\" in the US and UK and \".\" or \" \" in Europe."
> :group 'notmuch
> :type 'string)
>
> @@ -159,7 +159,7 @@ Typically \",\" in the US and UK and \".\" in Europe."
> (apply #'concat
> (number-to-string (car result))
> (mapcar (lambda (elem)
> - (format "%s%03d" notmuch-decimal-separator elem))
> + (format "%s%03d" notmuch-thousands-separator elem))
> (cdr result)))))
>
> (defun notmuch-hello-trim (search)
> --
> 1.7.8
>
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch
More information about the notmuch
mailing list