https://bugs.documentfoundation.org/show_bug.cgi?id=145175

--- Comment #9 from Chris Sherlock <chris.sherloc...@gmail.com> ---
Julien, I tested out your patch. It looks good. I respectfully disagree with
the reviewer - a straight loop is fine for now. 

My only question is: why would "Office.Commands/Execute" be a problem? I have a
utility that shows that "Setup/L10N" doesn't get cleared...  

Eval, if you want to improve upon the patch, why not fix it after the patch is
committed. I can't see any reason why this is more or less readable than having
to setup several lambdas for a custom built for_each_if() function! Which would
be replaced with std::ranges in C++20.

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to