Skip to content

Commit

Permalink
src: use visibility("default") exports on POSIX
Browse files Browse the repository at this point in the history
Electron uses this because Chromium builds with symbols
hidden by default.

Refs:https://github.com/electron/node/commit/
88b4941
Refs: https://github.com/electron/node/commit/
1293d1d7d0c33d3925da11ceccdce4eb2e927a43

PR-URL: #25893
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Gus Caplan <[email protected]>
  • Loading branch information
nornagon authored and addaleax committed Feb 6, 2019
1 parent cc4ae20 commit 2e80b91
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/js_native_api.h
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
#ifdef _WIN32
#define NAPI_EXTERN __declspec(dllexport)
#else
#define NAPI_EXTERN /* nothing */
#define NAPI_EXTERN __attribute__((visibility("default")))
#endif
#endif

Expand Down
2 changes: 1 addition & 1 deletion src/node.h
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
# define NODE_EXTERN __declspec(dllimport)
# endif
#else
# define NODE_EXTERN /* nothing */
# define NODE_EXTERN __attribute__((visibility("default")))
#endif

#ifdef BUILDING_NODE_EXTENSION
Expand Down

0 comments on commit 2e80b91

Please sign in to comment.