On Thu, Mar 14, 2013 at 3:06 AM, Derick Rethans <notifications@github.com>wrote:

> In mcon/connections.c:
>
> > @@ -282,10 +282,10 @@ void mongo_connection_destroy(mongo_con_manager *manager, void *data)
> >
> >  	/* Only close the connection if it matches the current PID */
> >  	if (current_pid == connection_pid) {
> > -		mongo_manager_log(manager, MLOG_CON, MLOG_FINE, "mongo_connection_destroy: Destroying connection object for '%s'", con->hash);
> > +		mongo_manager_log(manager, MLOG_CON, MLOG_FINE, "mongo_connection_destroy: Destroying connection object for %s", con->hash);
>
> On Wed, 13 Mar 2013, Hannes Magnusson wrote: > @@ -282,10 +282,10 @@ void
> mongo_connection_destroy(mongo_con_manager *manager, void *data) > > /*
> Only close the connection if it matches the current PID */ > if
> (current_pid == connection_pid) { > - mongo_manager_log(manager, MLOG_CON,
> MLOG_FINE, "mongo_connection_destroy: Destroying connection object for
> '%s'", con->hash); > + mongo_manager_log(manager, MLOG_CON, MLOG_FINE,
> "mongo_connection_destroy: Destroying connection object for %s",
> con->hash); I don't get it.. why removing the quotes? You explicitly asked
> for them to be added for other cases, how is this different?
> That's what I thought that I asked. I am still in favour of that, but then
> everything else in this file didn't... so I am in dubio.
>
Okidoki. Removing the quotes then o.O

-Hannes