Re: Smart pointer pretty printers

2018-01-11 Thread Juraj Oršulić
Hi Jonathan, Thanks for looking into this! It’s unfortunate that we can't know if the pointer in the iterator is legal to dereference. Nevertheless, inspecting smart pointers is a far more common use case, it's good that it has finally been sorted out. Thanks, Juraj On Tue, 9 Jan 2018 at

Re: Smart pointer pretty printers

2018-01-09 Thread Jonathan Wakely
On 09/01/18 18:50 +, Jonathan Wakely wrote: On 09/01/18 15:02 +, Jonathan Wakely wrote: On 09/01/18 14:59 +, Jonathan Wakely wrote: On 04/01/18 11:22 +0100, Juraj Oršulić wrote: Hi Jonathan (and the libstdc++ list). Can we revive this? I sent the patches for improving the smart

Re: Smart pointer pretty printers

2018-01-09 Thread Jonathan Wakely
On 09/01/18 15:02 +, Jonathan Wakely wrote: On 09/01/18 14:59 +, Jonathan Wakely wrote: On 04/01/18 11:22 +0100, Juraj Oršulić wrote: Hi Jonathan (and the libstdc++ list). Can we revive this? I sent the patches for improving the smart pointer pretty printers in March. They haven't been

Re: Smart pointer pretty printers

2018-01-09 Thread Jonathan Wakely
On 02/03/17 19:10 +0100, Juraj Oršulić wrote: On Fri, Feb 24, 2017 at 5:36 PM, Jonathan Wakely wrote: For a patch this size we do, so I'm not going to look at your patch. It will probably be simpler to just do it myself, and not worry about paperwork. If you want to

Re: Smart pointer pretty printers

2018-01-09 Thread Jonathan Wakely
On 09/01/18 14:59 +, Jonathan Wakely wrote: On 04/01/18 11:22 +0100, Juraj Oršulić wrote: Hi Jonathan (and the libstdc++ list). Can we revive this? I sent the patches for improving the smart pointer pretty printers in March. They haven't been reviewed. Thanks for the reminder. I'm testing

Re: Smart pointer pretty printers

2018-01-09 Thread Jonathan Wakely
On 04/01/18 11:22 +0100, Juraj Oršulić wrote: Hi Jonathan (and the libstdc++ list). Can we revive this? I sent the patches for improving the smart pointer pretty printers in March. They haven't been reviewed. Thanks for the reminder. I'm testing the attached patch, which has been rebased on