2015-09-07 22:10 GMT+02:00 Jonathan Wakely <jwak...@redhat.com>:
> On 07/09/15 20:53 +0100, Jonathan Wakely wrote:
>> On 07/09/15 21:04 +0200, Daniel Krügler wrote:
>>> In the suggested doc changes:
>>>
>>> +        When defined, <code>_GLIBCXX_ASSERTIONS</code> is defined
>>> +        automatically, so all the assertions that enables are also
>>> enabled
>>> +        in debug mode.
>>>
>>> there seems to be a typo, presumably it should be
>>>
>>> +        When defined, <code>_GLIBCXX_ASSERTIONS</code> is defined
>>> +        automatically, so all the assertions that <ins>it</ins>
>>> enables are also enabled
>>> +        in debug mode.
>>>
>>> instead?
>>
>>
>> It's correct as I wrote it, but your version is clearer so I'll change
>> it.
>>
>> My original can be read as "so all the assertions that that enables"
>> where the first "that" can be removed without changing the meaning.
>> Stoopid English ;-)
>
>
> I think this is even better:
>
>  When defined, <code>_GLIBCXX_ASSERTIONS</code> is defined
>  automatically, so all the assertions enabled by that macro are
>  also enabled in debug mode.
>
> Is that clear?

It's perfect!

Thanks,

- Daniel

Reply via email to