adam-markovics commented on a change in pull request #1252:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1252#discussion_r798547237



##########
File path: extensions/aws/processors/ListS3.h
##########
@@ -33,6 +33,11 @@ namespace org {
 namespace apache {
 namespace nifi {
 namespace minifi {
+
+namespace core {
+class CoreComponentStateManager;
+}

Review comment:
       Done.

##########
File path: extensions/aws/processors/ListS3.h
##########
@@ -86,7 +91,7 @@ class ListS3 : public S3Processor {
   bool write_object_tags_ = false;
   bool write_user_metadata_ = false;
   bool requester_pays_ = false;
-  std::unique_ptr<minifi::utils::ListingStateManager> state_manager_;
+  std::unique_ptr<minifi::utils::ListingStateManager> state_manager_ = nullptr;

Review comment:
       You are right, reverted. This was due to temporarily turning it into a 
raw pointer and it stayed that way.




-- 
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: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to