lmi
[Top][All Lists]
Advanced

[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

Attachment: pgpulLlyAIFLR.pgp
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]