Project

General

Profile

Actions
Copy link

Bug #9864

closed
RS RM

Wt::Signal doesn't properly bind to function object with 4 or more arguments

Bug #9864: Wt::Signal doesn't properly bind to function object with 4 or more arguments

Added by Roel Standaert over 4 years ago. Updated 8 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Romain Mardulyn
Target version:
4.12.2
Start date:
03/04/2022
Due date:
% Done:

100%

Estimated time:

Description

We'll need to revise how signals are connected to slots, we have some ConnectHelper specializations in signals.hpp for 0, 1, 2, or 3 arguments, but I feel like with C++14 we should be able to make it work with any number?

For example, this doesn't compile:

  Wt::Signal<std::string, std::string, std::string, std::string> signal;

  signal.connect([](const std::string &s1,
                    const std::string &s2,
                    const std::string &s3,
                    const std::string &s4) {
    assert(s1 == "a");
    assert(s2 == "b");
    assert(s3 == "c");
    assert(s4 == "d");
  });

  signal.emit("a", "b", "c", "d");

RM Updated by Romain Mardulyn over 1 year ago Actions
Copy link
 #1

  • Status changed from New to InProgress
  • Assignee set to Romain Mardulyn

RM Updated by Romain Mardulyn over 1 year ago Actions
Copy link
 #2

  • Status changed from InProgress to Review
  • Assignee deleted (Romain Mardulyn)

RM Updated by Romain Mardulyn over 1 year ago Actions
Copy link
 #3

  • Target version changed from future to 4.12.2

MV Updated by Matthias Van Ceulebroeck 11 months ago Actions
Copy link
 #4

  • Assignee set to Matthias Van Ceulebroeck

MV Updated by Matthias Van Ceulebroeck 11 months ago Actions
Copy link
 #5

  • Status changed from Review to Implemented @Emweb
  • Assignee changed from Matthias Van Ceulebroeck to Romain Mardulyn
  • % Done changed from 0 to 100

RM Updated by Romain Mardulyn 8 months ago Actions
Copy link
 #6

  • Status changed from Implemented @Emweb to Closed
Actions
Copy link

Also available in: PDF Atom