[tor-bugs] #25185 [Core Tor/Tor]: Create utilities for using Rust static strings in C

Tor Bug Tracker & Wiki blackhole at torproject.org
Tue Feb 13 22:44:21 UTC 2018


#25185: Create utilities for using Rust static strings in C
-----------------------------------+------------------------------------
 Reporter:  isis                   |          Owner:  isis
     Type:  enhancement            |         Status:  merge_ready
 Priority:  Medium                 |      Milestone:  Tor: 0.3.4.x-final
Component:  Core Tor/Tor           |        Version:
 Severity:  Normal                 |     Resolution:
 Keywords:  rust, review-group-32  |  Actual Points:
Parent ID:                         |         Points:  1
 Reviewer:  catalyst               |        Sponsor:  SponsorM
-----------------------------------+------------------------------------
Changes (by catalyst):

 * status:  needs_review => merge_ready


Comment:

 I think this looks good, though I might be missing subtle things because
 I'm still rather new to Rust.

 Minor documentation clarity nits:
 * The "Note" in the documentation for `cstr!` refers to "the above code
 compiles", but it actually means the first out of three examples rather
 than the example immediately preceding it.  Maybe replace it with "the
 first example above compiles"?
 * In that same "Note", "symbols table" should probably be "symbol table"
 (unless it's idiomatic in Rust documentation to use plural "symbols"
 there)
 These are fairly minor so feel free to merge as is or without further
 review after fixing the nits.

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


More information about the tor-bugs mailing list