This is the mail archive of the libstdc++@gcc.gnu.org mailing list for the libstdc++ project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Fix PR 17741


Chris Jefferson wrote:

PR 17741 is a simple problem, where the specialisations of find and find_if have not been uglified. This simple patch fixes it.

Tested on x86/linux.

Hi Chris.


I have two comments of the type once-and-for-all ;) ...

1- Please follow our conventions for the ChangeLog entry (just have a look to the file)
2- Please use diff -u (-p unfortunately is not very useful for C++) and typically -R (again, just have a look to the past postings for the details)


... and two more, specific to this patch:
   3- Do you actually have a testcase or not? ;)
   4- In my opinion, this kind of change is more suited for v7-branch.

Paolo.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]