[v8-dev] Re: Fix compilation for the case dcheck_always_on=1. (issue 920993003 by loi...@chromium.org)

2015-02-16 Thread ishell
On 2015/02/13 10:29:36, I haz the power (commit-bot) wrote: Patchset 4 (id:??) landed as https://crrev.com/2cfebcf92e8518f9c970ce72d2c4fabfae4e1e06 Cr-Commit-Position: refs/heads/master@{#26636} Thank you for taking care of this! https://codereview.chromium.org/920993003/ -- -- v8-dev mailing

[v8-dev] Re: Fix compilation for the case dcheck_always_on=1. (issue 920993003 by loi...@chromium.org)

2015-02-13 Thread commit-bot
Committed patchset #4 (id:80001) https://codereview.chromium.org/920993003/ -- -- v8-dev mailing list v8-dev@googlegroups.com 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 s

[v8-dev] Re: Fix compilation for the case dcheck_always_on=1. (issue 920993003 by loi...@chromium.org)

2015-02-13 Thread commit-bot
Patchset 4 (id:??) landed as https://crrev.com/2cfebcf92e8518f9c970ce72d2c4fabfae4e1e06 Cr-Commit-Position: refs/heads/master@{#26636} https://codereview.chromium.org/920993003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message

[v8-dev] Re: Fix compilation for the case dcheck_always_on=1. (issue 920993003 by loi...@chromium.org)

2015-02-13 Thread commit-bot
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/920993003/80001 https://codereview.chromium.org/920993003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscrib

[v8-dev] Re: Fix compilation for the case dcheck_always_on=1. (issue 920993003 by loi...@chromium.org)

2015-02-13 Thread commit-bot
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/920993003/60001 https://codereview.chromium.org/920993003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscrib

[v8-dev] Re: Fix compilation for the case dcheck_always_on=1. (issue 920993003 by loi...@chromium.org)

2015-02-13 Thread danno
lgtm https://codereview.chromium.org/920993003/ -- -- v8-dev mailing list v8-dev@googlegroups.com 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

[v8-dev] Re: Fix compilation for the case dcheck_always_on=1. (issue 920993003 by loi...@chromium.org)

2015-02-13 Thread loislo
comments addressed. PTAL https://codereview.chromium.org/920993003/ -- -- v8-dev mailing list v8-dev@googlegroups.com 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 rec

[v8-dev] Re: Fix compilation for the case dcheck_always_on=1. (issue 920993003 by loi...@chromium.org)

2015-02-13 Thread danno
Actually, I've been chatting with Ilya and I think the conditional interface is not the way to go. I've asked him to clean up the test to use the style of #ifdef VERIFY_HEAP + flag checking that is used elsewhere in the runtime for heap verification. https://codereview.chromium.org/920993003/

[v8-dev] Re: Fix compilation for the case dcheck_always_on=1. (issue 920993003 by loi...@chromium.org)

2015-02-13 Thread commit-bot
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/920993003/20001 https://codereview.chromium.org/920993003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscrib

[v8-dev] Re: Fix compilation for the case dcheck_always_on=1. (issue 920993003 by loi...@chromium.org)

2015-02-13 Thread svenpanne
lgtm LGTM, although I'm not totally convinced that this is enough. Conditional interfaces are a bad idea, and this #ifdef horror is the price we pay for this... :-/ https://codereview.chromium.org/920993003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev