Github user arpadboda commented on a diff in the pull request:
https://github.com/apache/nifi-minifi-cpp/pull/457#discussion_r239776939
--- Diff: extensions/mqtt/processors/AbstractMQTTProcessor.h ---
@@ -157,7 +157,6 @@ class AbstractMQTTProcessor : public core::Processor {
std::string securityPrivateKeyPassWord_;
};
-REGISTER_RESOURCE(AbstractMQTTProcessor);
--- End diff --
Good spot, however could you make the class abstact, too?
(REGISTER_RESOURCE fails to compile in case the argument is abstact class, so
it makes sure it doesn't happen again)
I'm also wondering whet making "Processor" abstact would make sense or not.
If you create a follow-up Jira, I can do as well.
---
