Re: [Bug 97499] EasyHack: Clean up default arguments in uses of C++ unordered containers

> On 12 Feb 2016, at 6:14 PM, jan iversen <jani@documentfoundation.org> wrote:
>
> Sent from my iPad, please excuse any misspellings 

Hey, I do the same :-) And your iPad is much nicer than mine!

> On 12 Feb 2016, at 07:56, Chris Sherlock <chris.sherlock79@gmail.com 
> <mailto:chris.sherlock79@gmail.com>> wrote:
>
> > What does ToBeReviewed indicate? Like, reviewed by whom and for what reason?
> ToBeReviewed was set on all easyHacks, because I have been tasked with walking through all of 
> them and
> - verify they are still valid
> - compress those with many comments to a single comment (remove all unneeded info)
> - verify there is a mentor (typically the reporter)
>
> We have 308 issues, so ToBeReviewed was introduced to make it easier for others to help. Due to a 
> funny behaivour of BZ it had some less nice side effects, like removing old whiteboard text (kept 
> in history).
>
> You will see the ToBeReviewed go away over the next period of time (after dev list is no longer 
> CC on the hacks.)
>
> rgds
> jan i.

Thanks Jan :-)

Chris