szaszm commented on code in PR #1349:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1349#discussion_r907261175


##########
libminifi/test/SingleProcessorTestController.h:
##########
@@ -57,6 +58,14 @@ class SingleProcessorTestController : public TestController {
     return trigger();
   }
 
+  auto trigger(std::initializer_list<std::tuple<const std::string_view, 
std::unordered_map<std::string, std::string>>> flow_files) {

Review Comment:
   If you remove the constructor, then you can use aggregate initialization 
without having to spell out the type name of the flow file data, and without 
implicit constructors. The downside is that I think you need to add an empty 
pair of braces for the attributes every time, but maybe even that is optional 
if you add a default member initializer.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to