Looks good.
https://codereview.chromium.org/1171943002/diff/1/tools/testrunner/local/progress.py
File tools/testrunner/local/progress.py (right):
https://codereview.chromium.org/1171943002/diff/1/tools/testrunner/local/progress.py#newcode70
tools/testrunner/local/progress.py:70: def SetRunner(self, runner):
On 2015/06/09 09:18:01, tandrii(chromium) wrote:
IMO, I'd keep the constructor.
Agreed. The constructor exists mostly for documentation purposes too --
the idea is that looking at the constructor is enough to understand the
object layout, as all fields are initialized there.
Am I right that SetRunner is really for documentation, not really for
anything
else? because bla.runner = runner still works anyways...
I guess unless setattr is implemented as you suggested above, exposing a
setter is the only [elegant] way to set the runner for all registered
progress indicators.
https://codereview.chromium.org/1171943002/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
