ariya/phantomjs — comment on commit e88e008ecf742f705feba07fa9ada3d21f802c82
File: src/utils.cpp line 116
By detro:

Now I get it!

Fine, I agree with the feature.
But I'd separate the "autorun" option of the CLI, from the parameter u are
adding to the Utils::load... API: it's a totally disconnected context. That
API is used in so many other places.

Even more keen to change that boolean autorun now...

On 28 February 2012 18:28, Jon Leighton <
reply@reply.github.com
> wrote:

> Actually, I am confused about what you're now asking. I didn't initially
> add the `__run()` thing, but the justification for it is that you won't see
> any errors etc if you open the inspector in your external browser after
> they have occurred. So you want to be able to start phantomjs and get the
> inspector in front of you before telling phantomjs to go ahead and actually
> run the script. Hence you have to type `__run()` in the inspector console.
>
> However, in my project Poltergeist, I don't want to have to force users to
> type `__run()` (see the pull request), so I added this autorun option to
> toggle that.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ariya/phantomjs/commit/e88e008ecf742f705feba07fa9ada3d21f802c82#commitcomment-1022313
>

--
*Ivan De Marino*
Coder, Technologist, Cook, Italian

blog.ivandemarino.me | www.linkedin.com/in/ivandemarino |
twitter.com/detronizator