Paul Jolly

Results 688 comments of Paul Jolly

> About the proposed change, I'll take more time to think about it, but my initial reaction is that it sounds much better. 👍 - if anything I think we...

@shurcooL - I've now pushed up the limit-based setting of `--stack_size` as a separate commit. Let me know what you think. Once we're happy with the result we can squash...

@shurcooL - thanks, appreciate your patience with the couple of bad fixes that crept in 👍 @neelance - happy to chat through my thinking/reasoning if that helps to get you...

@neelance - thanks for your comments. I've added a couple of responses. Suspect I won't have a chance to do anything further on the PR this week. So in case...

Thanks @neelance @shurcooL for the feedback. I've now pushed up a new version that makes specific the use of `$copyInterfaceVal` per your comments @neelance. To pick up your point that...

> At this point, I think it'd be better to factor it out into a separate PR (after GopherJS 1.9-1 release). Happy to. I'll do exactly that once 1.9 lands.

@shurcooL #687 raised for the separate `ulimit -s` and `--stack_size` changes. Hence the diff as part of this PR (which has also been rebased following the merge of the Go1.9...

@neelance - can you take a final look at this please? This issue came up again in the context of #783