[tor-bugs] #20887 [Core Tor/Tor]: DIRCACHE_MIN_MEM_MB does not stringify on FreeBSD, we should use %d instead

[Tor Bug Tracker & Wiki]

#20887: DIRCACHE_MIN_MEM_MB does not stringify on FreeBSD, we should use %d instead
------------------------------------------------+--------------------------
 Reporter:  teor                                |          Owner:  (none)
     Type:  defect                              |         Status:
                                                |  needs_revision
 Priority:  Medium                              |      Milestone:  Tor:
                                                |  0.3.4.x-final
Component:  Core Tor/Tor                        |        Version:  Tor:
                                                |  0.2.8.1-alpha
 Severity:  Normal                              |     Resolution:
 Keywords:  easy, refactor, logging, tor-relay  |  Actual Points:
Parent ID:                                      |         Points:
 Reviewer:                                      |        Sponsor:
------------------------------------------------+--------------------------

Comment (by neel):

 Replying to [comment:9 nickm]:
 > There's a problem with this patch -- it removes the blank spaces before
 the quotation marks at the end of some of the lines.  Without those blank
 spaces, there will not be appropriate spacing between words in the log
 message.
 >
 > For example, this code
 > {{{
 >      tor_asprintf(msg, "Being a directory cache (default) with less
 than"
 >                      "%d MB of memory is not recommended and may
 consume"
 >                      "most of the available resources, consider
 disabling "
 >                      "this functionality by setting the DirCache option
 "
 >                      "to 0.", DIRCACHE_MIN_MEM_MB);
 > }}}
 > will produce a log message beginning with:
 > > Being a directory cache (default) with less than300 MB of memory is
 not recommended and may consumemost of the available resources...
 >
 > Note the "than300" and "consumemost" problem.
 >
 > Also, while we're changing this, let's replace the comma before
 "consider" with a period: those are two separate sentences.


 I made the changes in the patch `b20887-p2.patch`.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/20887#comment:10>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online