On Tue, 22 Dec 2020, Martin Liška wrote: > May I please ask a native speaker for a correction of my wording?
That excludes myself ;), but let me give it a pass, and perhaps you can then post an updated patch (for Jonathan or another native speaker to go through)? > + <li>New warning <a > href="https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#index-Wtsan";><code>-Wtsan</code></a>, > enabled by default, > + warns about unsupported features in ThreadSanitizer > (<code>std::atomic_thread_fence</code>).</li> Is this atomic_thread_fence the only one and there may be different ones in the future? In that case maybe write something like "(currently <code>...</code>). > + <li>Series of if conditions, sharing a common index variable, can be > transformed into a switch statement if each "A series of" > + of them contains a comparison expession. Example: "expression" > + <li>GCOV data file format can effectively store a group of counters that > are all zero.</li> "The GCOV data..." > + <li>Some <code>gcov</code> tool options are renamed: > <code>-i,--json-format</code> to > + <code>-j,--json-format</code> and <code>-j,--human-readable</code> to > <code>-H,--human-readable</code>.</li> I suggest "have been renamed". Are -i and --json-format the short and long forms of the same option? If so, how about saying something like "Some short options of the <code>gcov</code> tool..."? And then <code>-i</code> (<code>--json-format</code>) to <code>j</code> and <code>-j</code> (<code>--json-format</code>) to <code>-H... > + <li>The <code>no_stack_protector</code> attribute has been added > to mark functions which should not be instrumented That line is a little very long. ;-) This is okay modulo the notes above. Thank you, Gerald
