Commit 3592d21
deps: fix addons compilation with VS2013
VS2013 does not support defaulting move constructor and assignment
operator. This adds explicit definitions of those methods for two
classes.
This fix is required because we still support building addons with
VS2013 and the incompatibility is in v8.h.
Fixes: nodejs/node-v8#4
Backport-PR-URL: #15393
PR-URL: #13263
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>1 parent 97828b2 commit 3592d21
2 files changed
+23
-4
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4104 | 4104 | | |
4105 | 4105 | | |
4106 | 4106 | | |
4107 | | - | |
| 4107 | + | |
| 4108 | + | |
| 4109 | + | |
4108 | 4110 | | |
4109 | 4111 | | |
4110 | | - | |
| 4112 | + | |
4111 | 4113 | | |
4112 | 4114 | | |
4113 | 4115 | | |
| |||
4180 | 4182 | | |
4181 | 4183 | | |
4182 | 4184 | | |
4183 | | - | |
| 4185 | + | |
| 4186 | + | |
| 4187 | + | |
4184 | 4188 | | |
4185 | | - | |
| 4189 | + | |
4186 | 4190 | | |
4187 | 4191 | | |
4188 | 4192 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7723 | 7723 | | |
7724 | 7724 | | |
7725 | 7725 | | |
| 7726 | + | |
| 7727 | + | |
| 7728 | + | |
| 7729 | + | |
| 7730 | + | |
| 7731 | + | |
| 7732 | + | |
| 7733 | + | |
7726 | 7734 | | |
7727 | 7735 | | |
7728 | 7736 | | |
| |||
7836 | 7844 | | |
7837 | 7845 | | |
7838 | 7846 | | |
| 7847 | + | |
| 7848 | + | |
| 7849 | + | |
| 7850 | + | |
| 7851 | + | |
| 7852 | + | |
| 7853 | + | |
7839 | 7854 | | |
7840 | 7855 | | |
7841 | 7856 | | |
| |||
0 commit comments