runtime/vm/native_arguments.cc - Issue 9189003: Move assert.h/assert.cc from runtime/vm to runtime/platform

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: runtime/vm/native_arguments.cc

Issue 9189003: Move assert.h/assert.cc from runtime/vm to runtime/platform (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart

Patch Set: Fixing vm/assert.h for _macos, _win, _arm and _x64 files Created 8 years, 11 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch | Annotate | Revision Log

OLD	NEW
1 // Copyright (c) 2011, the Dart project authors. Please see the AUTHORS file	1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file

2 // for details. All rights reserved. Use of this source code is governed by a	2 // for details. All rights reserved. Use of this source code is governed by a

3 // BSD-style license that can be found in the LICENSE file.	3 // BSD-style license that can be found in the LICENSE file.

4	4

	5 #include "platform/assert.h"

	6 #include "vm/globals.h"
	Mads Ager (chromium) 2012/01/13 14:42:08 Why is globals.h needed here? Why is globals.h needed here? Søren Gjesse 2012/01/16 08:58:00 vm/assert.h used to include vm/globals.h. It vas a Show quoted text On 2012/01/13 14:42:08, Mads Sig Ager wrote: > Why is globals.h needed here? vm/assert.h used to include vm/globals.h. It vas actually needed in natives_arguments.h, so moved it there instead.
5 #include "vm/native_arguments.h"	7 #include "vm/native_arguments.h"
	Ivan Posva 2012/01/13 23:22:06 First. First. Søren Gjesse 2012/01/16 08:58:00 Done. Show quoted text On 2012/01/13 23:22:06, Ivan Posva wrote: > First. Done.
6

7 #include "vm/assert.h"

8 #include "vm/object.h"	8 #include "vm/object.h"

9	9

10 namespace dart {	10 namespace dart {

11	11

12 void NativeArguments::SetReturn(const Object& value) const {	12 void NativeArguments::SetReturn(const Object& value) const {

13 *retval_ = value.raw();	13 *retval_ = value.raw();

14 }	14 }

15	15

16 } // namespace dart	16 } // namespace dart

OLD	NEW

« runtime/vm/isolate.cc ('K') | « runtime/vm/native_arguments.h ('k') | runtime/vm/object.h » ('j') | runtime/vm/object.cc » ('J')