|
OLD | NEW |
---|---|
(Empty) | |
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. | |
2 // Use of this source code is governed by a BSD-style license that can be | |
3 // found in the LICENSE file. | |
4 | |
5 #ifndef BASE_BASE_API_H_ | |
6 #define BASE_BASE_API_H_ | |
7 #pragma once | |
8 | |
9 #if !defined(BASE_IMPLEMENTATION) | |
10 #define BASE_IMPLEMENTATION 0 | |
wtc
2011/03/23 22:05:12
I think we should not define BASE_IMPLEMENTATION a
rvargas (doing something else)
2011/03/23 22:59:04
BASE_DLL and BASE_IMPLEMENTATION are defined on th
| |
11 #endif | |
12 | |
13 #if defined(WIN32) && defined(BASE_DLL) | |
wtc
2011/03/23 22:05:12
Nit: I think we prefer our own OS_WIN macro over W
rvargas (doing something else)
2011/03/23 22:59:04
That's an interesting point, but in order to do th
| |
14 #if BASE_IMPLEMENTATION | |
wtc
2011/03/25 18:17:29
Thank you for the reply. I agree base_api.h shoul
brettw
2011/03/26 18:09:46
This sounds fine to me.
rvargas (doing something else)
2011/03/28 18:52:21
Yeah. A CL with this change is up for review (http
| |
15 #define BASE_API __declspec(dllexport) | |
16 #else | |
17 #define BASE_API __declspec(dllimport) | |
18 #endif | |
19 #else | |
20 #define BASE_API | |
21 #endif | |
22 | |
23 #endif // BASE_BASE_API_H_ | |
OLD | NEW |