[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [lmi] [lmi-commits] valyuta/005 df368ca 6/9: Replace currency::d() w
From: |
Vadim Zeitlin |
Subject: |
Re: [lmi] [lmi-commits] valyuta/005 df368ca 6/9: Replace currency::d() with d(currency) almost everywhere |
Date: |
Wed, 20 Jan 2021 18:07:10 +0100 |
On Wed, 20 Jan 2021 00:51:32 -0500 (EST) Greg Chicares
<gchicares@sbcglobal.net> wrote:
GC> branch: valyuta/005
GC> commit df368ca51bc8aace950480d4120f30fafedc61b4
GC> Author: Gregory W. Chicares <gchicares@sbcglobal.net>
GC> Commit: Gregory W. Chicares <gchicares@sbcglobal.net>
GC>
GC> Replace currency::d() with d(currency) almost everywhere
I didn't like calling a method d() very much, but using a global function
called d() is IMO beyond the pale. If you really, really want to keep this
name for it (instead of calling it currency_as_double() or some more
acceptable compromise between length and readability), we need to at least
put both currency and d() in a namespace and then put "using
currency_ns::currency" in the global namespace, so that d() would be only
found using ADL.
Regards,
VZ
pgpulLlyAIFLR.pgp
Description: PGP signature
- Re: [lmi] [lmi-commits] valyuta/005 df368ca 6/9: Replace currency::d() with d(currency) almost everywhere,
Vadim Zeitlin <=