fixed directive for Net.cc + initialize double values in floodfill to prevent randomly failing test

Author: justadudewhohacks

cc/core/MatImgprocBindings.h

@@ -1677,12 +1677,12 @@ namespace MatImgprocBindings {
     }
 
     cv::Point2d seedPoint;
-    double newVal1;
+    double newVal1 = 0;
     cv::Vec3d newVal3 = cv::Vec3d();
     cv::Mat mask = cv::noArray().getMat();
-    double loDiff1;
+    double loDiff1 = 0;
     cv::Vec3d loDiff3 = cv::Vec3d();
-    double upDiff1;
+    double upDiff1 = 0;
     cv::Vec3d upDiff3 = cv::Vec3d();
     int flags = 4;
 

cc/modules/dnn/Net.cc

@@ -24,9 +24,6 @@ NAN_MODULE_INIT(Net::Init) {
   target->Set(Nan::New("Net").ToLocalChecked(), ctor->GetFunction());
 };
 
-//#endif
-
-#endif
 NAN_METHOD(Net::New) {
   Net* self = new Net();
   self->Wrap(info.Holder());
@@ -64,3 +61,5 @@ NAN_METHOD(Net::ForwardAsync) {
     info
   );
 }
+
+#endif

test/tests/core/Mat/imgprocTests.js

@@ -1241,7 +1241,7 @@ module.exports = (getTestImg) => {
     });
   });
 
-  describe('floodFill', () => {
+  describe.only('floodFill', () => {
     const expectOutput = (out) => {
       expect(!!out).to.be.true;
       expect(out).to.have.property('returnValue').to.be.a('number');