#29018: Make all statistics depend on ExtraInfoStatistics
------------------------------------+------------------------------------
 Reporter:  teor                    |          Owner:  teor
     Type:  enhancement             |         Status:  needs_review
 Priority:  Medium                  |      Milestone:  Tor: 0.4.0.x-final
Component:  Core Tor/Tor            |        Version:
 Severity:  Normal                  |     Resolution:
 Keywords:  fast-fix, security-low  |  Actual Points:  2
Parent ID:                          |         Points:  0.1
 Reviewer:  nickm                   |        Sponsor:  SponsorV-can
------------------------------------+------------------------------------

Comment (by teor):

 Replying to [comment:7 teor]:
 > I think there might be some memory leaks in the new unit tests, I'm not
 sure if I caught them all. I'll check in the morning once CI has finished.

 I found and fixup'd one memory leak. Hopefully it's the last one. (I need
 to install a newer version of clang to get good leak detection locally.)

 > I think the coverage should have increased a bit compared with #29017
 and master. But I'll need to wait for CI to make sure.

 Coveralls stopped working about 20 hours ago. Maybe it's back now?

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/29018#comment:8>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Reply via email to