Review of Jose Armando Garcia Sancio's std.log

Steven Schveighoffer schveiguy at yahoo.com
Wed Mar 7 06:39:52 PST 2012


On Wed, 07 Mar 2012 07:09:17 -0500, Steven Schveighoffer  
<schveiguy at yahoo.com> wrote:

> On Tue, 06 Mar 2012 14:39:28 -0500, Jose Armando Garcia  
> <jsancio at gmail.com> wrote:
>
>> On Tue, Mar 6, 2012 at 12:25 AM, Jonathan M Davis <jmdavisProg at gmx.com>  
>> wrote:
>>> On Tuesday, March 06, 2012 09:14:16 so wrote:
>>>> On Tuesday, 6 March 2012 at 07:46:14 UTC, Jacob Carlborg wrote:
>>>> > On 2012-03-06 02:32, Jonathan M Davis wrote:
>>>> >
>>>> > The user can then alias "log!info" to "info" if he/she wants to.
>>>>
>>>> Again, you are now forcing 2 common names instead of one as it is
>>>> now.
>>>> When you instantiate log!info where do you get "info" from?
>>>
>>> Yes. My mistake - probably because the time stuff typicall takes such a
>>> template argument as string, which would make this log!"info"(msg).  
>>> However,
>>> adding _log_ isn't necessarily bad, given that this is std.log that  
>>> we're
>>> talking about. It's info and the rest that are the problem.
>>>
>> Seriously everyone. What are we spending some much effort on this?
>
> Because naming is important.  It's very difficult to change names once  
> something is released.  I seriously could care less about implementation  
> (didn't even look at it).  That can be fixed.  Naming can't.
>
>> What is wrong with:
>>
>> import log = std.log;
>> log.info("cool");
>
> What is wrong with
>
> import std.log;
> log.info("cool");

alternatively:

log_info("cool");
linfo("cool");
lginfo("cool");

There are so many choices besides just "info."  We should use something  
else.

-Steve


More information about the Digitalmars-d mailing list