Skip to content

[eloquent backport] Don't specify calling convention in std::_Binder template (#952)#1006

Merged
ivanpauno merged 1 commit intoros2:eloquentfrom
theseankelly:eloquent_uwp
Feb 28, 2020
Merged

[eloquent backport] Don't specify calling convention in std::_Binder template (#952)#1006
ivanpauno merged 1 commit intoros2:eloquentfrom
theseankelly:eloquent_uwp

Conversation

@theseankelly
Copy link
Copy Markdown
Contributor

Fix for a build error on 32-bit Windows. Member functions use the
__thiscall convention by default which is incompatible with __cdecl.

Signed-off-by: Sean Kelly sean@seankelly.dev

Fix for a build error on 32-bit Windows. Member functions use the
__thiscall convention by default which is incompatible with __cdecl.

Signed-off-by: Sean Kelly <sean@seankelly.dev>
@ivanpauno
Copy link
Copy Markdown
Member

@ros-pull-request-builder retest this please.

1 similar comment
@ivanpauno
Copy link
Copy Markdown
Member

@ros-pull-request-builder retest this please.

@ivanpauno
Copy link
Copy Markdown
Member

The failures shown in the different runs of the checker were different, and completly unrelated to this PR (just flaky tests). I will proceed to merge.

@ivanpauno ivanpauno merged commit 863970e into ros2:eloquent Feb 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants