test/win/idl-includedirs/subdir/foo.idl - Issue 602073003: Add support for MIDL include directories

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: test/win/idl-includedirs/subdir/foo.idl

Issue 602073003: Add support for MIDL include directories (Closed) Base URL: http://gyp.googlecode.com/svn/trunk

Patch Set: Created 6 years, 2 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) 2014 The Chromium Authors. All rights reserved.	1 // Copyright (c) 2014 The Chromium Authors. All rights reserved.

2 // Use of this source code is governed by a BSD-style license that can be	2 // Use of this source code is governed by a BSD-style license that can be

3 // found in the LICENSE file.	3 // found in the LICENSE file.

4	4

5 This is a dummy .idl file that will trigger an error if it is not excluded from	5 import "oaidl.idl";

6 the build.	6 import "bar.idl";
	scottmg 2014/09/25 16:53:10 would this have been found based on "." probably b would this have been found based on "." probably being in the default search locations anyway? Shezan Baig (Bloomberg) 2014/09/25 17:00:21 Did you mean '.' relative to the 'subdir'? I don' Show quoted text On 2014/09/25 16:53:10, scottmg wrote: > would this have been found based on "." probably being in the default search > locations anyway? Did you mean '.' relative to the 'subdir'? I don't think that works, because the test would fail if I remove 'subdir' from 'midl_include_dirs'. scottmg 2014/09/25 17:05:31 Yes, that's what I was obtusely getting at. :) Tha Show quoted text On 2014/09/25 17:00:21, Shezan Baig (Bloomberg) wrote: > On 2014/09/25 16:53:10, scottmg wrote: > > would this have been found based on "." probably being in the default search > > locations anyway? > > Did you mean '.' relative to the 'subdir'? I don't think that works, because > the test would fail if I remove 'subdir' from 'midl_include_dirs'. Yes, that's what I was obtusely getting at. :) Thanks for confirming that it doesn't work without the midl_include_dirs addition.
	7

	8 [

	9 object,

	10 uuid(9C1100DD-51D4-4827-AE9F-3B8FAC4AED72),

	11 ]

	12 interface Foo : IUnknown {

	13 HRESULT FooFunction(Bar* bar);

	14 };

OLD	NEW

« test/win/idl-includedirs/subdir/bar.idl ('K') | « test/win/idl-includedirs/subdir/bar.idl ('k') | no next file » | no next file with comments »