Skip to content

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

Merged
jacobperron merged 1 commit intodashingfrom
jacob/dashing_backport_952
Mar 12, 2020
Merged

[dashing backport] Don't specify calling convention in std::_Binder template (#952)#1015
jacobperron merged 1 commit intodashingfrom
jacob/dashing_backport_952

Conversation

@jacobperron
Copy link
Copy Markdown
Member

Backport #952 to Dashing.

FYI @theseankelly

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>
@jacobperron jacobperron self-assigned this Mar 5, 2020
@jacobperron jacobperron changed the title Don't specify calling convention in std::_Binder template (#952) [dashing backport] Don't specify calling convention in std::_Binder template (#952) Mar 5, 2020
@nuclearsandwich
Copy link
Copy Markdown
Member

nuclearsandwich commented Mar 12, 2020

Jobs are being marked as failed due to issues with JUnit XML and the Jenkins plugins on CI right now. I've pasted the colcon test results summaries from each job below the badge.

  • Linux Build Status
    Summary: 1358 tests, 0 errors, 0 failures, 0 skipped

  • Linux-aarch64 Build Status
    Summary: 1260 tests, 0 errors, 0 failures, 0 skipped

  • macOS Build Status
    Summary: 1358 tests, 0 errors, 0 failures, 0 skipped
    Build is unstable due to a warning in tinydir_vendor unrelated to this PR.

  • Windows Build Status
    Summary: 1209 tests, 0 errors, 47 failures, 15 skipped
    Failures are with cyclonedds which are already present in Dashing. There's also a cppcheck failure due to the change in cppcheck version between CI and Dashing.

@jacobperron jacobperron merged commit 8c3fbce into dashing Mar 12, 2020
@delete-merged-branch delete-merged-branch bot deleted the jacob/dashing_backport_952 branch March 12, 2020 21:23
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