threadpool mit std::forward(), eigener Code compiliert nicht mehr

[Gerader Ast (Gast)]

Hallo eine Frage an die C++17 Experten:
Ich nutze die threadpool-Klasse von
https://github.com/fbastos1/thread_pool_cpp17/
die bisher sehr gut tut was sie soll.
Aber dort wurde ein Aufruf geändert, in der execute() - Funktion:

-       _fargs = std::make_tuple(std::forward<Args>(args)...)]() mutable {
+       _fargs = std::forward_as_tuple(std::forward<Args>(args)...)]()

Wenn ich das update, funktioniert mein Code nicht mehr:
Threadfunktion:

static void renderFractalBand(uint ystart, uint yend, CMainWnd* wnd);

Aufruf:

wnd->renderBand.push_back(wnd->fractalPool.execute(renderFractalBand, y, yend, wnd));

Das ist fractalPool und renderBand:

thread_pool fractalPool;
std::vector<std::future<void>> renderBand; // for threaded renderer

Der Compiler meint:

Error C2280: 'thread_pool::execute::<lambda_1>::<lambda_1>(const 
thread_pool::execute::<lambda_1> &)': attempting to reference a deleted 
function und viele weiter Zeilen...

Es scheint so, daß die Argumente des Futures kopiert werden müssen, was 
ja mit einem Pointer gehen sollte.
Da ich mich mit std::future usw. noch nicht so gut auskenne, was könnte 
ich machen damit es wieder geht?

[Oliver S. (oliverso)]

Auch wenn das jetzt nicht die übliche Hausaufgabe-Noob-Frage ist, sind 
Fragen der Art "ich habe irgendwo irgend eine Software heruntergeladen, 
damit irgendwas gemacht, und es funzt irgendwo irgendwas nicht" selten 
sinnvoll zu beantworten. Die klassische Antwort ist da "Fehler in Zeile 
42".

Ein vollständiger compilierbarer (bzw. in dem Fall nicht-compilierbarer) 
Code als Anhang wäre hilfreich.

Oliver

[Oliver S. (oliverso)]

Nachtrag: wie die Fehlermeldung vermuten lässt, und die Doku zu der 
Fehlermeldung bestätigt, wird da auf eine „deleted function“ 
zugegriffen, sehr wahrscheinlich ein fehlender oder auch absichtlich 
verhinderter copy- oder move-constructor.
Welche Klasse das auslöst, steht mit Sicherheit weiter unten in der 
Fehlermeldung, aber das musst du selber lesen. Das warum ist dann auch 
deine Sache.

https://docs.microsoft.com/de-de/cpp/error-messages/compiler-errors-1/compiler-error-c2280?view=msvc-160

Oliver

[Gerader Ast (Gast)]

Hallo, danke erstmal für die Antworten. Auch wenn es wohl schwieriger 
als gedacht ist.
Habe gerade das Beispiel von der Ersteller-Seite des Threadpools 
probiert,
es funktioniert auch nicht mehr mit der letzten Änderung des Erstellers 
in "threadpool.h".

Im Anhang findet ihr die Quellen einer Konsolenanwendung des Beispiels 
von der Erstellerseite.

Wenn ihr in "threadpool.h" die Zeile 88 von #if 0 zu #if 1 ändert, 
compiliert das Beispiel und läuft, ansonsten gleicher Fehler wie bei 
mir.
Und leider weiß ich nicht was da zu ändern wäre, vielleicht mal dem 
Ersteller des Threadpools mailen?

Hier nochmal das Beispiel:

// example from 
// https://github.com/fbastos1/thread_pool_cpp17/

#include <iostream>
#include <vector>
#include "threadpool.h"

int multiply(int x, int y) {
  return x * y;
}

int main() {
  thread_pool pool(4);
  std::vector<std::future<int>> futures;

  for (const int& x : { 2, 4, 7, 13 }) {
    futures.push_back(pool.execute(multiply, x, 2));
  }

  for (auto& fut : futures) {
    std::cout << fut.get() << std::endl;
  }

  return 0;
}

[Gerader Ast (Gast)]

Achso, vergessen: Visual Studio 2019 Windows.

[mh (Gast)]

Gibts nen Grund, warum du nicht auf github um Hilfe fragst? Das ist doch 
geradezu ideal, um nen Issue zu öffnen "Beispiel aus README.ME 
funktioniert nicht mit mvsc 2019". Übrigens der einzig offene Issue hat 
mit genau der Zeile zu tun ...

[Gerader Ast (Gast)]

Oh, danke. Habe da noch nicht nachgeschaut.

[Oliver S. (oliverso)]

Gerader Ast schrieb:
> die bisher sehr gut tut was sie soll.

Die einfachste Lösung ist allerdings noch viel naheliegender...
Never change a running system

Oliver