lordgamez commented on a change in pull request #1082:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1082#discussion_r640704122



##########
File path: extensions/librdkafka/tests/ConsumeKafkaTests.cpp
##########
@@ -422,7 +422,7 @@ class ConsumeKafkaContinuousPublishingTest : public 
ConsumeKafkaTest {
       int64_t value;
       org::apache::nifi::minifi::core::TimeUnit unit;
       
REQUIRE(org::apache::nifi::minifi::core::Property::StringToTime(property_string,
 value, unit));
-      int64_t value_as_ms;
+      int64_t value_as_ms = 0;

Review comment:
       You are right, it just came up as an error while compiling and I fixed 
it on the fly. I removed it and will have it in a separate PR.




-- 
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.

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


Reply via email to