On 8/2/19 9:48 AM, Jakub Jelinek wrote:
> On Fri, Aug 02, 2019 at 08:04:11AM +0200, Martin Liška wrote:
>> On 8/2/19 7:45 AM, Martin Liška wrote:
>>> I agree with that approach.
>>
>> I'm sending an example how it would look like for something
>> bigger:
>>
>> $ git diff 
>> e8a3be407068bfb9c82f0f6656b30d26cc2f484a~15..e8a3be407068bfb9c82f0f6656b30d26cc2f484a
>>  > patch && ./contrib/mklog patch > changelog.txt
>>
>> Thoughts?
> 
> Looks good to me, though of course I wonder what others have to say (if
> anyone cares).

Sure, let's wait for some time now.

> 
>>      * testsuite/23_containers/array/comparison_operators/constexpr.cc: New 
>> test.
>>      * testsuite/23_containers/array/tuple_interface/get_neg.cc (std::get):
>>      * testsuite/23_containers/array/tuple_interface/tuple_element_neg.cc:
>>      * testsuite/25_algorithms/adjacent_find/constexpr.cc: New test.
>>      * testsuite/25_algorithms/all_of/constexpr.cc: New test.
>>      * testsuite/25_algorithms/any_of/constexpr.cc: New test.
>>      * testsuite/25_algorithms/binary_search/constexpr.cc: New test.
>>      * testsuite/25_algorithms/constexpr_macro.cc: New test.
>>      * testsuite/25_algorithms/copy/58982.cc (test01):
>>      * testsuite/25_algorithms/copy/constexpr.cc: New test.
>>      * testsuite/25_algorithms/copy/move_iterators/69478.cc (test01):
>>      * testsuite/25_algorithms/copy_backward/constexpr.cc: New test.
>>      * testsuite/25_algorithms/copy_backward/move_iterators/69478.cc 
>> (test01):
> 
> Just a nit, some lines are too long and perhaps mklog could be improved too.
> On the first line above, one would normally wrap it as:
>       * testsuite/23_containers/array/comparison_operators/constexpr.cc:
>       New test.
> and on the last line above
>       * testsuite/25_algorithms/copy_backward/move_iterators/69478.cc
>       (test01):
> 
>       Jakub
> 

Good idea. What limit are we targeting in ChangeLog entries? 60 chars?

Martin

Reply via email to