2025-09-01 06:52 ftp://ftp.de.debian.org/debian/pool/main/libo/libopenshot/libopenshot_0.3.3+dfsg1.orig.tar.xz cppcheck-options: --library=posix --library=gnu --library=bsd --library=qt --library=opencv2 --library=openmp --inconclusive --enable=style,information --inline-suppr --template=daca2 --disable=missingInclude --suppress=unmatchedSuppression --check-library --debug-warnings --suppress=autoNoType --suppress=valueFlowBailout --suppress=bailoutUninitVar --suppress=symbolDatabaseWarning --suppress=normalCheckLevelConditionExpressions -D__GNUC__ --platform=unix64 -j4 platform: Linux-6.12.41+deb13-amd64-x86_64-with-glibc2.41 python: 3.13.5 client-version: 1.3.67 compiler: g++ (Debian 14.2.0-19) 14.2.0 cppcheck: head 2.18.0 head-info: 2ee39fe (2025-08-30 22:07:31 +0200) count: 475 1195 elapsed-time: 0.7 2.9 head-timing-info: old-timing-info: head results: libopenshot-0.3.3/examples/ExampleHtml.cpp:44:16: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable GRAVITY_BOTTOM_RIGHT [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/examples/Example.cpp:33:31: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable ReadOnly [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/examples/ExampleHtml.cpp:67:32: style: Variable 'w.info.metadata["comment"]' is reassigned a value before the old one has been used. [redundantAssignment] libopenshot-0.3.3/examples/ExampleHtml.cpp:66:32: note: w.info.metadata["comment"] is assigned libopenshot-0.3.3/examples/ExampleHtml.cpp:67:32: note: w.info.metadata["comment"] is overwritten libopenshot-0.3.3/examples/Example.cpp:61:19: style: Modulo of one is always equal to zero [moduloofone] libopenshot-0.3.3/examples/Example_opencv.cpp:47:42: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable WINDOW_NORMAL [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/examples/Example_opencv.cpp:82:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/AudioDevices.cpp:29:11: style: Variable 'types' can be declared as reference to const [constVariableReference] libopenshot-0.3.3/src/FFmpegUtilities.h:70:0: error: failed to evaluate #if condition, undefined function-like macro invocation: AV_VERSION_INT( ... ) [preprocessorErrorDirective] libopenshot-0.3.3/src/FFmpegUtilities.h:70:2: error: failed to evaluate #if condition, undefined function-like macro invocation: AV_VERSION_INT( ... ) [preprocessorErrorDirective] libopenshot-0.3.3/src/AudioDevices.cpp:34:23: style: Consider using std::transform algorithm instead of a raw loop. [useStlAlgorithm] libopenshot-0.3.3/src/AudioBufferSource.cpp:77:25: performance:inconclusive: Either there is a missing 'override', or the member function 'openshot::AudioBufferSource::prepareToPlay' can be static. [functionStatic] libopenshot-0.3.3/src/AudioBufferSource.h:47:8: note: Either there is a missing 'override', or the member function 'openshot::AudioBufferSource::prepareToPlay' can be static. libopenshot-0.3.3/src/AudioBufferSource.cpp:77:25: note: Either there is a missing 'override', or the member function 'openshot::AudioBufferSource::prepareToPlay' can be static. libopenshot-0.3.3/src/AudioBufferSource.cpp:80:25: performance:inconclusive: Either there is a missing 'override', or the member function 'openshot::AudioBufferSource::releaseResources' can be static. [functionStatic] libopenshot-0.3.3/src/AudioBufferSource.h:50:8: note: Either there is a missing 'override', or the member function 'openshot::AudioBufferSource::releaseResources' can be static. libopenshot-0.3.3/src/AudioBufferSource.cpp:80:25: note: Either there is a missing 'override', or the member function 'openshot::AudioBufferSource::releaseResources' can be static. libopenshot-0.3.3/src/AudioBufferSource.h:37:3: style: Class 'AudioBufferSource' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/AudioResampler.cpp:38:2: warning: Class 'AudioResampler' does not have a copy constructor which is recommended since it has dynamic memory/resource management. [noCopyConstructor] libopenshot-0.3.3/src/AudioResampler.cpp:38:2: warning: Class 'AudioResampler' does not have a operator= which is recommended since it has dynamic memory/resource management. [noOperatorEq] libopenshot-0.3.3/src/AudioResampler.h:46:3: style: Class 'AudioResampler' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/examples/Example_opencv.cpp:143:8: style: The if condition is the same as the previous if condition [duplicateCondition] libopenshot-0.3.3/examples/Example_opencv.cpp:77:25: note: 'SMOOTH_VIDEO' is assigned value 'false' here. libopenshot-0.3.3/examples/Example_opencv.cpp:78:34: note: 'OBJECT_DETECTION_DATA' is assigned value 'false' here. libopenshot-0.3.3/examples/Example_opencv.cpp:121:8: note: First condition libopenshot-0.3.3/examples/Example_opencv.cpp:143:8: note: Second condition libopenshot-0.3.3/examples/Example_opencv.cpp:82:34: style: Condition 'OBJECT_DETECTION_DATA' is always false [knownConditionTrueFalse] libopenshot-0.3.3/examples/Example_opencv.cpp:78:34: note: Assignment 'OBJECT_DETECTION_DATA=false', assigned value is 0 libopenshot-0.3.3/examples/Example_opencv.cpp:82:34: note: Condition 'OBJECT_DETECTION_DATA' is always false libopenshot-0.3.3/examples/Example_opencv.cpp:94:8: style: Condition 'TRACK_DATA' is always true [knownConditionTrueFalse] libopenshot-0.3.3/examples/Example_opencv.cpp:76:23: note: Assignment 'TRACK_DATA=true', assigned value is 1 libopenshot-0.3.3/examples/Example_opencv.cpp:94:8: note: Condition 'TRACK_DATA' is always true libopenshot-0.3.3/examples/Example_opencv.cpp:188:26: performance: Function parameter 'key' should be passed by const reference. [passedByValue] libopenshot-0.3.3/examples/Example_opencv.cpp:188:38: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/examples/Example_opencv.cpp:188:52: performance: Function parameter 'type' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/CacheBase.cpp:49:42: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable arrayValue [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/CacheBase.cpp:21:12: warning: Member variable 'CacheBase::needs_range_processing' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/CacheBase.cpp:21:12: warning: Member variable 'CacheBase::range_version' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/CacheBase.cpp:24:12: warning: Member variable 'CacheBase::needs_range_processing' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/CacheBase.cpp:24:12: warning: Member variable 'CacheBase::range_version' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/CacheBase.h:97:11: style:inconclusive: Technically the member function 'openshot::CacheBase::GetMaxBytes' can be const. [functionConst] libopenshot-0.3.3/src/CacheBase.cpp:26:2: warning: Class 'CacheBase' does not have a copy constructor which is recommended since it has dynamic memory/resource management. [noCopyConstructor] libopenshot-0.3.3/src/CacheBase.cpp:26:2: warning: Class 'CacheBase' does not have a operator= which is recommended since it has dynamic memory/resource management. [noOperatorEq] libopenshot-0.3.3/src/Json.h:21:54: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/CacheBase.h:113:42: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/CrashHandler.cpp:39:17: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable SA_SIGINFO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/CrashHandler.cpp:100:12: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable stderr [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/CrashHandler.cpp:285:21: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable left [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/Fraction.h:76:24: style:inconclusive: Technically the member function 'openshot::Fraction::operator/' can be const. [functionConst] libopenshot-0.3.3/src/Fraction.h:42:2: style: Class 'Fraction' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Fraction.h:45:2: style: Class 'Fraction' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Fraction.h:48:2: style: Class 'Fraction' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Coordinate.h:53:2: style: Class 'Coordinate' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:32:3: style: Class 'ExceptionBase' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:48:9: style: Class 'FrameExceptionBase' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:65:9: style: Class 'FileExceptionBase' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:106:3: style: Class 'DecklinkError' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:121:3: style: Class 'ErrorDecodingAudio' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:136:3: style: Class 'ErrorEncodingAudio' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:151:3: style: Class 'ErrorEncodingVideo' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:166:3: style: Class 'InvalidChannels' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:181:3: style: Class 'InvalidCodec' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:211:3: style: Class 'InvalidFormat' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:226:3: style: Class 'InvalidJSON' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:241:3: style: Class 'InvalidOptions' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:256:3: style: Class 'InvalidSampleRate' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:294:3: style: Class 'NoStreamsFound' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:357:3: style: Class 'OutOfMemory' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:372:3: style: Class 'ReaderClosed' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:387:3: style: Class 'ResampleError' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:408:3: style: Class 'TooManySeeks' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:424:3: style: Class 'WriterClosed' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Exceptions.h:93:12: style: The destructor '~ChunkNotFound' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:93:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:108:12: style: The destructor '~DecklinkError' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:108:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:123:12: style: The destructor '~ErrorDecodingAudio' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:123:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:138:12: style: The destructor '~ErrorEncodingAudio' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:138:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:153:12: style: The destructor '~ErrorEncodingVideo' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:153:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:168:12: style: The destructor '~InvalidChannels' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:168:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:183:12: style: The destructor '~InvalidCodec' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:183:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:198:12: style: The destructor '~InvalidFile' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:198:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:213:12: style: The destructor '~InvalidFormat' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:213:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:228:12: style: The destructor '~InvalidJSON' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:228:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:243:12: style: The destructor '~InvalidOptions' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:243:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:258:12: style: The destructor '~InvalidSampleRate' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:258:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:274:12: style: The destructor '~InvalidJSONKey' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:274:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:296:12: style: The destructor '~NoStreamsFound' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:296:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:314:12: style: The destructor '~OutOfBoundsFrame' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:314:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:338:12: style: The destructor '~OutOfBoundsPoint' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:338:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:359:12: style: The destructor '~OutOfMemory' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:359:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:374:12: style: The destructor '~ReaderClosed' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:374:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:389:12: style: The destructor '~ResampleError' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:389:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:410:12: style: The destructor '~TooManySeeks' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:410:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:426:12: style: The destructor '~WriterClosed' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/Exceptions.h:33:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Exceptions.h:426:12: note: Destructor in derived class libopenshot-0.3.3/src/Exceptions.h:32:29: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:48:40: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:65:39: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:65:60: performance: Function parameter 'file_path' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:91:29: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:106:29: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:121:34: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:136:34: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:151:34: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:166:31: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:166:52: performance: Function parameter 'file_path' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:181:28: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:181:49: performance: Function parameter 'file_path' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:196:27: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:196:48: performance: Function parameter 'file_path' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:211:29: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:211:50: performance: Function parameter 'file_path' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:226:27: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:226:48: performance: Function parameter 'file_path' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:241:30: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:241:51: performance: Function parameter 'file_path' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:256:33: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:256:54: performance: Function parameter 'file_path' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:272:30: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:272:51: performance: Function parameter 'json' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:294:30: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:294:51: performance: Function parameter 'file_path' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:312:32: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:336:32: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:357:27: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:357:48: performance: Function parameter 'file_path' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:372:28: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:372:49: performance: Function parameter 'file_path' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:387:29: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:387:50: performance: Function parameter 'file_path' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:408:28: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:408:49: performance: Function parameter 'file_path' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:424:28: performance: Function parameter 'message' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Exceptions.h:424:49: performance: Function parameter 'file_path' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Coordinate.cpp:53:44: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Color.cpp:47:20: style:inconclusive: Technically the member function 'openshot::Color::GetColorHex' can be const. [functionConst] libopenshot-0.3.3/src/Color.h:52:14: note: Technically the member function 'openshot::Color::GetColorHex' can be const. libopenshot-0.3.3/src/Color.cpp:47:20: note: Technically the member function 'openshot::Color::GetColorHex' can be const. libopenshot-0.3.3/src/Point.h:76:2: style: Class 'Point' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Point.h:85:2: style: Class 'Point' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/KeyFrame.h:64:3: style: Class 'Keyframe' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/KeyFrame.h:67:3: style: Class 'Keyframe' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Coordinate.h:58:33: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Point.h:106:33: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/KeyFrame.h:127:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Color.cpp:28:68: performance: Function parameter 'Alpha' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Color.cpp:40:26: performance: Function parameter 'color_hex' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Color.cpp:100:39: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/ZmqLogger.h:49:3: warning: Member variable 'ZmqLogger::enabled' is not initialized in the constructor. [uninitMemberVarPrivate] libopenshot-0.3.3/src/ZmqLogger.h:49:3: warning: Member variable 'ZmqLogger::context' is not initialized in the constructor. [uninitMemberVarPrivate] libopenshot-0.3.3/src/ZmqLogger.h:49:3: warning: Member variable 'ZmqLogger::publisher' is not initialized in the constructor. [uninitMemberVarPrivate] libopenshot-0.3.3/src/ZmqLogger.h:59:3: warning:inconclusive: Member variable 'ZmqLogger::loggerMutex' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/ZmqLogger.h:59:3: warning:inconclusive: Member variable 'ZmqLogger::connection' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/ZmqLogger.h:59:3: warning:inconclusive: Member variable 'ZmqLogger::file_path' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/ZmqLogger.h:59:3: warning:inconclusive: Member variable 'ZmqLogger::log_file' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/ZmqLogger.h:59:3: warning:inconclusive: Member variable 'ZmqLogger::enabled' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/ZmqLogger.h:59:3: warning: Member variable 'ZmqLogger::context' is not initialized in the copy constructor. [uninitMemberVarPrivate] libopenshot-0.3.3/src/ZmqLogger.h:59:3: warning: Member variable 'ZmqLogger::publisher' is not initialized in the copy constructor. [uninitMemberVarPrivate] libopenshot-0.3.3/src/CrashHandler.cpp:256:19: warning: If memory allocation fails, then there is a possible null pointer dereference: symbollist [nullPointerOutOfMemory] libopenshot-0.3.3/src/CrashHandler.cpp:207:39: note: Assuming allocation function fails libopenshot-0.3.3/src/CrashHandler.cpp:207:39: note: Assignment 'symbollist=backtrace_symbols(addrlist,addrlen)', assigned value is 0 libopenshot-0.3.3/src/CrashHandler.cpp:256:19: note: Null pointer dereference libopenshot-0.3.3/src/CrashHandler.cpp:278:10: style: Variable 'fname' can be declared as pointer to const [constVariablePointer] libopenshot-0.3.3/src/Fraction.cpp:50:15: style:inconclusive: Technically the member function 'openshot::Fraction::GreatestCommonDenominator' can be const. [functionConst] libopenshot-0.3.3/src/Fraction.h:51:6: note: Technically the member function 'openshot::Fraction::GreatestCommonDenominator' can be const. libopenshot-0.3.3/src/Fraction.cpp:50:15: note: Technically the member function 'openshot::Fraction::GreatestCommonDenominator' can be const. libopenshot-0.3.3/src/Fraction.cpp:35:17: style:inconclusive: Technically the member function 'openshot::Fraction::ToFloat' can be const. [functionConst] libopenshot-0.3.3/src/Fraction.h:57:8: note: Technically the member function 'openshot::Fraction::ToFloat' can be const. libopenshot-0.3.3/src/Fraction.cpp:35:17: note: Technically the member function 'openshot::Fraction::ToFloat' can be const. libopenshot-0.3.3/src/Fraction.cpp:45:15: style:inconclusive: Technically the member function 'openshot::Fraction::ToInt' can be const. [functionConst] libopenshot-0.3.3/src/Fraction.h:63:6: note: Technically the member function 'openshot::Fraction::ToInt' can be const. libopenshot-0.3.3/src/Fraction.cpp:45:15: note: Technically the member function 'openshot::Fraction::ToInt' can be const. libopenshot-0.3.3/src/Fraction.cpp:56:6: style: The scope of the variable 't' can be reduced. [variableScope] libopenshot-0.3.3/src/Fraction.cpp:27:37: performance: Function parameter 'vector' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/OpenShotVersion.cpp:18:26: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Version [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/Json.cpp:16:60: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Point.cpp:87:39: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/KeyFrame.cpp:343:37: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable arrayValue [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/KeyFrame.cpp:401:57: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Y [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/KeyFrame.cpp:471:15: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable right [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/KeyFrame.cpp:485:15: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable right [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/KeyFrame.cpp:525:26: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable X [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/KeyFrame.cpp:534:31: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Y [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/Profiles.cpp:47:33: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable ReadOnly [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/Profiles.cpp:132:22: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable fixed [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/Profiles.cpp:215:34: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable objectValue [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/Qt/VideoPlaybackThread.cpp:36:34: style:inconclusive: Either there is a missing 'override', or the member function 'openshot::VideoPlaybackThread::getCurrentFramePosition' can be const. [functionConst] libopenshot-0.3.3/src/Qt/VideoPlaybackThread.h:44:10: note: Either there is a missing 'override', or the member function 'openshot::VideoPlaybackThread::getCurrentFramePosition' can be const. libopenshot-0.3.3/src/Qt/VideoPlaybackThread.cpp:36:34: note: Either there is a missing 'override', or the member function 'openshot::VideoPlaybackThread::getCurrentFramePosition' can be const. libopenshot-0.3.3/src/Qt/VideoPlaybackThread.h:39:2: style: Class 'VideoPlaybackThread' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/Profiles.h:143:3: style: Class 'Profile' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/KeyFrame.cpp:166:35: performance: Function parameter 'p' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/KeyFrame.cpp:221:39: performance: Function parameter 'p' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/KeyFrame.cpp:226:40: performance: Function parameter 'p' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/KeyFrame.cpp:346:18: performance: Range variable 'existing_point' should be declared as const reference. [iterateByValue] libopenshot-0.3.3/src/KeyFrame.cpp:355:42: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/KeyFrame.cpp:430:34: performance: Function parameter 'p' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/KeyFrame.cpp:462:49: performance: Function parameter 'p' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Profiles.cpp:231:41: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/Profiles.cpp:42:2: style: Instance of 'Profile' object is destroyed immediately. [unusedScopedObject] libopenshot-0.3.3/src/Qt/VideoCacheThread.cpp:29:20: warning: Member variable 'VideoCacheThread::requested_display_frame' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/Qt/VideoCacheThread.h:37:10: style:inconclusive: Member variable 'VideoCacheThread::current_display_frame' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/Qt/VideoCacheThread.cpp:31:16: note: Member variable 'VideoCacheThread::current_display_frame' is in the wrong place in the initializer list. libopenshot-0.3.3/src/Qt/VideoCacheThread.h:37:10: note: Member variable 'VideoCacheThread::current_display_frame' is in the wrong place in the initializer list. libopenshot-0.3.3/src/Qt/VideoCacheThread.h:42:10: style:inconclusive: Member variable 'VideoCacheThread::timeline_max_frame' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/Qt/VideoCacheThread.cpp:33:2: note: Member variable 'VideoCacheThread::timeline_max_frame' is in the wrong place in the initializer list. libopenshot-0.3.3/src/Qt/VideoCacheThread.h:42:10: note: Member variable 'VideoCacheThread::timeline_max_frame' is in the wrong place in the initializer list. libopenshot-0.3.3/src/Qt/VideoCacheThread.cpp:135:28: style:inconclusive: Either there is a missing 'override', or the member function 'openshot::VideoCacheThread::isReady' can be const. [functionConst] libopenshot-0.3.3/src/Qt/VideoCacheThread.h:84:14: note: Either there is a missing 'override', or the member function 'openshot::VideoCacheThread::isReady' can be const. libopenshot-0.3.3/src/Qt/VideoCacheThread.cpp:135:28: note: Either there is a missing 'override', or the member function 'openshot::VideoCacheThread::isReady' can be const. libopenshot-0.3.3/src/Qt/VideoCacheThread.cpp:55:30: warning: Either the condition 'reader' is redundant or there is possible null pointer dereference: t. [nullPointerRedundantCheck] libopenshot-0.3.3/src/Qt/VideoCacheThread.cpp:85:30: note: Assuming that condition 'reader' is not redundant libopenshot-0.3.3/src/Qt/VideoCacheThread.cpp:52:21: note: t is assigned '(Timeline*)reader' here. libopenshot-0.3.3/src/Qt/VideoCacheThread.cpp:55:30: note: Null pointer dereference libopenshot-0.3.3/src/Qt/VideoCacheThread.cpp:52:23: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] libopenshot-0.3.3/src/Qt/VideoCacheThread.cpp:186:31: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] libopenshot-0.3.3/src/Qt/VideoCacheThread.cpp:148:23: style: Variable 's' can be declared as pointer to const [constVariablePointer] libopenshot-0.3.3/src/Qt/VideoCacheThread.cpp:186:27: style: Variable 't' can be declared as pointer to const [constVariablePointer] libopenshot-0.3.3/src/Settings.h:40:3: warning:inconclusive: Member variable 'Settings::HARDWARE_DECODER' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/Settings.h:40:3: warning:inconclusive: Member variable 'Settings::HIGH_QUALITY_SCALING' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/Settings.h:40:3: warning:inconclusive: Member variable 'Settings::OMP_THREADS' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/Settings.h:40:3: warning:inconclusive: Member variable 'Settings::FF_THREADS' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/Settings.h:40:3: warning:inconclusive: Member variable 'Settings::DE_LIMIT_HEIGHT_MAX' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/Settings.h:40:3: warning:inconclusive: Member variable 'Settings::DE_LIMIT_WIDTH_MAX' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/Settings.h:40:3: warning:inconclusive: Member variable 'Settings::HW_DE_DEVICE_SET' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/Settings.h:40:3: warning:inconclusive: Member variable 'Settings::HW_EN_DEVICE_SET' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/Settings.h:40:3: warning:inconclusive: Member variable 'Settings::VIDEO_CACHE_PERCENT_AHEAD' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/Settings.h:40:3: warning:inconclusive: Member variable 'Settings::VIDEO_CACHE_MIN_PREROLL_FRAMES' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/Settings.h:40:3: warning:inconclusive: Member variable 'Settings::VIDEO_CACHE_MAX_PREROLL_FRAMES' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/Settings.h:40:3: warning:inconclusive: Member variable 'Settings::VIDEO_CACHE_MAX_FRAMES' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/Settings.h:40:3: warning:inconclusive: Member variable 'Settings::ENABLE_PLAYBACK_CACHING' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/Settings.h:40:3: warning:inconclusive: Member variable 'Settings::PLAYBACK_AUDIO_DEVICE_NAME' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/Settings.h:40:3: warning:inconclusive: Member variable 'Settings::PLAYBACK_AUDIO_DEVICE_TYPE' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/Settings.h:40:3: warning:inconclusive: Member variable 'Settings::PATH_OPENSHOT_INSTALL' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/Settings.h:40:3: warning:inconclusive: Member variable 'Settings::DEBUG_TO_STDERR' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] libopenshot-0.3.3/src/Settings.cpp:44:8: style: Variable 'env_debug' can be declared as pointer to const [constVariablePointer] libopenshot-0.3.3/src/TrackedObjectBase.cpp:32:46: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable objectValue [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/TrackedObjectBase.cpp:22:21: warning: Member variable 'TrackedObjectBase::parentClip' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/TrackedObjectBase.cpp:25:21: warning: Member variable 'TrackedObjectBase::parentClip' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/TrackedObjectBase.h:37:15: style:inconclusive: Member variable 'TrackedObjectBase::id' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/TrackedObjectBase.cpp:26:32: note: Member variable 'TrackedObjectBase::id' is in the wrong place in the initializer list. libopenshot-0.3.3/src/TrackedObjectBase.h:37:15: note: Member variable 'TrackedObjectBase::id' is in the wrong place in the initializer list. libopenshot-0.3.3/src/TrackedObjectBase.cpp:28:33: performance:inconclusive: Technically the member function 'openshot::TrackedObjectBase::add_property_choice_json' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/TrackedObjectBase.h:86:15: note: Technically the member function 'openshot::TrackedObjectBase::add_property_choice_json' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/TrackedObjectBase.cpp:28:33: note: Technically the member function 'openshot::TrackedObjectBase::add_property_choice_json' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/ZmqLogger.cpp:116:0: error: failed to evaluate #if condition, undefined function-like macro invocation: ZMQ_MAKE_VERSION( ... ) [preprocessorErrorDirective] libopenshot-0.3.3/src/ZmqLogger.cpp:116:2: error: failed to evaluate #if condition, undefined function-like macro invocation: ZMQ_MAKE_VERSION( ... ) [preprocessorErrorDirective] libopenshot-0.3.3/src/TrackedObjectBase.h:58:15: performance: Function 'Id()' should return member 'id' by const reference. [returnByReference] libopenshot-0.3.3/src/TrackedObjectBase.h:60:23: performance: Function parameter '_id' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/TrackedObjectBase.h:79:42: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/TrackedObjectBase.cpp:25:51: performance: Function parameter '_id' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/TrackedObjectBase.cpp:29:15: performance: Function parameter 'name' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/audio_effects/Compressor.cpp:39:7: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/Compressor.cpp:127:22: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/Delay.cpp:33:7: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/Delay.cpp:116:22: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/Delay.h:54:3: style: Class 'Delay' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/audio_effects/Distortion.cpp:37:7: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/Distortion.cpp:154:2: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable coefficients [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/Distortion.cpp:168:22: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/Distortion.cpp:228:86: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable HARD_CLIPPING [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/Delay.cpp:77:10: style: The scope of the variable 'out' can be reduced. [variableScope] libopenshot-0.3.3/src/audio_effects/Delay.cpp:21:23: style:inconclusive: Function 'Delay' argument 1 names different: declaration 'new_delay_time' definition 'delay_time'. [funcArgNamesDifferent] libopenshot-0.3.3/src/audio_effects/Delay.h:54:18: note: Function 'Delay' argument 1 names different: declaration 'new_delay_time' definition 'delay_time'. libopenshot-0.3.3/src/audio_effects/Delay.cpp:21:23: note: Function 'Delay' argument 1 names different: declaration 'new_delay_time' definition 'delay_time'. libopenshot-0.3.3/src/audio_effects/Delay.cpp:124:39: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/audio_effects/Echo.cpp:35:7: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/Echo.cpp:119:22: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/Compressor.cpp:140:44: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/audio_effects/Delay.cpp:98:25: warning: Uninitialized variable: local_write_position [uninitvar] libopenshot-0.3.3/src/audio_effects/Delay.cpp:68:32: note: Assuming condition is false libopenshot-0.3.3/src/audio_effects/Delay.cpp:98:25: note: Uninitialized variable: local_write_position libopenshot-0.3.3/src/audio_effects/Delay.cpp:77:14: style: Variable 'out' is assigned a value that is never used. [unreadVariable] libopenshot-0.3.3/src/audio_effects/Echo.cpp:81:10: style: The scope of the variable 'out' can be reduced. [variableScope] libopenshot-0.3.3/src/audio_effects/Echo.cpp:129:38: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/audio_effects/Echo.cpp:101:24: warning: Uninitialized variable: local_write_position [uninitvar] libopenshot-0.3.3/src/audio_effects/Echo.cpp:72:32: note: Assuming condition is false libopenshot-0.3.3/src/audio_effects/Echo.cpp:101:24: note: Uninitialized variable: local_write_position libopenshot-0.3.3/src/audio_effects/Echo.cpp:81:14: style: Variable 'out' is assigned a value that is never used. [unreadVariable] libopenshot-0.3.3/src/audio_effects/Distortion.cpp:179:44: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/audio_effects/Distortion.cpp:52:11: style: Variable 'filter' can be declared as pointer to const [constVariablePointer] libopenshot-0.3.3/src/audio_effects/Expander.cpp:38:7: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/Expander.cpp:133:22: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/Expander.cpp:146:42: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/audio_effects/ParametricEQ.cpp:37:7: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/ParametricEQ.cpp:95:4: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable coefficients [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/ParametricEQ.cpp:185:22: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/ParametricEQ.cpp:245:77: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LOW_PASS [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/Robotization.cpp:40:7: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/Robotization.cpp:96:22: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/Robotization.cpp:150:69: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable FFT_SIZE_128 [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/audio_effects/ParametricEQ.h:46:12: style:inconclusive: Member variable 'ParametricEQ::q_factor' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/audio_effects/ParametricEQ.cpp:24:36: note: Member variable 'ParametricEQ::q_factor' is in the wrong place in the initializer list. libopenshot-0.3.3/src/audio_effects/ParametricEQ.h:46:12: note: Member variable 'ParametricEQ::q_factor' is in the wrong place in the initializer list. libopenshot-0.3.3/src/audio_effects/ParametricEQ.cpp:196:46: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/audio_effects/ParametricEQ.cpp:54:12: style: Variable 'filter' can be declared as pointer to const [constVariablePointer] libopenshot-0.3.3/src/audio_effects/STFT.h:24:9: warning: Member variable 'STFT::num_samples' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/audio_effects/STFT.h:24:9: warning: Member variable 'STFT::fft_size' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/audio_effects/STFT.h:24:9: warning: Member variable 'STFT::input_buffer_length' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/audio_effects/STFT.h:24:9: warning: Member variable 'STFT::output_buffer_length' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/audio_effects/STFT.h:24:9: warning: Member variable 'STFT::overlap' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/audio_effects/STFT.h:24:9: warning: Member variable 'STFT::hop_size' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/audio_effects/STFT.h:24:9: warning: Member variable 'STFT::window_type' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/audio_effects/STFT.h:24:9: warning: Member variable 'STFT::window_scale_factor' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/audio_effects/STFT.h:24:9: warning: Member variable 'STFT::input_buffer_write_position' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/audio_effects/STFT.h:24:9: warning: Member variable 'STFT::output_buffer_write_position' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/audio_effects/STFT.h:24:9: warning: Member variable 'STFT::output_buffer_read_position' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/audio_effects/STFT.h:24:9: warning: Member variable 'STFT::samples_since_last_FFT' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/audio_effects/STFT.h:24:9: warning: Member variable 'STFT::current_input_buffer_write_position' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/audio_effects/STFT.h:24:9: warning: Member variable 'STFT::current_output_buffer_write_position' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/audio_effects/STFT.h:24:9: warning: Member variable 'STFT::current_output_buffer_read_position' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/audio_effects/STFT.h:24:9: warning: Member variable 'STFT::current_samples_since_last_FFT' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/audio_effects/Robotization.h:80:4: style: Class 'RobotizationEffect' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/audio_effects/Robotization.cpp:106:46: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/audio_effects/Robotization.cpp:54:31: style: Variable 'num_input_channels' is assigned a value that is never used. [unreadVariable] libopenshot-0.3.3/src/audio_effects/Robotization.cpp:56:24: style: Variable 'num_samples' is assigned a value that is never used. [unreadVariable] libopenshot-0.3.3/src/audio_effects/Robotization.cpp:54:12: style: Variable 'num_input_channels' is assigned a value that is never used. [unreadVariable] libopenshot-0.3.3/src/audio_effects/Robotization.cpp:56:12: style: Variable 'num_samples' is assigned a value that is never used. [unreadVariable] libopenshot-0.3.3/src/effects/ObjectDetection.cpp:43:2: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable selectedObjectIndex [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/ObjectDetection.cpp:54:2: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable selectedObjectIndex [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/ObjectDetection.cpp:64:7: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/ObjectDetection.cpp:84:38: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Antialiasing [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/ObjectDetection.cpp:175:46: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable in [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/ObjectDetection.cpp:279:52: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable arrayValue [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/ObjectDetection.cpp:342:22: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/ObjectDetection.cpp:386:3: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable protobuf_data_path [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/ObjectDetection.cpp:452:26: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable selectedObjectIndex [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/Stabilizer.cpp:52:7: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/Stabilizer.cpp:78:18: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable CV_64F [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/Stabilizer.cpp:112:41: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable in [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/Stabilizer.cpp:168:22: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/Stabilizer.cpp:201:3: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable protobuf_data_path [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/Stabilizer.cpp:218:89: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable zoom [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/Tracker.cpp:42:2: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable trackedData [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/Tracker.cpp:59:2: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable trackedData [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/Tracker.cpp:75:7: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/Tracker.cpp:108:46: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Antialiasing [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/Tracker.cpp:143:52: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable arrayValue [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/Tracker.cpp:174:22: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable class_name [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/Tracker.cpp:221:12: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable num [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/effects/Tracker.cpp:272:35: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable trackedObjects [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/sort_filter/KalmanTracker.cpp:22:42: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable CV_32F [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/sort_filter/KalmanTracker.h:21:2: warning: Member variable 'KalmanTracker::confidence' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/sort_filter/KalmanTracker.h:21:2: warning: Member variable 'KalmanTracker::classId' is not initialized in the constructor. [uninitMemberVar] libopenshot-0.3.3/src/effects/ObjectDetection.cpp:101:36: style: C-style pointer casting [cstyleCast] libopenshot-0.3.3/src/effects/ObjectDetection.cpp:390:4: style: Statements following 'throw' will never be executed. [unreachableCode] libopenshot-0.3.3/src/effects/ObjectDetection.cpp:214:11: style: Local variable 'i' shadows outer variable [shadowVariable] libopenshot-0.3.3/src/effects/ObjectDetection.cpp:196:14: note: Shadowed declaration libopenshot-0.3.3/src/effects/ObjectDetection.cpp:214:11: note: Shadow variable libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:86:4: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:320:74: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable DBL_EPSILON [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:433:6: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable DBL_MAX [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/CVObjectDetection.cpp:39:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/CVObjectDetection.cpp:74:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/sort_filter/sort.cpp:23:11: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable DBL_EPSILON [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/sort_filter/sort.cpp:164:32: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable x [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/AudioWaveformer.cpp:24:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/AudioWaveformer.cpp:56:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/AudioWaveformer.cpp:88:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/AudioWaveformer.cpp:110:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/AudioWaveformer.cpp:138:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/CVStabilizer.cpp:33:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/CVStabilizer.cpp:77:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/CVTracker.cpp:36:19: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable InvalidJSON [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/CVTracker.cpp:71:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/CVTracker.cpp:116:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/CVTracker.cpp:33:16: style: Variable 'badTracker' can be declared as pointer to const [constVariablePointer] libopenshot-0.3.3/tests/CacheDisk.cpp:87:33: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/CacheDisk.cpp:142:33: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/CacheDisk.cpp:180:33: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/CacheMemory.cpp:277:33: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Caption.cpp:30:18: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cout [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/ChromaKey.cpp:50:22: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable transparent [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/sort_filter/sort.cpp:18:21: performance:inconclusive: Technically the member function 'SortTracker::GetIOU' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/sort_filter/sort.hpp:41:9: note: Technically the member function 'SortTracker::GetIOU' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/sort_filter/sort.cpp:18:21: note: Technically the member function 'SortTracker::GetIOU' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/sort_filter/sort.cpp:30:21: performance:inconclusive: Technically the member function 'SortTracker::GetCentroidsDistance' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/sort_filter/sort.hpp:42:9: note: Technically the member function 'SortTracker::GetCentroidsDistance' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/sort_filter/sort.cpp:30:21: note: Technically the member function 'SortTracker::GetCentroidsDistance' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/sort_filter/sort.hpp:36:2: style: Class 'SortTracker' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] libopenshot-0.3.3/src/sort_filter/sort.cpp:45:43: style:inconclusive: Function 'update' argument 1 names different: declaration 'detection' definition 'detections_cv'. [funcArgNamesDifferent] libopenshot-0.3.3/src/sort_filter/sort.hpp:40:36: note: Function 'update' argument 1 names different: declaration 'detection' definition 'detections_cv'. libopenshot-0.3.3/src/sort_filter/sort.cpp:45:43: note: Function 'update' argument 1 names different: declaration 'detection' definition 'detections_cv'. libopenshot-0.3.3/src/sort_filter/sort.cpp:45:117: performance: Function parameter 'confidences' should be passed by const reference. [passedByValue] libopenshot-0.3.3/src/sort_filter/sort.cpp:45:147: performance: Function parameter 'classIds' should be passed by const reference. [passedByValue] libopenshot-0.3.3/tests/Color.cpp:90:19: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable red [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Color.cpp:178:45: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable InvalidJSON [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Coordinate.cpp:70:45: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable InvalidJSON [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Clip.cpp:42:21: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable ANCHOR_CANVAS [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Clip.cpp:55:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Clip.cpp:75:29: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable ReaderClosed [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Clip.cpp:179:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Clip.cpp:226:50: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Clip.cpp:249:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Clip.cpp:355:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Clip.cpp:378:39: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Clip.cpp:444:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Crop.cpp:45:22: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable green [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Crop.cpp:70:22: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable green [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Crop.cpp:113:45: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable blue [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FFmpegReader.cpp:29:36: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable InvalidFile [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FFmpegReader.cpp:36:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FFmpegReader.cpp:47:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FFmpegReader.cpp:77:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FFmpegReader.cpp:123:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FFmpegReader.cpp:180:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FFmpegReader.cpp:196:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FFmpegReader.cpp:232:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FFmpegReader.cpp:265:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FFmpegReader.cpp:296:11: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FFmpegWriter.cpp:32:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FFmpegWriter.cpp:79:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FFmpegWriter.cpp:121:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FFmpegWriter.cpp:192:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Frame.cpp:66:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Frame.cpp:88:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Frame.cpp:110:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FFmpegReader.cpp:55:9: style: Variable 'samples' can be declared as pointer to const [constVariablePointer] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:193:26: performance:inconclusive: Technically the member function 'HungarianAlgorithm::buildassignmentvector' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/sort_filter/Hungarian.h:31:7: note: Technically the member function 'HungarianAlgorithm::buildassignmentvector' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:193:26: note: Technically the member function 'HungarianAlgorithm::buildassignmentvector' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:213:26: performance:inconclusive: Technically the member function 'HungarianAlgorithm::computeassignmentcost' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/sort_filter/Hungarian.h:32:7: note: Technically the member function 'HungarianAlgorithm::computeassignmentcost' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:213:26: note: Technically the member function 'HungarianAlgorithm::computeassignmentcost' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/tests/Point.cpp:34:56: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable BEZIER [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Point.cpp:44:57: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable BEZIER [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Point.cpp:65:57: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable BEZIER [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Point.cpp:72:45: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LINEAR [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Point.cpp:84:45: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable BEZIER [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Point.cpp:96:45: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable CONSTANT [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Point.cpp:110:25: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AUTO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Point.cpp:124:25: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable MANUAL [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Point.cpp:148:46: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable InvalidJSON [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Point.cpp:180:54: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LINEAR [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:87:3: warning: If memory allocation fails, then there is a possible null pointer dereference: distMatrix [nullPointerOutOfMemory] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:79:31: note: Assuming allocation function fails libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:79:15: note: Assignment 'distMatrix=(double*)std::malloc(nOfElements*sizeof(double))', assigned value is 0 libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:87:3: note: Null pointer dereference libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:282:7: warning: If memory allocation fails, then there is a possible null pointer dereference: coveredColumns [nullPointerOutOfMemory] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:91:33: note: Assuming allocation function fails libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:91:19: note: Assignment 'coveredColumns=(bool*)std::calloc(nOfColumns,sizeof(bool))', assigned value is 0 libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:176:73: note: Calling function 'step2b', 6th argument 'coveredColumns' value is 0 libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:282:7: note: Null pointer dereference libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:80:29: error: If memory allocation fails: pointer addition with NULL pointer. [nullPointerArithmeticOutOfMemory] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:79:31: note: Assuming allocation function fails libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:79:15: note: Assignment 'distMatrix=(double*)std::malloc(nOfElements*sizeof(double))', assigned value is 0 libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:80:29: note: Null pointer addition libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:79:15: style: C-style pointer casting [cstyleCast] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:91:19: style: C-style pointer casting [cstyleCast] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:92:16: style: C-style pointer casting [cstyleCast] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:93:15: style: C-style pointer casting [cstyleCast] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:94:16: style: C-style pointer casting [cstyleCast] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:95:18: style: C-style pointer casting [cstyleCast] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:67:56: style: The scope of the variable 'columnEnd' can be reduced. [variableScope] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:219:11: style: The scope of the variable 'col' can be reduced. [variableScope] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:242:25: style: The scope of the variable 'columnEnd' can be reduced. [variableScope] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:365:27: style: The scope of the variable 'primeRow' can be reduced. [variableScope] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:63:10: style:inconclusive: Function 'assignmentoptimal' argument 3 names different: declaration 'distMatrix' definition 'distMatrixIn'. [funcArgNamesDifferent] libopenshot-0.3.3/src/sort_filter/Hungarian.h:30:64: note: Function 'assignmentoptimal' argument 3 names different: declaration 'distMatrix' definition 'distMatrixIn'. libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:63:10: note: Function 'assignmentoptimal' argument 3 names different: declaration 'distMatrix' definition 'distMatrixIn'. libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:27:26: style: Parameter 'DistMatrix' can be declared as reference to const [constParameterReference] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:67:40: style: Variable 'distMatrixEnd' can be declared as pointer to const [constVariablePointer] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:67:56: style: Variable 'columnEnd' can be declared as pointer to const [constVariablePointer] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:195:8: style: Parameter 'starMatrix' can be declared as pointer to const [constParameterPointer] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:214:7: style: Parameter 'assignment' can be declared as pointer to const [constParameterPointer] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:216:10: style: Parameter 'distMatrix' can be declared as pointer to const [constParameterPointer] libopenshot-0.3.3/src/sort_filter/Hungarian.cpp:242:25: style: Variable 'columnEnd' can be declared as pointer to const [constVariablePointer] libopenshot-0.3.3/tests/FrameMapper.cpp:33:43: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable PULLDOWN_NONE [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FrameMapper.cpp:57:49: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable PULLDOWN_CLASSIC [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FrameMapper.cpp:70:43: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable PULLDOWN_CLASSIC [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FrameMapper.cpp:87:43: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable PULLDOWN_ADVANCED [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FrameMapper.cpp:107:43: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable PULLDOWN_NONE [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FrameMapper.cpp:124:43: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable PULLDOWN_CLASSIC [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FrameMapper.cpp:144:43: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable PULLDOWN_ADVANCED [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FrameMapper.cpp:164:43: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable PULLDOWN_NONE [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FrameMapper.cpp:179:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FrameMapper.cpp:259:39: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable PULLDOWN_NONE [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FrameMapper.cpp:396:40: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable PULLDOWN_NONE [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FrameMapper.cpp:540:37: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable PULLDOWN_NONE [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FrameMapper.cpp:631:24: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable PULLDOWN_CLASSIC [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/FrameMapper.cpp:642:39: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable PULLDOWN_NONE [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Profiles.cpp:38:17: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Profiles.cpp:73:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Profiles.cpp:87:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Profiles.cpp:102:17: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/QtImageReader.cpp:28:37: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable InvalidFile [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/QtImageReader.cpp:35:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/QtImageReader.cpp:46:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:39:34: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable OutOfBoundsPoint [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:48:35: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable OutOfBoundsPoint [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:62:26: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable X [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:74:26: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable X [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:84:48: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable BEZIER [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:103:48: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable BEZIER [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:126:48: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable BEZIER [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:149:48: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LINEAR [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:169:48: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable CONSTANT [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:222:57: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable X [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:250:78: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable X [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:267:28: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Y [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:289:48: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable BEZIER [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:331:48: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LINEAR [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:371:26: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Y [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:386:26: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Y [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:393:48: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable CONSTANT [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:406:38: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable CONSTANT [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:426:20: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LINEAR [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:468:18: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable CONSTANT [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/KeyFrame.cpp:476:20: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable CONSTANT [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Settings.cpp:36:32: style: Condition 's->HIGH_QUALITY_SCALING==true' is always true [knownConditionTrueFalse] libopenshot-0.3.3/tests/Settings.cpp:33:28: note: Assignment 's->HIGH_QUALITY_SCALING=true', assigned value is 1 libopenshot-0.3.3/tests/Settings.cpp:36:32: note: Condition 's->HIGH_QUALITY_SCALING==true' is always true libopenshot-0.3.3/tests/ReaderBase.cpp:34:14: performance:inconclusive: Either there is a missing 'override', or the member function 'TEST_CASE::TestReader::GetCache' can be static. [functionStatic] libopenshot-0.3.3/tests/ReaderBase.cpp:36:8: performance:inconclusive: Either there is a missing 'override', or the member function 'TEST_CASE::TestReader::Close' can be static. [functionStatic] libopenshot-0.3.3/tests/ReaderBase.cpp:37:8: performance:inconclusive: Either there is a missing 'override', or the member function 'TEST_CASE::TestReader::Open' can be static. [functionStatic] libopenshot-0.3.3/tests/ReaderBase.cpp:38:15: performance:inconclusive: Either there is a missing 'override', or the member function 'TEST_CASE::TestReader::Json' can be static. [functionStatic] libopenshot-0.3.3/tests/ReaderBase.cpp:39:8: performance:inconclusive: Either there is a missing 'override', or the member function 'TEST_CASE::TestReader::SetJson' can be static. [functionStatic] libopenshot-0.3.3/tests/ReaderBase.cpp:41:8: performance:inconclusive: Either there is a missing 'override', or the member function 'TEST_CASE::TestReader::SetJsonValue' can be static. [functionStatic] libopenshot-0.3.3/tests/ReaderBase.cpp:42:8: style:inconclusive: Either there is a missing 'override', or the member function 'TEST_CASE::TestReader::IsOpen' can be const. [functionConst] libopenshot-0.3.3/tests/ReaderBase.cpp:43:15: performance:inconclusive: Either there is a missing 'override', or the member function 'TEST_CASE::TestReader::Name' can be static. [functionStatic] libopenshot-0.3.3/tests/ReaderBase.cpp:50:18: style: Condition 't1.Name()=="TestReader"' is always true [knownConditionTrueFalse] libopenshot-0.3.3/tests/ReaderBase.cpp:50:15: note: Calling function 'Name' returns "TestReader" libopenshot-0.3.3/tests/ReaderBase.cpp:50:18: note: Condition 't1.Name()=="TestReader"' is always true libopenshot-0.3.3/tests/ReaderBase.cpp:54:20: style: Condition 't1.IsOpen()==true' is always true [knownConditionTrueFalse] libopenshot-0.3.3/tests/ReaderBase.cpp:54:17: note: Calling function 'IsOpen' returns 1 libopenshot-0.3.3/tests/ReaderBase.cpp:54:20: note: Condition 't1.IsOpen()==true' is always true libopenshot-0.3.3/tests/ReaderBase.cpp:56:22: style: Condition 't1.GetCache()==nullptr' is always true [knownConditionTrueFalse] libopenshot-0.3.3/tests/ReaderBase.cpp:56:19: note: Calling function 'GetCache' returns 0 libopenshot-0.3.3/tests/ReaderBase.cpp:56:22: note: Condition 't1.GetCache()==nullptr' is always true libopenshot-0.3.3/tests/ReaderBase.cpp:60:18: style: Condition 't1.Json()==""' is always true [knownConditionTrueFalse] libopenshot-0.3.3/tests/ReaderBase.cpp:60:15: note: Calling function 'Json' returns "" libopenshot-0.3.3/tests/ReaderBase.cpp:60:18: note: Condition 't1.Json()==""' is always true libopenshot-0.3.3/tests/Settings.cpp:45:7: style: Variable 'envvar' can be declared as pointer to const [constVariablePointer] libopenshot-0.3.3/tests/ReaderBase.cpp:39:28: performance: Function parameter 'value' should be passed by const reference. [passedByValue] libopenshot-0.3.3/tests/Timeline.cpp:34:39: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:50:38: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:95:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:115:39: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:139:39: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:149:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable TEST_MEDIA_PATH [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:236:50: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:305:50: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:333:50: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:445:50: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:488:50: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:548:50: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:572:50: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:606:50: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:666:67: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:702:67: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:750:50: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:804:50: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LAYOUT_STEREO [valueFlowBailoutIncompleteVar] libopenshot-0.3.3/tests/Timeline.cpp:125:22: style: Condition 't1.info.width==600' is always true [knownConditionTrueFalse] libopenshot-0.3.3/tests/Timeline.cpp:122:18: note: Assignment 't1.info.width=600', assigned value is 600 libopenshot-0.3.3/tests/Timeline.cpp:125:22: note: Condition 't1.info.width==600' is always true libopenshot-0.3.3/tests/Timeline.cpp:133:23: style: Condition 't1.info.height==400' is always true [knownConditionTrueFalse] libopenshot-0.3.3/tests/Timeline.cpp:129:19: note: Assignment 't1.info.height=400', assigned value is 400 libopenshot-0.3.3/tests/Timeline.cpp:133:23: note: Condition 't1.info.height==400' is always true libopenshot-0.3.3/tests/Timeline.cpp:851:34: style: C-style pointer casting [cstyleCast] libopenshot-0.3.3/tests/Timeline.cpp:478:8: style: Variable 'matched3' can be declared as pointer to const [constVariablePointer] libopenshot-0.3.3/tests/Timeline.cpp:682:28: style: Variable 'f' is assigned a value that is never used. [unreadVariable] libopenshot-0.3.3/tests/Timeline.cpp:726:28: style: Variable 'f' is assigned a value that is never used. [unreadVariable] diff: 2.18.0 libopenshot-0.3.3/src/AudioReaderSource.cpp:96:25: performance:inconclusive: Either there is a missing 'override', or the member function 'openshot::AudioReaderSource::prepareToPlay' can be static. [functionStatic] libopenshot-0.3.3/src/AudioReaderSource.h:58:8: note: Either there is a missing 'override', or the member function 'openshot::AudioReaderSource::prepareToPlay' can be static. libopenshot-0.3.3/src/AudioReaderSource.cpp:96:25: note: Either there is a missing 'override', or the member function 'openshot::AudioReaderSource::prepareToPlay' can be static. 2.18.0 libopenshot-0.3.3/src/AudioReaderSource.cpp:99:25: performance:inconclusive: Either there is a missing 'override', or the member function 'openshot::AudioReaderSource::releaseResources' can be static. [functionStatic] libopenshot-0.3.3/src/AudioReaderSource.h:61:8: note: Either there is a missing 'override', or the member function 'openshot::AudioReaderSource::releaseResources' can be static. libopenshot-0.3.3/src/AudioReaderSource.cpp:99:25: note: Either there is a missing 'override', or the member function 'openshot::AudioReaderSource::releaseResources' can be static. 2.18.0 libopenshot-0.3.3/src/AudioReaderSource.h:34:10: style:inconclusive: Member variable 'AudioReaderSource::stream_position' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/AudioReaderSource.cpp:24:37: note: Member variable 'AudioReaderSource::stream_position' is in the wrong place in the initializer list. libopenshot-0.3.3/src/AudioReaderSource.h:34:10: note: Member variable 'AudioReaderSource::stream_position' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/AudioReaderSource.h:35:7: style:inconclusive: Member variable 'AudioReaderSource::frame_position' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/AudioReaderSource.cpp:23:26: note: Member variable 'AudioReaderSource::frame_position' is in the wrong place in the initializer list. libopenshot-0.3.3/src/AudioReaderSource.h:35:7: note: Member variable 'AudioReaderSource::frame_position' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/AudioReaderSource.h:36:7: style:inconclusive: Member variable 'AudioReaderSource::speed' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/AudioReaderSource.cpp:24:27: note: Member variable 'AudioReaderSource::speed' is in the wrong place in the initializer list. libopenshot-0.3.3/src/AudioReaderSource.h:36:7: note: Member variable 'AudioReaderSource::speed' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/AudioReaderSource.h:39:26: style:inconclusive: Member variable 'AudioReaderSource::frame' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/AudioReaderSource.cpp:23:83: note: Member variable 'AudioReaderSource::frame' is in the wrong place in the initializer list. libopenshot-0.3.3/src/AudioReaderSource.h:39:26: note: Member variable 'AudioReaderSource::frame' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/AudioReaderSource.h:78:8: performance:inconclusive: Either there is a missing 'override', or the member function 'openshot::AudioReaderSource::setLooping' can be static. [functionStatic] 2.18.0 libopenshot-0.3.3/src/AudioResampler.cpp:38:2: warning: Class 'AudioResampler' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] head libopenshot-0.3.3/src/AudioResampler.cpp:38:2: warning: Class 'AudioResampler' does not have a copy constructor which is recommended since it has dynamic memory/resource management. [noCopyConstructor] 2.18.0 libopenshot-0.3.3/src/AudioResampler.cpp:38:2: warning: Class 'AudioResampler' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] head libopenshot-0.3.3/src/AudioResampler.cpp:38:2: warning: Class 'AudioResampler' does not have a operator= which is recommended since it has dynamic memory/resource management. [noOperatorEq] 2.18.0 libopenshot-0.3.3/src/AudioWaveformer.cpp:21:46: style:inconclusive: Function 'AudioWaveformer' argument 1 names different: declaration 'reader' definition 'new_reader'. [funcArgNamesDifferent] libopenshot-0.3.3/src/AudioWaveformer.h:85:37: note: Function 'AudioWaveformer' argument 1 names different: declaration 'reader' definition 'new_reader'. libopenshot-0.3.3/src/AudioWaveformer.cpp:21:46: note: Function 'AudioWaveformer' argument 1 names different: declaration 'reader' definition 'new_reader'. 2.18.0 libopenshot-0.3.3/src/AudioWaveformer.cpp:92:32: style: Variable 'samples' can be declared as pointer to const [constVariablePointer] 2.18.0 libopenshot-0.3.3/src/AudioWaveformer.h:85:9: style: Class 'AudioWaveformer' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:121:105: style:inconclusive: Function 'postprocess' argument 3 names different: declaration 'frame_number' definition 'frameId'. [funcArgNamesDifferent] libopenshot-0.3.3/src/CVObjectDetection.h:98:93: note: Function 'postprocess' argument 3 names different: declaration 'frame_number' definition 'frameId'. libopenshot-0.3.3/src/CVObjectDetection.cpp:121:105: note: Function 'postprocess' argument 3 names different: declaration 'frame_number' definition 'frameId'. 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:121:92: style:inconclusive: Function 'postprocess' argument 2 names different: declaration 'out' definition 'outs'. [funcArgNamesDifferent] libopenshot-0.3.3/src/CVObjectDetection.h:98:81: note: Function 'postprocess' argument 2 names different: declaration 'out' definition 'outs'. libopenshot-0.3.3/src/CVObjectDetection.cpp:121:92: note: Function 'postprocess' argument 2 names different: declaration 'out' definition 'outs'. 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:133:23: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:133:39: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable data [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:165:19: style: Consider using std::copy algorithm instead of a raw loop. [useStlAlgorithm] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:249:25: performance:inconclusive: Technically the member function 'openshot::CVObjectDetection::iou' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/CVObjectDetection.h:95:14: note: Technically the member function 'openshot::CVObjectDetection::iou' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/CVObjectDetection.cpp:249:25: note: Technically the member function 'openshot::CVObjectDetection::iou' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:28:20: warning: Member variable 'CVObjectDetection::end' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:28:20: warning: Member variable 'CVObjectDetection::progress' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:28:20: warning: Member variable 'CVObjectDetection::start' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:28:50: performance: Function parameter 'processInfoJson' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:314:30: style: Unused variable: pbFrameData [unusedVariable] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:323:54: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable out [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:338:25: performance:inconclusive: Technically the member function 'openshot::CVObjectDetection::AddFrameDataToProto' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/CVObjectDetection.h:118:14: note: Technically the member function 'openshot::CVObjectDetection::AddFrameDataToProto' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/CVObjectDetection.cpp:338:25: note: Technically the member function 'openshot::CVObjectDetection::AddFrameDataToProto' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:359:51: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:372:14: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cout [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:37:43: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable DNN_BACKEND_CUDA [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:429:48: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable in [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:461:17: style: Local variable 'i' shadows outer variable [shadowVariable] libopenshot-0.3.3/src/CVObjectDetection.cpp:445:17: note: Shadowed declaration libopenshot-0.3.3/src/CVObjectDetection.cpp:461:17: note: Shadow variable 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:46:73: style:inconclusive: Function 'detectObjectsClip' argument 2 names different: declaration 'start' definition '_start'. [funcArgNamesDifferent] libopenshot-0.3.3/src/CVObjectDetection.h:110:62: note: Function 'detectObjectsClip' argument 2 names different: declaration 'start' definition '_start'. libopenshot-0.3.3/src/CVObjectDetection.cpp:46:73: note: Function 'detectObjectsClip' argument 2 names different: declaration 'start' definition '_start'. 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:46:88: style:inconclusive: Function 'detectObjectsClip' argument 3 names different: declaration 'end' definition '_end'. [funcArgNamesDifferent] libopenshot-0.3.3/src/CVObjectDetection.h:110:78: note: Function 'detectObjectsClip' argument 3 names different: declaration 'end' definition '_end'. libopenshot-0.3.3/src/CVObjectDetection.cpp:46:88: note: Function 'detectObjectsClip' argument 3 names different: declaration 'end' definition '_end'. 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:471:17: style: Variable 'objectId' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:471:26: style: Variable 'objectId' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.cpp:97:68: style:inconclusive: Function 'DetectObjects' argument 2 names different: declaration 'frame_number' definition 'frameId'. [funcArgNamesDifferent] libopenshot-0.3.3/src/CVObjectDetection.h:93:57: note: Function 'DetectObjects' argument 2 names different: declaration 'frame_number' definition 'frameId'. libopenshot-0.3.3/src/CVObjectDetection.cpp:97:68: note: Function 'DetectObjects' argument 2 names different: declaration 'frame_number' definition 'frameId'. 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.h:38:9: warning:inconclusive: Member variable 'CVDetectionData::frameId' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.h:40:30: performance: Function parameter '_classIds' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.h:41:32: performance: Function parameter '_confidences' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.h:42:43: performance: Function parameter '_boxes' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.h:44:30: performance: Function parameter '_objectIds' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.h:46:13: performance: Variable 'classIds' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.h:47:13: performance: Variable 'confidences' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.h:48:13: performance: Variable 'boxes' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.h:50:13: performance: Variable 'objectIds' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] 2.18.0 libopenshot-0.3.3/src/CVObjectDetection.h:75:21: style:inconclusive: Member variable 'CVObjectDetection::processingDevice' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/CVObjectDetection.cpp:29:48: note: Member variable 'CVObjectDetection::processingDevice' is in the wrong place in the initializer list. libopenshot-0.3.3/src/CVObjectDetection.h:75:21: note: Member variable 'CVObjectDetection::processingDevice' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:146:17: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable width [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:187:45: style:inconclusive: Technically the member function 'CVStabilization::ComputeFramesTrajectory' can be const. [functionConst] libopenshot-0.3.3/src/CVStabilization.h:99:32: note: Technically the member function 'CVStabilization::ComputeFramesTrajectory' can be const. libopenshot-0.3.3/src/CVStabilization.cpp:187:45: note: Technically the member function 'CVStabilization::ComputeFramesTrajectory' can be const. 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:208:49: style:inconclusive: Technically the member function 'CVStabilization::SmoothTrajectory' can be const. [functionConst] libopenshot-0.3.3/src/CVStabilization.h:100:36: note: Technically the member function 'CVStabilization::SmoothTrajectory' can be const. libopenshot-0.3.3/src/CVStabilization.cpp:208:49: note: Technically the member function 'CVStabilization::SmoothTrajectory' can be const. 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:208:95: style: Parameter 'trajectory' can be declared as reference to const [constParameterReference] 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:239:101: style: Parameter 'smoothed_trajectory' can be declared as reference to const [constParameterReference] 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:239:50: style:inconclusive: Technically the member function 'CVStabilization::GenNewCamPosition' can be const. [functionConst] libopenshot-0.3.3/src/CVStabilization.h:103:37: note: Technically the member function 'CVStabilization::GenNewCamPosition' can be const. libopenshot-0.3.3/src/CVStabilization.cpp:239:50: note: Technically the member function 'CVStabilization::GenNewCamPosition' can be const. 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:285:50: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable out [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:298:23: performance:inconclusive: Technically the member function 'CVStabilization::AddFrameDataToProto' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/CVStabilization.h:120:10: note: Technically the member function 'CVStabilization::AddFrameDataToProto' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/CVStabilization.cpp:298:23: note: Technically the member function 'CVStabilization::AddFrameDataToProto' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:29:18: warning: Member variable 'CVStabilization::avr_da' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:29:18: warning: Member variable 'CVStabilization::avr_dx' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:29:18: warning: Member variable 'CVStabilization::avr_dy' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:29:18: warning: Member variable 'CVStabilization::max_da' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:29:18: warning: Member variable 'CVStabilization::max_dx' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:29:18: warning: Member variable 'CVStabilization::max_dy' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:29:18: warning: Member variable 'CVStabilization::progress' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:29:46: performance: Function parameter 'processInfoJson' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:339:49: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:379:49: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable in [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/CVStabilization.cpp:72:27: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable width [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/CVStabilization.h:41:5: warning:inconclusive: Member variable 'TransformParam::da' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVStabilization.h:41:5: warning:inconclusive: Member variable 'TransformParam::dx' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVStabilization.h:41:5: warning:inconclusive: Member variable 'TransformParam::dy' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVStabilization.h:56:5: warning:inconclusive: Member variable 'CamTrajectory::a' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVStabilization.h:56:5: warning:inconclusive: Member variable 'CamTrajectory::x' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVStabilization.h:56:5: warning:inconclusive: Member variable 'CamTrajectory::y' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVTracker.cpp:118:26: style: Variable 'fd' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/CVTracker.cpp:145:29: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable width [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/CVTracker.cpp:166:33: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable width [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/CVTracker.cpp:187:55: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable x2 [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/CVTracker.cpp:214:28: style: Unused variable: pbFrameData [unusedVariable] 2.18.0 libopenshot-0.3.3/src/CVTracker.cpp:223:54: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable out [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/CVTracker.cpp:238:17: performance:inconclusive: Technically the member function 'openshot::CVTracker::AddFrameDataToProto' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/CVTracker.h:126:9: note: Technically the member function 'openshot::CVTracker::AddFrameDataToProto' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/CVTracker.cpp:238:17: note: Technically the member function 'openshot::CVTracker::AddFrameDataToProto' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/CVTracker.cpp:267:43: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CVTracker.cpp:29:12: warning: Member variable 'CVTracker::progress' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/CVTracker.cpp:29:34: performance: Function parameter 'processInfoJson' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CVTracker.cpp:334:53: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable in [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/CVTracker.cpp:37:67: performance: Function parameter 'trackerType' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CVTracker.h:57:3: style: Struct 'FrameData' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/CacheBase.cpp:26:2: warning: Class 'CacheBase' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] head libopenshot-0.3.3/src/CacheBase.cpp:26:2: warning: Class 'CacheBase' does not have a copy constructor which is recommended since it has dynamic memory/resource management. [noCopyConstructor] 2.18.0 libopenshot-0.3.3/src/CacheBase.cpp:26:2: warning: Class 'CacheBase' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] head libopenshot-0.3.3/src/CacheBase.cpp:26:2: warning: Class 'CacheBase' does not have a operator= which is recommended since it has dynamic memory/resource management. [noOperatorEq] 2.18.0 libopenshot-0.3.3/src/CacheDisk.cpp:121:35: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable WriteOnly [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/CacheDisk.cpp:132:13: style: Variable 'samples' can be declared as pointer to const [constVariablePointer] 2.18.0 libopenshot-0.3.3/src/CacheDisk.cpp:172:68: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Format_RGBA8888_Premultiplied [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/CacheDisk.cpp:26:34: performance: Function parameter 'cache_path' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CacheDisk.cpp:26:58: performance: Function parameter 'format' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CacheDisk.cpp:276:15: style: Consider using std::accumulate algorithm instead of a raw loop. [useStlAlgorithm] 2.18.0 libopenshot-0.3.3/src/CacheDisk.cpp:303:4: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] 2.18.0 libopenshot-0.3.3/src/CacheDisk.cpp:329:4: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] 2.18.0 libopenshot-0.3.3/src/CacheDisk.cpp:32:2: performance: Variable 'image_format' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] 2.18.0 libopenshot-0.3.3/src/CacheDisk.cpp:350:4: style: Consider using std::find_if algorithm instead of a raw loop. [useStlAlgorithm] 2.18.0 libopenshot-0.3.3/src/CacheDisk.cpp:42:34: performance: Function parameter 'cache_path' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CacheDisk.cpp:42:58: performance: Function parameter 'format' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CacheDisk.cpp:450:43: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CacheDisk.cpp:48:2: performance: Variable 'image_format' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] 2.18.0 libopenshot-0.3.3/src/CacheDisk.cpp:57:38: performance: Function parameter 'cache_path' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CacheDisk.h:103:8: style: The function 'Remove' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:94:16: note: Virtual function in base class libopenshot-0.3.3/src/CacheDisk.h:103:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheDisk.h:106:15: style: The function 'Json' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:112:23: note: Virtual function in base class libopenshot-0.3.3/src/CacheDisk.h:106:15: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheDisk.h:107:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CacheDisk.h:107:8: style: The function 'SetJson' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:113:16: note: Virtual function in base class libopenshot-0.3.3/src/CacheDisk.h:107:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheDisk.h:108:15: style: The function 'JsonValue' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:114:23: note: Virtual function in base class libopenshot-0.3.3/src/CacheDisk.h:108:15: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheDisk.h:109:8: style: The function 'SetJsonValue' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:115:16: note: Virtual function in base class libopenshot-0.3.3/src/CacheDisk.h:109:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheDisk.h:63:4: style: The destructor '~CacheDisk' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:116:12: note: Virtual destructor in base class libopenshot-0.3.3/src/CacheDisk.h:63:4: note: Destructor in derived class 2.18.0 libopenshot-0.3.3/src/CacheDisk.h:67:8: style: The function 'Add' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:62:16: note: Virtual function in base class libopenshot-0.3.3/src/CacheDisk.h:67:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheDisk.h:70:8: style: The function 'Clear' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:65:16: note: Virtual function in base class libopenshot-0.3.3/src/CacheDisk.h:70:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheDisk.h:74:8: style: The function 'Contains' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:69:16: note: Virtual function in base class libopenshot-0.3.3/src/CacheDisk.h:74:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheDisk.h:77:11: style: The function 'Count' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:72:19: note: Virtual function in base class libopenshot-0.3.3/src/CacheDisk.h:77:11: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheDisk.h:81:36: style: The function 'GetFrame' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:76:44: note: Virtual function in base class libopenshot-0.3.3/src/CacheDisk.h:81:36: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheDisk.h:84:49: style: The function 'GetFrames' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:79:57: note: Virtual function in base class libopenshot-0.3.3/src/CacheDisk.h:84:49: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheDisk.h:87:11: style: The function 'GetBytes' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:82:19: note: Virtual function in base class libopenshot-0.3.3/src/CacheDisk.h:87:11: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheDisk.h:90:36: style: The function 'GetSmallestFrame' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:85:44: note: Virtual function in base class libopenshot-0.3.3/src/CacheDisk.h:90:36: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheDisk.h:98:8: style: The function 'Remove' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:89:16: note: Virtual function in base class libopenshot-0.3.3/src/CacheDisk.h:98:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheMemory.cpp:147:15: style: Consider using std::accumulate algorithm instead of a raw loop. [useStlAlgorithm] 2.18.0 libopenshot-0.3.3/src/CacheMemory.cpp:174:4: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] 2.18.0 libopenshot-0.3.3/src/CacheMemory.cpp:187:4: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] 2.18.0 libopenshot-0.3.3/src/CacheMemory.cpp:208:4: style: Consider using std::find_if algorithm instead of a raw loop. [useStlAlgorithm] 2.18.0 libopenshot-0.3.3/src/CacheMemory.cpp:295:45: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CacheMemory.h:43:3: style: Class 'CacheMemory' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/CacheMemory.h:46:12: style: The destructor '~CacheMemory' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:116:12: note: Virtual destructor in base class libopenshot-0.3.3/src/CacheMemory.h:46:12: note: Destructor in derived class 2.18.0 libopenshot-0.3.3/src/CacheMemory.h:50:8: style: The function 'Add' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:62:16: note: Virtual function in base class libopenshot-0.3.3/src/CacheMemory.h:50:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheMemory.h:53:8: style: The function 'Clear' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:65:16: note: Virtual function in base class libopenshot-0.3.3/src/CacheMemory.h:53:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheMemory.h:57:8: style: The function 'Contains' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:69:16: note: Virtual function in base class libopenshot-0.3.3/src/CacheMemory.h:57:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheMemory.h:60:11: style: The function 'Count' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:72:19: note: Virtual function in base class libopenshot-0.3.3/src/CacheMemory.h:60:11: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheMemory.h:64:36: style: The function 'GetFrame' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:76:44: note: Virtual function in base class libopenshot-0.3.3/src/CacheMemory.h:64:36: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheMemory.h:67:49: style: The function 'GetFrames' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:79:57: note: Virtual function in base class libopenshot-0.3.3/src/CacheMemory.h:67:49: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheMemory.h:70:11: style: The function 'GetBytes' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:82:19: note: Virtual function in base class libopenshot-0.3.3/src/CacheMemory.h:70:11: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheMemory.h:73:36: style: The function 'GetSmallestFrame' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:85:44: note: Virtual function in base class libopenshot-0.3.3/src/CacheMemory.h:73:36: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheMemory.h:81:8: style: The function 'Remove' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:89:16: note: Virtual function in base class libopenshot-0.3.3/src/CacheMemory.h:81:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheMemory.h:86:8: style: The function 'Remove' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:94:16: note: Virtual function in base class libopenshot-0.3.3/src/CacheMemory.h:86:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheMemory.h:89:15: style: The function 'Json' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:112:23: note: Virtual function in base class libopenshot-0.3.3/src/CacheMemory.h:89:15: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheMemory.h:90:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/CacheMemory.h:90:8: style: The function 'SetJson' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:113:16: note: Virtual function in base class libopenshot-0.3.3/src/CacheMemory.h:90:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheMemory.h:91:15: style: The function 'JsonValue' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:114:23: note: Virtual function in base class libopenshot-0.3.3/src/CacheMemory.h:91:15: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/CacheMemory.h:92:8: style: The function 'SetJsonValue' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/CacheBase.h:115:16: note: Virtual function in base class libopenshot-0.3.3/src/CacheMemory.h:92:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/ChunkReader.cpp:119:28: style:inconclusive: Technically the member function 'openshot::ChunkReader::find_chunk_frame' can be const. [functionConst] libopenshot-0.3.3/src/ChunkReader.h:93:17: note: Technically the member function 'openshot::ChunkReader::find_chunk_frame' can be const. libopenshot-0.3.3/src/ChunkReader.cpp:119:28: note: Technically the member function 'openshot::ChunkReader::find_chunk_frame' can be const. 2.18.0 libopenshot-0.3.3/src/ChunkReader.cpp:152:6: style: The statement 'if (is_open) is_open=false' is logically equivalent to 'is_open=false'. [duplicateConditionalAssign] libopenshot-0.3.3/src/ChunkReader.cpp:155:11: note: Assignment 'is_open=false' libopenshot-0.3.3/src/ChunkReader.cpp:152:6: note: Condition 'is_open' is redundant 2.18.0 libopenshot-0.3.3/src/ChunkReader.cpp:160:75: performance: Function parameter 'folder' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ChunkReader.cpp:160:95: performance: Function parameter 'extension' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ChunkReader.cpp:267:45: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ChunkReader.cpp:41:49: performance: Function parameter 'path' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ChunkReader.cpp:56:14: style: The scope of the variable 'line' can be reduced. [variableScope] 2.18.0 libopenshot-0.3.3/src/ChunkReader.h:110:8: style: Virtual function 'Close' is called from constructor 'ChunkReader(std::string path,ChunkVersion chunk_version)' at line 37. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/ChunkReader.cpp:37:2: note: Calling Close libopenshot-0.3.3/src/ChunkReader.h:110:8: note: Close is a virtual function 2.18.0 libopenshot-0.3.3/src/ChunkReader.h:114:11: style:inconclusive: Technically the member function 'openshot::ChunkReader::GetChunkSize' can be const. [functionConst] 2.18.0 libopenshot-0.3.3/src/ChunkReader.h:136:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ChunkReader.h:141:8: style: Virtual function 'Open' is called from constructor 'ChunkReader(std::string path,ChunkVersion chunk_version)' at line 36. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/ChunkReader.cpp:36:2: note: Calling Open libopenshot-0.3.3/src/ChunkReader.h:141:8: note: Open is a virtual function 2.18.0 libopenshot-0.3.3/src/ChunkReader.h:82:8: style:inconclusive: Member variable 'ChunkReader::is_open' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/ChunkReader.cpp:24:37: note: Member variable 'ChunkReader::is_open' is in the wrong place in the initializer list. libopenshot-0.3.3/src/ChunkReader.h:82:8: note: Member variable 'ChunkReader::is_open' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/ChunkReader.h:84:25: style:inconclusive: Member variable 'ChunkReader::local_reader' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/ChunkReader.cpp:24:77: note: Member variable 'ChunkReader::local_reader' is in the wrong place in the initializer list. libopenshot-0.3.3/src/ChunkReader.h:84:25: note: Member variable 'ChunkReader::local_reader' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/ChunkWriter.cpp:19:14: warning: Member variable 'ChunkWriter::writer_final' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/ChunkWriter.cpp:19:14: warning: Member variable 'ChunkWriter::writer_preview' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/ChunkWriter.cpp:19:14: warning: Member variable 'ChunkWriter::writer_thumb' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/ChunkWriter.cpp:244:14: style: Variable 'chunk_count' is reassigned a value before the old one has been used. [redundantAssignment] libopenshot-0.3.3/src/ChunkWriter.cpp:234:14: note: chunk_count is assigned libopenshot-0.3.3/src/ChunkWriter.cpp:244:14: note: chunk_count is overwritten 2.18.0 libopenshot-0.3.3/src/ChunkWriter.cpp:260:41: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/ChunkWriter.cpp:265:45: performance: Function parameter 'path' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ChunkWriter.cpp:274:19: performance:inconclusive: Technically the member function 'openshot::ChunkWriter::is_chunk_valid' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/ChunkWriter.h:88:8: note: Technically the member function 'openshot::ChunkWriter::is_chunk_valid' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/ChunkWriter.cpp:274:19: note: Technically the member function 'openshot::ChunkWriter::is_chunk_valid' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/ChunkWriter.cpp:274:19: style: Unused private function: 'ChunkWriter::is_chunk_valid' [unusedPrivateFunction] libopenshot-0.3.3/src/ChunkWriter.h:88:8: note: Unused private function: 'ChunkWriter::is_chunk_valid' libopenshot-0.3.3/src/ChunkWriter.cpp:274:19: note: Unused private function: 'ChunkWriter::is_chunk_valid' 2.18.0 libopenshot-0.3.3/src/ChunkWriter.cpp:41:75: performance: Function parameter 'folder' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ChunkWriter.cpp:41:95: performance: Function parameter 'extension' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ChunkWriter.h:104:11: style:inconclusive: Technically the member function 'openshot::ChunkWriter::GetChunkSize' can be const. [functionConst] 2.18.0 libopenshot-0.3.3/src/ChunkWriter.h:107:8: style: The function 'IsOpen' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/WriterBase.h:83:16: note: Virtual function in base class libopenshot-0.3.3/src/ChunkWriter.h:107:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/ChunkWriter.h:110:8: style: The function 'Open' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/WriterBase.h:101:16: note: Virtual function in base class libopenshot-0.3.3/src/ChunkWriter.h:110:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/ChunkWriter.h:118:8: style: The function 'WriteFrame' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/WriterBase.h:86:16: note: Virtual function in base class libopenshot-0.3.3/src/ChunkWriter.h:118:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/ChunkWriter.h:129:8: style: The function 'WriteFrame' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/WriterBase.h:89:16: note: Virtual function in base class libopenshot-0.3.3/src/ChunkWriter.h:129:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/ChunkWriter.h:65:15: style:inconclusive: Member variable 'ChunkWriter::path' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/ChunkWriter.cpp:20:25: note: Member variable 'ChunkWriter::path' is in the wrong place in the initializer list. libopenshot-0.3.3/src/ChunkWriter.h:65:15: note: Member variable 'ChunkWriter::path' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/ChunkWriter.h:66:11: style:inconclusive: Member variable 'ChunkWriter::chunk_count' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/ChunkWriter.cpp:20:55: note: Member variable 'ChunkWriter::chunk_count' is in the wrong place in the initializer list. libopenshot-0.3.3/src/ChunkWriter.h:66:11: note: Member variable 'ChunkWriter::chunk_count' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/ChunkWriter.h:69:8: style:inconclusive: Member variable 'ChunkWriter::is_open' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/ChunkWriter.cpp:21:112: note: Member variable 'ChunkWriter::is_open' is in the wrong place in the initializer list. libopenshot-0.3.3/src/ChunkWriter.h:69:8: note: Member variable 'ChunkWriter::is_open' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/ChunkWriter.h:76:11: style:inconclusive: Member variable 'ChunkWriter::last_frame_needed' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/ChunkWriter.cpp:21:86: note: Member variable 'ChunkWriter::last_frame_needed' is in the wrong place in the initializer list. libopenshot-0.3.3/src/ChunkWriter.h:76:11: note: Member variable 'ChunkWriter::last_frame_needed' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/Clip.cpp:1164:37: style: Parameter 'effect' can be declared as pointer to const [constParameterPointer] 2.18.0 libopenshot-0.3.3/src/Clip.cpp:1173:12: performance:inconclusive: Technically the member function 'openshot::Clip::apply_background' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/Clip.h:131:8: note: Technically the member function 'openshot::Clip::apply_background' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/Clip.cpp:1173:12: note: Technically the member function 'openshot::Clip::apply_background' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/Clip.cpp:1177:35: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Antialiasing [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Clip.cpp:1210:12: performance:inconclusive: Technically the member function 'openshot::Clip::isNear' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/Clip.h:158:8: note: Technically the member function 'openshot::Clip::isNear' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/Clip.cpp:1210:12: note: Technically the member function 'openshot::Clip::isNear' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/Clip.cpp:1227:28: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Format_RGBA8888_Premultiplied [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Clip.cpp:1301:6: style: Local variable 'alpha' shadows outer variable [shadowVariable] libopenshot-0.3.3/src/Clip.h:310:22: note: Shadowed declaration libopenshot-0.3.3/src/Clip.cpp:1301:6: note: Shadow variable 2.18.0 libopenshot-0.3.3/src/Clip.cpp:1309:13: performance:inconclusive: Technically the member function 'openshot::Clip::scale_size' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/Clip.h:164:9: note: Technically the member function 'openshot::Clip::scale_size' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/Clip.cpp:1309:13: note: Technically the member function 'openshot::Clip::scale_size' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/Clip.cpp:1313:64: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable KeepAspectRatio [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Clip.cpp:1391:28: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/Clip.cpp:1529:8: style: Variable 'parent' can be declared as pointer to const [constVariablePointer] 2.18.0 libopenshot-0.3.3/src/Clip.cpp:161:24: performance: Function parameter 'path' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Clip.cpp:172:105: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable npos [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Clip.cpp:204:5: warning: Class 'Clip' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] 2.18.0 libopenshot-0.3.3/src/Clip.cpp:204:5: warning: Class 'Clip' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] 2.18.0 libopenshot-0.3.3/src/Clip.cpp:241:39: performance: Function parameter 'object_id' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Clip.cpp:475:27: style: Consider using std::find_if algorithm instead of a raw loop. [useStlAlgorithm] 2.18.0 libopenshot-0.3.3/src/Clip.cpp:501:19: performance:inconclusive: Technically the member function 'openshot::Clip::get_file_extension' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/Clip.h:149:15: note: Technically the member function 'openshot::Clip::get_file_extension' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/Clip.cpp:501:19: note: Technically the member function 'openshot::Clip::get_file_extension' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/Clip.cpp:501:50: performance: Function parameter 'path' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Clip.cpp:653:15: performance:inconclusive: Technically the member function 'openshot::Clip::adjust_frame_number_minimum' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/Clip.h:128:11: note: Technically the member function 'openshot::Clip::adjust_frame_number_minimum' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/Clip.cpp:653:15: note: Technically the member function 'openshot::Clip::adjust_frame_number_minimum' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/Clip.cpp:692:38: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable transparent [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Clip.cpp:889:38: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable arrayValue [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Clip.cpp:907:38: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Clip.h:109:8: style:inconclusive: Member variable 'Clip::is_open' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/Clip.cpp:135:71: note: Member variable 'Clip::is_open' is in the wrong place in the initializer list. libopenshot-0.3.3/src/Clip.h:109:8: note: Member variable 'Clip::is_open' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/Clip.h:109:8: style:inconclusive: Member variable 'Clip::is_open' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/Clip.cpp:142:99: note: Member variable 'Clip::is_open' is in the wrong place in the initializer list. libopenshot-0.3.3/src/Clip.h:109:8: note: Member variable 'Clip::is_open' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/Clip.h:109:8: style:inconclusive: Member variable 'Clip::is_open' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/Clip.cpp:161:87: note: Member variable 'Clip::is_open' is in the wrong place in the initializer list. libopenshot-0.3.3/src/Clip.h:109:8: note: Member variable 'Clip::is_open' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/Clip.h:184:3: style: Class 'Clip' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/Clip.h:188:3: style: Class 'Clip' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/Clip.h:191:12: style: The destructor '~Clip' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/ClipBase.h:111:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Clip.h:191:12: note: Destructor in derived class 2.18.0 libopenshot-0.3.3/src/Clip.h:200:15: performance: Function 'GetAttachedId()' should return member 'parentObjectId' by const reference. [returnByReference] 2.18.0 libopenshot-0.3.3/src/Clip.h:202:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Clip.h:224:8: style: Virtual function 'Close' is called from constructor 'Clip(openshot::ReaderBase*new_reader)' at line 149. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/Clip.cpp:149:2: note: Calling Close libopenshot-0.3.3/src/Clip.h:224:8: note: Close is a virtual function 2.18.0 libopenshot-0.3.3/src/Clip.h:224:8: style: Virtual function 'Close' is called from destructor '~Clip()' at line 237. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/Clip.cpp:237:2: note: Calling Close libopenshot-0.3.3/src/Clip.h:224:8: note: Close is a virtual function 2.18.0 libopenshot-0.3.3/src/Clip.h:233:36: performance: Function 'Effects()' should return member 'effects' by const reference. [returnByReference] 2.18.0 libopenshot-0.3.3/src/Clip.h:272:8: style: Virtual function 'Open' is called from constructor 'Clip(openshot::ReaderBase*new_reader)' at line 148. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/Clip.cpp:148:2: note: Calling Open libopenshot-0.3.3/src/Clip.h:272:8: note: Open is a virtual function 2.18.0 libopenshot-0.3.3/src/Clip.h:289:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Clip.h:302:8: style:inconclusive: Technically the member function 'openshot::Clip::Waveform' can be const. [functionConst] 2.18.0 libopenshot-0.3.3/src/Clip.h:49:42: style: Parameter 'lhs' can be declared as pointer to const [constParameterPointer] 2.18.0 libopenshot-0.3.3/src/Clip.h:49:69: style: Parameter 'rhs' can be declared as pointer to const [constParameterPointer] 2.18.0 libopenshot-0.3.3/src/Clip.h:49:8: style:inconclusive: Technically the member function 'openshot::CompareClipEffects::operator()' can be const. [functionConst] 2.18.0 libopenshot-0.3.3/src/Clip.h:92:24: warning: The class 'Clip' defines member variable with name 'getFrameMutex' also defined in its parent class 'ReaderBase'. [duplInheritedMember] libopenshot-0.3.3/src/ReaderBase.h:79:24: note: Parent variable 'ReaderBase::getFrameMutex' libopenshot-0.3.3/src/Clip.h:92:24: note: Derived variable 'Clip::getFrameMutex' 2.18.0 libopenshot-0.3.3/src/ClipBase.cpp:102:39: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable objectValue [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/ClipBase.cpp:132:23: performance:inconclusive: Technically the member function 'openshot::ClipBase::add_property_choice_json' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/ClipBase.h:47:15: note: Technically the member function 'openshot::ClipBase::add_property_choice_json' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/ClipBase.cpp:132:23: note: Technically the member function 'openshot::ClipBase::add_property_choice_json' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/ClipBase.cpp:132:60: performance: Function parameter 'name' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ClipBase.cpp:135:45: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable objectValue [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/ClipBase.cpp:25:30: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/ClipBase.cpp:36:30: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/ClipBase.cpp:47:30: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/ClipBase.cpp:58:30: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/ClipBase.cpp:96:102: performance: Function parameter 'memo' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ClipBase.cpp:96:23: performance:inconclusive: Technically the member function 'openshot::ClipBase::add_property_json' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/ClipBase.h:44:15: note: Technically the member function 'openshot::ClipBase::add_property_json' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/ClipBase.cpp:96:23: note: Technically the member function 'openshot::ClipBase::add_property_json' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/ClipBase.cpp:96:53: performance: Function parameter 'name' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ClipBase.cpp:96:84: performance: Function parameter 'type' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ClipBase.h:103:42: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ClipBase.h:60:30: style: Parameter 'a' can be declared as reference to const [constParameterReference] 2.18.0 libopenshot-0.3.3/src/ClipBase.h:60:8: style:inconclusive: Technically the member function 'openshot::ClipBase::operator<' can be const. [functionConst] 2.18.0 libopenshot-0.3.3/src/ClipBase.h:61:31: style: Parameter 'a' can be declared as reference to const [constParameterReference] 2.18.0 libopenshot-0.3.3/src/ClipBase.h:61:8: style:inconclusive: Technically the member function 'openshot::ClipBase::operator<=' can be const. [functionConst] 2.18.0 libopenshot-0.3.3/src/ClipBase.h:62:30: style: Parameter 'a' can be declared as reference to const [constParameterReference] 2.18.0 libopenshot-0.3.3/src/ClipBase.h:62:8: style:inconclusive: Technically the member function 'openshot::ClipBase::operator>' can be const. [functionConst] 2.18.0 libopenshot-0.3.3/src/ClipBase.h:63:31: style: Parameter 'a' can be declared as reference to const [constParameterReference] 2.18.0 libopenshot-0.3.3/src/ClipBase.h:63:8: style:inconclusive: Technically the member function 'openshot::ClipBase::operator>=' can be const. [functionConst] 2.18.0 libopenshot-0.3.3/src/ClipBase.h:85:15: performance: Function 'Id()' should return member 'id' by const reference. [returnByReference] 2.18.0 libopenshot-0.3.3/src/ClipBase.h:94:23: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ClipProcessingJobs.cpp:19:80: performance: Function parameter 'processInfoJson' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ClipProcessingJobs.cpp:23:62: performance: Function parameter 'json' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ClipProcessingJobs.h:36:15: style:inconclusive: Member variable 'ClipProcessingJobs::processInfoJson' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/ClipProcessingJobs.cpp:20:33: note: Member variable 'ClipProcessingJobs::processInfoJson' is in the wrong place in the initializer list. libopenshot-0.3.3/src/ClipProcessingJobs.h:36:15: note: Member variable 'ClipProcessingJobs::processInfoJson' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/Color.h:63:33: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/DummyReader.cpp:167:45: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/DummyReader.cpp:72:16: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/DummyReader.cpp:96:6: style: The statement 'if (is_open) is_open=false' is logically equivalent to 'is_open=false'. [duplicateConditionalAssign] libopenshot-0.3.3/src/DummyReader.cpp:99:11: note: Assignment 'is_open=false' libopenshot-0.3.3/src/DummyReader.cpp:96:6: note: Condition 'is_open' is redundant 2.18.0 libopenshot-0.3.3/src/DummyReader.h:107:12: style: The destructor '~DummyReader' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/ReaderBase.h:128:12: note: Virtual destructor in base class libopenshot-0.3.3/src/DummyReader.h:107:12: note: Destructor in derived class 2.18.0 libopenshot-0.3.3/src/DummyReader.h:130:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/DummyReader.h:89:36: style:inconclusive: Member variable 'DummyReader::image_frame' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/DummyReader.cpp:50:74: note: Member variable 'DummyReader::image_frame' is in the wrong place in the initializer list. libopenshot-0.3.3/src/DummyReader.h:89:36: note: Member variable 'DummyReader::image_frame' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/DummyReader.h:89:36: style:inconclusive: Member variable 'DummyReader::image_frame' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/DummyReader.cpp:58:49: note: Member variable 'DummyReader::image_frame' is in the wrong place in the initializer list. libopenshot-0.3.3/src/DummyReader.h:89:36: note: Member variable 'DummyReader::image_frame' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/DummyReader.h:89:36: style:inconclusive: Member variable 'DummyReader::image_frame' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/DummyReader.cpp:66:72: note: Member variable 'DummyReader::image_frame' is in the wrong place in the initializer list. libopenshot-0.3.3/src/DummyReader.h:89:36: note: Member variable 'DummyReader::image_frame' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/EffectBase.cpp:119:13: style: Variable 'parentTimeline' can be declared as pointer to const [constVariablePointer] 2.18.0 libopenshot-0.3.3/src/EffectBase.cpp:211:46: performance: Function parameter 'parentEffect_id' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/EffectBase.cpp:45:18: style:inconclusive: Technically the member function 'openshot::EffectBase::DisplayInfo' can be const. [functionConst] libopenshot-0.3.3/src/EffectBase.h:72:8: note: Technically the member function 'openshot::EffectBase::DisplayInfo' can be const. libopenshot-0.3.3/src/EffectBase.cpp:45:18: note: Technically the member function 'openshot::EffectBase::DisplayInfo' can be const. 2.18.0 libopenshot-0.3.3/src/EffectBase.cpp:46:15: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable fixed [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/EffectBase.cpp:60:17: performance:inconclusive: Technically the member function 'openshot::EffectBase::constrain' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/EffectBase.h:75:7: note: Technically the member function 'openshot::EffectBase::constrain' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/EffectBase.cpp:60:17: note: Technically the member function 'openshot::EffectBase::constrain' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/EffectBase.cpp:98:44: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/EffectBase.h:100:16: style: The function 'SetJsonValue' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/ClipBase.h:105:16: note: Virtual function in base class libopenshot-0.3.3/src/EffectBase.h:100:16: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/EffectBase.h:105:67: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/EffectBase.h:121:12: style: The destructor '~EffectBase' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/ClipBase.h:111:12: note: Virtual destructor in base class libopenshot-0.3.3/src/EffectBase.h:121:12: note: Destructor in derived class 2.18.0 libopenshot-0.3.3/src/EffectBase.h:53:2: style: The class 'EffectBase' does not declare a constructor although it has private member variables which likely require initialization. [noConstructor] 2.18.0 libopenshot-0.3.3/src/EffectBase.h:97:23: style: The function 'Json' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/ClipBase.h:102:23: note: Virtual function in base class libopenshot-0.3.3/src/EffectBase.h:97:23: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/EffectBase.h:98:16: style: The function 'SetJson' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/ClipBase.h:103:16: note: Virtual function in base class libopenshot-0.3.3/src/EffectBase.h:98:16: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/EffectBase.h:98:42: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/EffectBase.h:99:23: style: The function 'JsonValue' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/ClipBase.h:104:23: note: Virtual function in base class libopenshot-0.3.3/src/EffectBase.h:99:23: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/EffectInfo.cpp:26:25: performance:inconclusive: Technically the member function 'openshot::EffectInfo::CreateEffect' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/EffectInfo.h:32:15: note: Technically the member function 'openshot::EffectInfo::CreateEffect' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/EffectInfo.cpp:26:25: note: Technically the member function 'openshot::EffectInfo::CreateEffect' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/EffectInfo.cpp:26:50: performance: Function parameter 'effect_type' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:103:3: error: Exception thrown in function declared not to throw exceptions. [throwInNoexceptFunction] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:107:21: style:inconclusive: Technically the member function 'openshot::AudioLocation::is_near' can be const. [functionConst] libopenshot-0.3.3/src/AudioLocation.h:29:8: note: Technically the member function 'openshot::AudioLocation::is_near' can be const. libopenshot-0.3.3/src/FFmpegReader.cpp:107:21: note: Technically the member function 'openshot::AudioLocation::is_near' can be const. 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1300:20: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1301:20: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1301:42: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1398:10: style: Variable 'video_length' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1398:23: style: Variable 'video_length' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1454:15: style: Variable 't' can be declared as pointer to const [constVariablePointer] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1454:19: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1470:7: style: The scope of the variable 'possible_height' can be reduced. [variableScope] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1489:2: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AV_PIX_FMT_RGBA [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1489:2: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1538:31: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AV_NOPTS_VALUE [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1565:16: style: Variable 'data_size' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1565:6: style: The scope of the variable 'data_size' can be reduced. [variableScope] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1590:12: style: Variable 'used' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1590:7: style: Variable 'used' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1611:13: style: Variable 'data_size' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1689:19: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1707:26: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1838:7: style: Condition '!seek_worked' is always true [knownConditionTrueFalse] libopenshot-0.3.3/src/FFmpegReader.cpp:1834:22: note: Assignment 'seek_worked=false', assigned value is 0 libopenshot-0.3.3/src/FFmpegReader.cpp:1838:7: note: Condition '!seek_worked' is always true 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1840:72: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AVSEEK_FLAG_BACKWARD [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1911:23: style:inconclusive: Technically the member function 'openshot::FFmpegReader::GetPacketPTS' can be const. [functionConst] libopenshot-0.3.3/src/FFmpegReader.h:189:11: note: Technically the member function 'openshot::FFmpegReader::GetPacketPTS' can be const. libopenshot-0.3.3/src/FFmpegReader.cpp:1911:23: note: Technically the member function 'openshot::FFmpegReader::GetPacketPTS' can be const. 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:1914:55: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AV_NOPTS_VALUE [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:2032:10: style: Local variable 'video_pts' shadows outer variable [shadowVariable] libopenshot-0.3.3/src/FFmpegReader.h:143:11: note: Shadowed declaration libopenshot-0.3.3/src/FFmpegReader.cpp:2032:10: note: Shadow variable 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:2044:10: style: Local variable 'audio_pts' shadows outer variable [shadowVariable] libopenshot-0.3.3/src/FFmpegReader.h:142:11: note: Shadowed declaration libopenshot-0.3.3/src/FFmpegReader.cpp:2044:10: note: Shadow variable 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:2132:20: style:inconclusive: Technically the member function 'openshot::FFmpegReader::IsPartialFrame' can be const. [functionConst] libopenshot-0.3.3/src/FFmpegReader.h:195:8: note: Technically the member function 'openshot::FFmpegReader::IsPartialFrame' can be const. libopenshot-0.3.3/src/FFmpegReader.cpp:2132:20: note: Technically the member function 'openshot::FFmpegReader::IsPartialFrame' can be const. 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:2344:20: performance:inconclusive: Technically the member function 'openshot::FFmpegReader::RemoveAVFrame' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/FFmpegReader.h:207:8: note: Technically the member function 'openshot::FFmpegReader::RemoveAVFrame' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/FFmpegReader.cpp:2344:20: note: Technically the member function 'openshot::FFmpegReader::RemoveAVFrame' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:2356:20: performance:inconclusive: Technically the member function 'openshot::FFmpegReader::RemoveAVPacket' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/FFmpegReader.h:210:8: note: Technically the member function 'openshot::FFmpegReader::RemoveAVPacket' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/FFmpegReader.cpp:2356:20: note: Technically the member function 'openshot::FFmpegReader::RemoveAVPacket' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:237:8: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable codec_type [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:2384:46: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:664:20: style:inconclusive: Technically the member function 'openshot::FFmpegReader::HasAlbumArt' can be const. [functionConst] libopenshot-0.3.3/src/FFmpegReader.h:192:8: note: Technically the member function 'openshot::FFmpegReader::HasAlbumArt' can be const. libopenshot-0.3.3/src/FFmpegReader.cpp:664:20: note: Technically the member function 'openshot::FFmpegReader::HasAlbumArt' can be const. 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:669:55: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AV_DISPOSITION_ATTACHED_PIC [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:715:49: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AV_TIME_BASE [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:852:49: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AV_TIME_BASE [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:901:20: style:inconclusive: Technically the member function 'openshot::FFmpegReader::GetIsDurationKnown' can be const. [functionConst] libopenshot-0.3.3/src/FFmpegReader.h:273:8: note: Technically the member function 'openshot::FFmpegReader::GetIsDurationKnown' can be const. libopenshot-0.3.3/src/FFmpegReader.cpp:901:20: note: Technically the member function 'openshot::FFmpegReader::GetIsDurationKnown' can be const. 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:957:16: style: Condition '!enable_seek' is always true [knownConditionTrueFalse] libopenshot-0.3.3/src/FFmpegReader.cpp:953:9: note: Assuming that condition 'enable_seek' is not redundant libopenshot-0.3.3/src/FFmpegReader.cpp:957:16: note: Condition '!enable_seek' is always true 2.18.0 libopenshot-0.3.3/src/FFmpegReader.cpp:974:18: style: Variable 'check_seek' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.h:103:15: style:inconclusive: Member variable 'FFmpegReader::path' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/FFmpegReader.cpp:73:5: note: Member variable 'FFmpegReader::path' is in the wrong place in the initializer list. libopenshot-0.3.3/src/FFmpegReader.h:103:15: note: Member variable 'FFmpegReader::path' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/FFmpegReader.h:105:20: style:inconclusive: Member variable 'FFmpegReader::pFormatCtx' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/FFmpegReader.cpp:76:19: note: Member variable 'FFmpegReader::pFormatCtx' is in the wrong place in the initializer list. libopenshot-0.3.3/src/FFmpegReader.h:105:20: note: Member variable 'FFmpegReader::pFormatCtx' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/FFmpegReader.h:112:13: style:inconclusive: Member variable 'FFmpegReader::packet' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/FFmpegReader.cpp:75:29: note: Member variable 'FFmpegReader::packet' is in the wrong place in the initializer list. libopenshot-0.3.3/src/FFmpegReader.h:112:13: note: Member variable 'FFmpegReader::packet' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/FFmpegReader.h:114:8: style:inconclusive: Member variable 'FFmpegReader::is_open' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/FFmpegReader.cpp:73:99: note: Member variable 'FFmpegReader::is_open' is in the wrong place in the initializer list. libopenshot-0.3.3/src/FFmpegReader.h:114:8: note: Member variable 'FFmpegReader::is_open' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/FFmpegReader.h:115:8: style:inconclusive: Member variable 'FFmpegReader::is_duration_known' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/FFmpegReader.cpp:74:58: note: Member variable 'FFmpegReader::is_duration_known' is in the wrong place in the initializer list. libopenshot-0.3.3/src/FFmpegReader.h:115:8: note: Member variable 'FFmpegReader::is_duration_known' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/FFmpegReader.h:116:8: style:inconclusive: Member variable 'FFmpegReader::check_interlace' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/FFmpegReader.cpp:73:38: note: Member variable 'FFmpegReader::check_interlace' is in the wrong place in the initializer list. libopenshot-0.3.3/src/FFmpegReader.h:116:8: note: Member variable 'FFmpegReader::check_interlace' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/FFmpegReader.h:130:8: style:inconclusive: Member variable 'FFmpegReader::is_seeking' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/FFmpegReader.cpp:72:20: note: Member variable 'FFmpegReader::is_seeking' is in the wrong place in the initializer list. libopenshot-0.3.3/src/FFmpegReader.h:130:8: note: Member variable 'FFmpegReader::is_seeking' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/FFmpegReader.h:240:3: style: Class 'FFmpegReader' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.h:243:12: style: The destructor '~FFmpegReader' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/ReaderBase.h:128:12: note: Virtual destructor in base class libopenshot-0.3.3/src/FFmpegReader.h:243:12: note: Destructor in derived class 2.18.0 libopenshot-0.3.3/src/FFmpegReader.h:246:8: style: Virtual function 'Close' is called from constructor 'FFmpegReader(const std::string&path,bool inspect_reader=true)' at line 96. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/FFmpegReader.cpp:96:3: note: Calling Close libopenshot-0.3.3/src/FFmpegReader.h:246:8: note: Close is a virtual function 2.18.0 libopenshot-0.3.3/src/FFmpegReader.h:246:8: style: Virtual function 'Close' is called from destructor '~FFmpegReader()' at line 103. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/FFmpegReader.cpp:103:3: note: Calling Close libopenshot-0.3.3/src/FFmpegReader.h:246:8: note: Close is a virtual function 2.18.0 libopenshot-0.3.3/src/FFmpegReader.h:265:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.h:270:8: style: Virtual function 'Open' is called from constructor 'FFmpegReader(const std::string&path,bool inspect_reader=true)' at line 95. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/FFmpegReader.cpp:95:3: note: Calling Open libopenshot-0.3.3/src/FFmpegReader.h:270:8: note: Open is a virtual function 2.18.0 libopenshot-0.3.3/src/FFmpegReader.h:58:11: style:inconclusive: Technically the member function 'openshot::PacketStatus::packets_read' can be const. [functionConst] 2.18.0 libopenshot-0.3.3/src/FFmpegReader.h:63:11: style:inconclusive: Technically the member function 'openshot::PacketStatus::packets_decoded' can be const. [functionConst] 2.18.0 libopenshot-0.3.3/src/FFmpegUtilities.h:123:35: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AV_PIX_FMT_FLAG_ALPHA [valueFlowBailoutIncompleteVar] head libopenshot-0.3.3/src/FFmpegUtilities.h:70:0: error: failed to evaluate #if condition, undefined function-like macro invocation: AV_VERSION_INT( ... ) [preprocessorErrorDirective] head libopenshot-0.3.3/src/FFmpegUtilities.h:70:2: error: failed to evaluate #if condition, undefined function-like macro invocation: AV_VERSION_INT( ... ) [preprocessorErrorDirective] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:1041:18: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AVMEDIA_TYPE_AUDIO [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:1176:18: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AVMEDIA_TYPE_VIDEO [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:1416:49: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable FF_INPUT_BUFFER_PADDING_SIZE [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:1482:173: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AV_CODEC_ID_MPEG2VIDEO [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:148:23: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AV_CODEC_ID_NONE [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:1579:32: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:163:64: performance: Function parameter 'codec' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:1649:77: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AV_SAMPLE_FMT_S16 [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:1649:96: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:1723:20: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:1737:27: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:1839:27: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:1853:5: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:1868:21: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:1918:34: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:2040:16: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:2042:3: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:2082:104: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:2082:34: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AV_PIX_FMT_RGBA [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:2101:2: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:2101:2: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:2136:65: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AVFMT_RAWPICTURE [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:2305:19: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable SWS_FAST_BILINEAR [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:276:48: performance: Function parameter 'codec' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:286:64: performance: Function parameter 'codec' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:323:48: performance: Function parameter 'codec' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:333:61: performance: Function parameter 'name' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:346:59: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AV_FIELD_BB [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:603:45: performance: Function parameter 'codec_name' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:637:29: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AVFMT_NOFILE [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:780:74: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AVMEDIA_TYPE_VIDEO [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.cpp:982:29: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AVFMT_NOFILE [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.h:119:8: style:inconclusive: Member variable 'FFmpegWriter::is_writing' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/FFmpegWriter.cpp:79:71: note: Member variable 'FFmpegWriter::is_writing' is in the wrong place in the initializer list. libopenshot-0.3.3/src/FFmpegWriter.h:119:8: note: Member variable 'FFmpegWriter::is_writing' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.h:120:8: style:inconclusive: Member variable 'FFmpegWriter::is_open' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/FFmpegWriter.cpp:80:79: note: Member variable 'FFmpegWriter::is_open' is in the wrong place in the initializer list. libopenshot-0.3.3/src/FFmpegWriter.h:120:8: note: Member variable 'FFmpegWriter::is_open' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.h:130:19: style:inconclusive: Member variable 'FFmpegWriter::video_codec_ctx' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/FFmpegWriter.cpp:79:25: note: Member variable 'FFmpegWriter::video_codec_ctx' is in the wrong place in the initializer list. libopenshot-0.3.3/src/FFmpegWriter.h:130:19: note: Member variable 'FFmpegWriter::video_codec_ctx' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.h:132:15: style:inconclusive: Member variable 'FFmpegWriter::img_convert_ctx' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/FFmpegWriter.cpp:78:38: note: Member variable 'FFmpegWriter::img_convert_ctx' is in the wrong place in the initializer list. libopenshot-0.3.3/src/FFmpegWriter.h:132:15: note: Member variable 'FFmpegWriter::img_convert_ctx' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.h:135:12: style:inconclusive: Member variable 'FFmpegWriter::audio_encoder_buffer' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/FFmpegWriter.cpp:81:76: note: Member variable 'FFmpegWriter::audio_encoder_buffer' is in the wrong place in the initializer list. libopenshot-0.3.3/src/FFmpegWriter.h:135:12: note: Member variable 'FFmpegWriter::audio_encoder_buffer' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.h:143:7: style:inconclusive: Member variable 'FFmpegWriter::initial_audio_input_frame_size' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/FFmpegWriter.cpp:78:3: note: Member variable 'FFmpegWriter::initial_audio_input_frame_size' is in the wrong place in the initializer list. libopenshot-0.3.3/src/FFmpegWriter.h:143:7: note: Member variable 'FFmpegWriter::initial_audio_input_frame_size' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.h:146:15: style:inconclusive: Member variable 'FFmpegWriter::avr' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/FFmpegWriter.cpp:80:50: note: Member variable 'FFmpegWriter::avr' is in the wrong place in the initializer list. libopenshot-0.3.3/src/FFmpegWriter.h:146:15: note: Member variable 'FFmpegWriter::avr' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.h:212:3: style: Class 'FFmpegWriter' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.h:218:8: style: The function 'IsOpen' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/WriterBase.h:83:16: note: Virtual function in base class libopenshot-0.3.3/src/FFmpegWriter.h:218:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.h:224:8: style: The function 'Open' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/WriterBase.h:101:16: note: Virtual function in base class libopenshot-0.3.3/src/FFmpegWriter.h:224:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.h:307:8: style: The function 'WriteFrame' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/WriterBase.h:86:16: note: Virtual function in base class libopenshot-0.3.3/src/FFmpegWriter.h:307:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/FFmpegWriter.h:315:8: style: The function 'WriteFrame' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/WriterBase.h:89:16: note: Virtual function in base class libopenshot-0.3.3/src/FFmpegWriter.h:315:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/Frame.cpp:138:32: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable SmoothTransformation [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:165:19: style: The scope of the variable 'lines' can be reduced. [variableScope] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:166:19: style: The scope of the variable 'labels' can be reduced. [variableScope] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:216:39: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Format_RGBA8888_Premultiplied [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:380:16: style:inconclusive: Technically the member function 'openshot::Frame::GetBytes' can be const. [functionConst] libopenshot-0.3.3/src/Frame.h:205:11: note: Technically the member function 'openshot::Frame::GetBytes' can be const. libopenshot-0.3.3/src/Frame.cpp:380:16: note: Technically the member function 'openshot::Frame::GetBytes' can be const. 2.18.0 libopenshot-0.3.3/src/Frame.cpp:450:36: style:inconclusive: Function 'SetFrameNumber' argument 1 names different: declaration 'number' definition 'new_number'. [funcArgNamesDifferent] libopenshot-0.3.3/src/Frame.h:257:31: note: Function 'SetFrameNumber' argument 1 names different: declaration 'number' definition 'new_number'. libopenshot-0.3.3/src/Frame.cpp:450:36: note: Function 'SetFrameNumber' argument 1 names different: declaration 'number' definition 'new_number'. 2.18.0 libopenshot-0.3.3/src/Frame.cpp:456:39: style:inconclusive: Function 'GetSamplesPerFrame' argument 1 names different: declaration 'frame_number' definition 'number'. [funcArgNamesDifferent] libopenshot-0.3.3/src/Frame.h:229:41: note: Function 'GetSamplesPerFrame' argument 1 names different: declaration 'frame_number' definition 'number'. libopenshot-0.3.3/src/Frame.cpp:456:39: note: Function 'GetSamplesPerFrame' argument 1 names different: declaration 'frame_number' definition 'number'. 2.18.0 libopenshot-0.3.3/src/Frame.cpp:45:8: warning: Member variable 'Frame::audio_reversed' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:484:12: style:inconclusive: Technically the member function 'openshot::Frame::GetSamplesPerFrame' can be const. [functionConst] libopenshot-0.3.3/src/Frame.h:226:7: note: Technically the member function 'openshot::Frame::GetSamplesPerFrame' can be const. libopenshot-0.3.3/src/Frame.cpp:484:12: note: Technically the member function 'openshot::Frame::GetSamplesPerFrame' can be const. 2.18.0 libopenshot-0.3.3/src/Frame.cpp:490:12: style:inconclusive: Technically the member function 'openshot::Frame::GetHeight' can be const. [functionConst] libopenshot-0.3.3/src/Frame.h:223:7: note: Technically the member function 'openshot::Frame::GetHeight' can be const. libopenshot-0.3.3/src/Frame.cpp:490:12: note: Technically the member function 'openshot::Frame::GetHeight' can be const. 2.18.0 libopenshot-0.3.3/src/Frame.cpp:496:12: style:inconclusive: Technically the member function 'openshot::Frame::GetWidth' can be const. [functionConst] libopenshot-0.3.3/src/Frame.h:238:7: note: Technically the member function 'openshot::Frame::GetWidth' can be const. libopenshot-0.3.3/src/Frame.cpp:496:12: note: Technically the member function 'openshot::Frame::GetWidth' can be const. 2.18.0 libopenshot-0.3.3/src/Frame.cpp:502:12: style:inconclusive: Technically the member function 'openshot::Frame::SampleRate' can be const. [functionConst] libopenshot-0.3.3/src/Frame.h:244:7: note: Technically the member function 'openshot::Frame::SampleRate' can be const. libopenshot-0.3.3/src/Frame.cpp:502:12: note: Technically the member function 'openshot::Frame::SampleRate' can be const. 2.18.0 libopenshot-0.3.3/src/Frame.cpp:508:22: style:inconclusive: Technically the member function 'openshot::Frame::ChannelsLayout' can be const. [functionConst] libopenshot-0.3.3/src/Frame.h:170:27: note: Technically the member function 'openshot::Frame::ChannelsLayout' can be const. libopenshot-0.3.3/src/Frame.cpp:508:22: note: Technically the member function 'openshot::Frame::ChannelsLayout' can be const. 2.18.0 libopenshot-0.3.3/src/Frame.cpp:515:30: performance: Function parameter 'path' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:515:61: performance: Function parameter 'format' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:526:32: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable SmoothTransformation [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:543:107: performance: Function parameter 'overlay_path' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:543:35: performance: Function parameter 'path' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:543:84: performance: Function parameter 'mask_path' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:544:15: performance: Function parameter 'background_color' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:544:65: performance: Function parameter 'format' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:548:34: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Format_RGBA8888_Premultiplied [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:59:8: warning: Member variable 'Frame::audio_reversed' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:62:65: performance: Function parameter 'color' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:62:8: warning: Member variable 'Frame::audio_reversed' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:666:12: performance:inconclusive: Technically the member function 'openshot::Frame::constrain' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/Frame.h:114:7: note: Technically the member function 'openshot::Frame::constrain' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/Frame.cpp:666:12: note: Technically the member function 'openshot::Frame::constrain' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/Frame.cpp:66:8: warning: Member variable 'Frame::audio_reversed' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:677:65: performance: Function parameter 'new_color' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:692:58: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Format_RGBA8888_Premultiplied [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:715:4: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable QImageCleanupFunction [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:716:3: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:71:8: warning:inconclusive: Member variable 'Frame::addingAudioMutex' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:71:8: warning:inconclusive: Member variable 'Frame::addingImageMutex' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:71:8: warning:inconclusive: Member variable 'Frame::audio_reversed' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:71:8: warning:inconclusive: Member variable 'Frame::previewApp' is not assigned in the copy constructor. Should it be copied? [missingMemberCopy] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:733:33: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Format_RGBA8888_Premultiplied [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:760:43: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Format_RGBA8888_Premultiplied [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:78:15: warning:inconclusive: Member variable 'Frame::audio_reversed' is not assigned a value in 'Frame::operator='. [operatorEqVarError] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:78:15: warning:inconclusive: Member variable 'Frame::previewApp' is not assigned a value in 'Frame::operator='. [operatorEqVarError] 2.18.0 libopenshot-0.3.3/src/Frame.cpp:792:55: style:inconclusive: Function 'ResizeAudio' argument 3 names different: declaration 'sample_rate' definition 'rate'. [funcArgNamesDifferent] libopenshot-0.3.3/src/Frame.h:241:50: note: Function 'ResizeAudio' argument 3 names different: declaration 'sample_rate' definition 'rate'. libopenshot-0.3.3/src/Frame.cpp:792:55: note: Function 'ResizeAudio' argument 3 names different: declaration 'sample_rate' definition 'rate'. 2.18.0 libopenshot-0.3.3/src/Frame.cpp:792:75: style:inconclusive: Function 'ResizeAudio' argument 4 names different: declaration 'channel_layout' definition 'layout'. [funcArgNamesDifferent] libopenshot-0.3.3/src/Frame.h:241:87: note: Function 'ResizeAudio' argument 4 names different: declaration 'channel_layout' definition 'layout'. libopenshot-0.3.3/src/Frame.cpp:792:75: note: Function 'ResizeAudio' argument 4 names different: declaration 'channel_layout' definition 'layout'. 2.18.0 libopenshot-0.3.3/src/Frame.cpp:931:3: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cout [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Frame.h:100:7: style:inconclusive: Member variable 'Frame::channels' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/Frame.cpp:49:4: note: Member variable 'Frame::channels' is in the wrong place in the initializer list. libopenshot-0.3.3/src/Frame.h:100:7: note: Member variable 'Frame::channels' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/Frame.h:102:7: style:inconclusive: Member variable 'Frame::width' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/Frame.cpp:47:20: note: Member variable 'Frame::width' is in the wrong place in the initializer list. libopenshot-0.3.3/src/Frame.h:102:7: note: Member variable 'Frame::width' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/Frame.h:106:11: style:inconclusive: Member variable 'Frame::max_audio_sample' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/Frame.cpp:52:4: note: Member variable 'Frame::max_audio_sample' is in the wrong place in the initializer list. libopenshot-0.3.3/src/Frame.h:106:11: note: Member variable 'Frame::max_audio_sample' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/Frame.h:211:22: style:inconclusive: Technically the member function 'openshot::Frame::GetPixelRatio' can be const. [functionConst] 2.18.0 libopenshot-0.3.3/src/Frame.h:99:22: style:inconclusive: Member variable 'Frame::pixel_ratio' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/Frame.cpp:48:4: note: Member variable 'Frame::pixel_ratio' is in the wrong place in the initializer list. libopenshot-0.3.3/src/Frame.h:99:22: note: Member variable 'Frame::pixel_ratio' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/FrameMapper.cpp:1026:8: style: Variable 'parent' can be declared as pointer to const [constVariablePointer] 2.18.0 libopenshot-0.3.3/src/FrameMapper.cpp:128:8: style: Variable 'parent' can be declared as pointer to const [constVariablePointer] 2.18.0 libopenshot-0.3.3/src/FrameMapper.cpp:229:32: style: Condition 'field>0' is always true [knownConditionTrueFalse] libopenshot-0.3.3/src/FrameMapper.cpp:169:24: note: Assignment 'field=1', assigned value is 1 libopenshot-0.3.3/src/FrameMapper.cpp:229:32: note: Condition 'field>0' is always true 2.18.0 libopenshot-0.3.3/src/FrameMapper.cpp:25:14: warning: Member variable 'FrameMapper::resampler' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/FrameMapper.cpp:25:55: style:inconclusive: Function 'FrameMapper' argument 2 names different: declaration 'target_fps' definition 'target'. [funcArgNamesDifferent] libopenshot-0.3.3/src/FrameMapper.h:236:44: note: Function 'FrameMapper' argument 2 names different: declaration 'target_fps' definition 'target'. libopenshot-0.3.3/src/FrameMapper.cpp:25:55: note: Function 'FrameMapper' argument 2 names different: declaration 'target_fps' definition 'target'. 2.18.0 libopenshot-0.3.3/src/FrameMapper.cpp:29:2: performance: Variable 'original' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] 2.18.0 libopenshot-0.3.3/src/FrameMapper.cpp:382:56: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Frame [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FrameMapper.cpp:517:21: warning: Either the condition 'odd_frame' is redundant or there is possible null pointer dereference: mapped_frame. [nullPointerRedundantCheck] libopenshot-0.3.3/src/FrameMapper.cpp:524:7: note: Assuming that condition 'odd_frame' is not redundant libopenshot-0.3.3/src/FrameMapper.cpp:522:38: note: Assignment to 'odd_frame=mapped_frame' libopenshot-0.3.3/src/FrameMapper.cpp:517:21: note: Null pointer dereference 2.18.0 libopenshot-0.3.3/src/FrameMapper.cpp:518:25: warning: Either the condition 'odd_frame' is redundant or there is possible null pointer dereference: mapped_frame. [nullPointerRedundantCheck] libopenshot-0.3.3/src/FrameMapper.cpp:524:7: note: Assuming that condition 'odd_frame' is not redundant libopenshot-0.3.3/src/FrameMapper.cpp:522:38: note: Assignment to 'odd_frame=mapped_frame' libopenshot-0.3.3/src/FrameMapper.cpp:518:25: note: Null pointer dereference 2.18.0 libopenshot-0.3.3/src/FrameMapper.cpp:665:45: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FrameMapper.cpp:750:45: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/FrameMapper.cpp:787:67: style:inconclusive: Function 'ChangeMapping' argument 2 names different: declaration 'pulldown' definition 'target_pulldown'. [funcArgNamesDifferent] libopenshot-0.3.3/src/FrameMapper.h:242:56: note: Function 'ChangeMapping' argument 2 names different: declaration 'pulldown' definition 'target_pulldown'. libopenshot-0.3.3/src/FrameMapper.cpp:787:67: note: Function 'ChangeMapping' argument 2 names different: declaration 'pulldown' definition 'target_pulldown'. 2.18.0 libopenshot-0.3.3/src/FrameMapper.cpp:863:28: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/FrameMapper.cpp:866:8: style: The scope of the variable 'valF' can be reduced. [variableScope] 2.18.0 libopenshot-0.3.3/src/FrameMapper.cpp:892:67: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AV_SAMPLE_FMT_S16 [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/FrameMapper.cpp:895:3: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/FrameMapper.cpp:931:15: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/FrameMapper.h:197:12: style:inconclusive: Member variable 'FrameMapper::target' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/FrameMapper.cpp:26:19: note: Member variable 'FrameMapper::target' is in the wrong place in the initializer list. libopenshot-0.3.3/src/FrameMapper.h:197:12: note: Member variable 'FrameMapper::target' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/FrameMapper.h:202:9: style:inconclusive: Member variable 'FrameMapper::parent_position' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/FrameMapper.cpp:26:89: note: Member variable 'FrameMapper::parent_position' is in the wrong place in the initializer list. libopenshot-0.3.3/src/FrameMapper.h:202:9: note: Member variable 'FrameMapper::parent_position' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/FrameMapper.h:239:12: style: The destructor '~FrameMapper' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/ReaderBase.h:128:12: note: Virtual destructor in base class libopenshot-0.3.3/src/FrameMapper.h:239:12: note: Destructor in derived class 2.18.0 libopenshot-0.3.3/src/FrameMapper.h:245:8: style: Virtual function 'Close' is called from destructor '~FrameMapper()' at line 58. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/FrameMapper.cpp:58:2: note: Calling Close libopenshot-0.3.3/src/FrameMapper.h:245:8: note: Close is a virtual function 2.18.0 libopenshot-0.3.3/src/FrameMapper.h:269:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ProcessingController.h:74:41: performance: Function parameter 'message' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Qt/AudioPlaybackThread.cpp:111:9: style: Variable 'possible_rates' can be declared as const array [constVariable] 2.18.0 libopenshot-0.3.3/src/Qt/AudioPlaybackThread.cpp:282:60: style: Condition 'is_playing' is always true [knownConditionTrueFalse] libopenshot-0.3.3/src/Qt/AudioPlaybackThread.cpp:254:44: note: Assuming that condition 'is_playing' is not redundant libopenshot-0.3.3/src/Qt/AudioPlaybackThread.cpp:282:60: note: Condition 'is_playing' is always true 2.18.0 libopenshot-0.3.3/src/Qt/AudioPlaybackThread.cpp:73:68: style: Consider using std::any_of algorithm instead of a raw loop. [useStlAlgorithm] 2.18.0 libopenshot-0.3.3/src/Qt/AudioPlaybackThread.h:45:34: performance: Variable 'initialise_error' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] 2.18.0 libopenshot-0.3.3/src/Qt/AudioPlaybackThread.h:45:3: warning: Member variable 'AudioDeviceManagerSingleton::defaultSampleRate' is not initialized in the constructor. [uninitMemberVarPrivate] 2.18.0 libopenshot-0.3.3/src/Qt/AudioPlaybackThread.h:91:3: style: Class 'AudioPlaybackThread' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/Qt/PlayerDemo.cpp:140:13: style: The scope of the variable 'project_json' can be reduced. [variableScope] 2.18.0 libopenshot-0.3.3/src/Qt/PlayerDemo.cpp:143:35: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable ReadOnly [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Qt/PlayerDemo.cpp:51:24: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable StrongFocus [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Qt/PlayerDemo.cpp:70:26: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Key_Space [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Qt/PlayerDemo.h:36:5: style: Class 'PlayerDemo' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/Qt/PlayerPrivate.h:33:13: style:inconclusive: Member variable 'PlayerPrivate::playback_frames' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/Qt/PlayerPrivate.cpp:27:77: note: Member variable 'PlayerPrivate::playback_frames' is in the wrong place in the initializer list. libopenshot-0.3.3/src/Qt/PlayerPrivate.h:33:13: note: Member variable 'PlayerPrivate::playback_frames' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/Qt/PlayerPrivate.h:34:10: style:inconclusive: Member variable 'PlayerPrivate::video_position' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/Qt/PlayerPrivate.cpp:26:39: note: Member variable 'PlayerPrivate::video_position' is in the wrong place in the initializer list. libopenshot-0.3.3/src/Qt/PlayerPrivate.h:34:10: note: Member variable 'PlayerPrivate::video_position' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/Qt/PlayerPrivate.h:36:24: style:inconclusive: Member variable 'PlayerPrivate::reader' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/Qt/PlayerPrivate.cpp:27:17: note: Member variable 'PlayerPrivate::reader' is in the wrong place in the initializer list. libopenshot-0.3.3/src/Qt/PlayerPrivate.h:36:24: note: Member variable 'PlayerPrivate::reader' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/Qt/PlayerPrivate.h:48:2: style: Class 'PlayerPrivate' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/Qt/VideoRenderWidget.cpp:26:38: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable black [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Qt/VideoRenderWidget.cpp:80:47: style:inconclusive: Function 'present' argument 1 names different: declaration 'image' definition 'm'. [funcArgNamesDifferent] libopenshot-0.3.3/src/Qt/VideoRenderWidget.h:47:32: note: Function 'present' argument 1 names different: declaration 'image' definition 'm'. libopenshot-0.3.3/src/Qt/VideoRenderWidget.cpp:80:47: note: Function 'present' argument 1 names different: declaration 'image' definition 'm'. 2.18.0 libopenshot-0.3.3/src/Qt/VideoRenderWidget.h:35:5: style: Class 'VideoRenderWidget' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/Qt/VideoRenderer.cpp:16:16: warning: Member variable 'VideoRenderer::override_widget' is not initialized in the constructor. [uninitMemberVar] 2.18.0 libopenshot-0.3.3/src/Qt/VideoRenderer.h:29:5: style: Class 'VideoRenderer' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/Qt/VideoRenderer.h:30:6: style: The destructor '~VideoRenderer' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/RendererBase.h:42:11: note: Virtual destructor in base class libopenshot-0.3.3/src/Qt/VideoRenderer.h:30:6: note: Destructor in derived class 2.18.0 libopenshot-0.3.3/src/Qt/VideoRenderer.h:33:10: style: The function 'OverrideWidget' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/RendererBase.h:38:15: note: Virtual function in base class libopenshot-0.3.3/src/Qt/VideoRenderer.h:33:10: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/Qt/VideoRenderer.h:40:10: style: The function 'render' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/RendererBase.h:44:15: note: Virtual function in base class libopenshot-0.3.3/src/Qt/VideoRenderer.h:40:10: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/QtHtmlReader.cpp:202:46: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/QtHtmlReader.cpp:50:59: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Format_RGBA8888_Premultiplied [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/QtHtmlReader.h:124:8: style: Virtual function 'Open' is called from constructor 'QtHtmlReader()' at line 31. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/QtHtmlReader.cpp:31:2: note: Calling Open libopenshot-0.3.3/src/QtHtmlReader.h:124:8: note: Open is a virtual function 2.18.0 libopenshot-0.3.3/src/QtHtmlReader.h:124:8: style: Virtual function 'Open' is called from constructor 'QtHtmlReader(int width,int height,int x_offset,int y_offset,GravityType gravity,std::string html,std::string css,std::string background_color)' at line 39. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/QtHtmlReader.cpp:39:2: note: Calling Open libopenshot-0.3.3/src/QtHtmlReader.h:124:8: note: Open is a virtual function 2.18.0 libopenshot-0.3.3/src/QtHtmlReader.h:76:15: style:inconclusive: Member variable 'QtHtmlReader::html' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/QtHtmlReader.cpp:36:91: note: Member variable 'QtHtmlReader::html' is in the wrong place in the initializer list. libopenshot-0.3.3/src/QtHtmlReader.h:76:15: note: Member variable 'QtHtmlReader::html' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/QtHtmlReader.h:99:8: style: Virtual function 'Close' is called from constructor 'QtHtmlReader()' at line 32. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/QtHtmlReader.cpp:32:2: note: Calling Close libopenshot-0.3.3/src/QtHtmlReader.h:99:8: note: Close is a virtual function 2.18.0 libopenshot-0.3.3/src/QtHtmlReader.h:99:8: style: Virtual function 'Close' is called from constructor 'QtHtmlReader(int width,int height,int x_offset,int y_offset,GravityType gravity,std::string html,std::string css,std::string background_color)' at line 40. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/QtHtmlReader.cpp:40:2: note: Calling Close libopenshot-0.3.3/src/QtHtmlReader.h:99:8: note: Close is a virtual function 2.18.0 libopenshot-0.3.3/src/QtImageReader.cpp:171:49: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable SmoothTransformation [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/QtImageReader.cpp:202:20: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/QtImageReader.cpp:238:27: style: Variable 't' can be declared as pointer to const [constVariablePointer] 2.18.0 libopenshot-0.3.3/src/QtImageReader.cpp:238:31: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/QtImageReader.cpp:289:9: style: Condition '!loaded' is always true [knownConditionTrueFalse] libopenshot-0.3.3/src/QtImageReader.cpp:254:19: note: Assignment 'loaded=false', assigned value is 0 libopenshot-0.3.3/src/QtImageReader.cpp:289:9: note: Condition '!loaded' is always true 2.18.0 libopenshot-0.3.3/src/QtImageReader.cpp:28:42: performance: Function parameter 'path' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/QtImageReader.cpp:302:56: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable KeepAspectRatio [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/QtImageReader.cpp:339:47: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/QtImageReader.h:105:3: style: Class 'QtImageReader' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/QtImageReader.h:107:12: style: The destructor '~QtImageReader' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/ReaderBase.h:128:12: note: Virtual destructor in base class libopenshot-0.3.3/src/QtImageReader.h:107:12: note: Destructor in derived class 2.18.0 libopenshot-0.3.3/src/QtImageReader.h:110:8: style: Virtual function 'Close' is called from constructor 'QtImageReader(std::string path,bool inspect_reader=true)' at line 33. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/QtImageReader.cpp:33:9: note: Calling Close libopenshot-0.3.3/src/QtImageReader.h:110:8: note: Close is a virtual function 2.18.0 libopenshot-0.3.3/src/QtImageReader.h:110:8: style: Virtual function 'Close' is called from destructor '~QtImageReader()' at line 39. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/QtImageReader.cpp:39:5: note: Calling Close libopenshot-0.3.3/src/QtImageReader.h:110:8: note: Close is a virtual function 2.18.0 libopenshot-0.3.3/src/QtImageReader.h:130:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/QtImageReader.h:135:8: style: Virtual function 'Open' is called from constructor 'QtImageReader(std::string path,bool inspect_reader=true)' at line 32. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/QtImageReader.cpp:32:9: note: Calling Open libopenshot-0.3.3/src/QtImageReader.h:135:8: note: Open is a virtual function 2.18.0 libopenshot-0.3.3/src/QtPlayer.cpp:123:21: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/QtPlayer.cpp:221:23: style:inconclusive: Technically the member function 'openshot::QtPlayer::GetRendererQObject' can be const. [functionConst] libopenshot-0.3.3/src/QtPlayer.h:90:10: note: Technically the member function 'openshot::QtPlayer::GetRendererQObject' can be const. libopenshot-0.3.3/src/QtPlayer.cpp:221:23: note: Technically the member function 'openshot::QtPlayer::GetRendererQObject' can be const. 2.18.0 libopenshot-0.3.3/src/QtPlayer.cpp:222:22: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/QtPlayer.cpp:36:7: warning: Class 'QtPlayer' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] 2.18.0 libopenshot-0.3.3/src/QtPlayer.cpp:36:7: warning: Class 'QtPlayer' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] 2.18.0 libopenshot-0.3.3/src/QtPlayer.cpp:50:20: performance:inconclusive: Technically the member function 'openshot::QtPlayer::CloseAudioDevice' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/QtPlayer.h:46:7: note: Technically the member function 'openshot::QtPlayer::CloseAudioDevice' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/QtPlayer.cpp:50:20: note: Technically the member function 'openshot::QtPlayer::CloseAudioDevice' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/QtPlayer.cpp:75:31: performance:inconclusive: Technically the member function 'openshot::QtPlayer::GetAudioDeviceNames' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/QtPlayer.h:55:18: note: Technically the member function 'openshot::QtPlayer::GetAudioDeviceNames' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/QtPlayer.cpp:75:31: note: Technically the member function 'openshot::QtPlayer::GetAudioDeviceNames' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/QtPlayer.cpp:90:24: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/QtPlayer.h:102:7: style: The function 'Reader' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/PlayerBase.h:82:16: note: Virtual function in base class libopenshot-0.3.3/src/QtPlayer.h:102:7: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/QtPlayer.h:105:24: style: The function 'Reader' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/PlayerBase.h:79:33: note: Virtual function in base class libopenshot-0.3.3/src/QtPlayer.h:105:24: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/QtPlayer.h:108:8: style: The function 'Volume' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/PlayerBase.h:85:17: note: Virtual function in base class libopenshot-0.3.3/src/QtPlayer.h:108:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/QtPlayer.h:111:7: style: The function 'Volume' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/PlayerBase.h:88:16: note: Virtual function in base class libopenshot-0.3.3/src/QtPlayer.h:111:7: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/QtPlayer.h:43:11: style: The destructor '~QtPlayer' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/PlayerBase.h:90:12: note: Virtual destructor in base class libopenshot-0.3.3/src/QtPlayer.h:43:11: note: Destructor in derived class 2.18.0 libopenshot-0.3.3/src/QtPlayer.h:61:7: style: The function 'Play' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/PlayerBase.h:58:16: note: Virtual function in base class libopenshot-0.3.3/src/QtPlayer.h:61:7: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/QtPlayer.h:64:7: style: The function 'Loading' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/PlayerBase.h:52:16: note: Virtual function in base class libopenshot-0.3.3/src/QtPlayer.h:64:7: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/QtPlayer.h:67:25: style: The function 'Mode' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/PlayerBase.h:55:24: note: Virtual function in base class libopenshot-0.3.3/src/QtPlayer.h:67:25: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/QtPlayer.h:70:7: style: The function 'Pause' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/PlayerBase.h:61:16: note: Virtual function in base class libopenshot-0.3.3/src/QtPlayer.h:70:7: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/QtPlayer.h:73:10: style: The function 'Position' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/PlayerBase.h:64:19: note: Virtual function in base class libopenshot-0.3.3/src/QtPlayer.h:73:10: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/QtPlayer.h:76:7: style: The function 'Seek' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/PlayerBase.h:67:16: note: Virtual function in base class libopenshot-0.3.3/src/QtPlayer.h:76:7: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/QtPlayer.h:93:8: style: The function 'Speed' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/PlayerBase.h:70:17: note: Virtual function in base class libopenshot-0.3.3/src/QtPlayer.h:93:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/QtPlayer.h:96:7: style: The function 'Speed' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/PlayerBase.h:73:16: note: Virtual function in base class libopenshot-0.3.3/src/QtPlayer.h:96:7: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/QtPlayer.h:99:7: style: The function 'Stop' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/PlayerBase.h:76:16: note: Virtual function in base class libopenshot-0.3.3/src/QtPlayer.h:99:7: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/QtTextReader.cpp:221:46: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/QtTextReader.cpp:41:55: performance: Function parameter 'color' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/QtTextReader.cpp:56:59: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Format_RGBA8888_Premultiplied [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/QtTextReader.h:107:8: style: Virtual function 'Close' is called from constructor 'QtTextReader()' at line 30. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/QtTextReader.cpp:30:2: note: Calling Close libopenshot-0.3.3/src/QtTextReader.h:107:8: note: Close is a virtual function 2.18.0 libopenshot-0.3.3/src/QtTextReader.h:107:8: style: Virtual function 'Close' is called from constructor 'QtTextReader(int width,int height,int x_offset,int y_offset,GravityType gravity,std::string text,QFont font,std::string text_color,std::string background_color)' at line 38. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/QtTextReader.cpp:38:2: note: Calling Close libopenshot-0.3.3/src/QtTextReader.h:107:8: note: Close is a virtual function 2.18.0 libopenshot-0.3.3/src/QtTextReader.h:132:8: style: Virtual function 'Open' is called from constructor 'QtTextReader()' at line 29. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/QtTextReader.cpp:29:2: note: Calling Open libopenshot-0.3.3/src/QtTextReader.h:132:8: note: Open is a virtual function 2.18.0 libopenshot-0.3.3/src/QtTextReader.h:132:8: style: Virtual function 'Open' is called from constructor 'QtTextReader(int width,int height,int x_offset,int y_offset,GravityType gravity,std::string text,QFont font,std::string text_color,std::string background_color)' at line 37. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/QtTextReader.cpp:37:2: note: Calling Open libopenshot-0.3.3/src/QtTextReader.h:132:8: note: Open is a virtual function 2.18.0 libopenshot-0.3.3/src/ReaderBase.cpp:121:34: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable objectValue [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/ReaderBase.cpp:237:101: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] 2.18.0 libopenshot-0.3.3/src/ReaderBase.cpp:62:15: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable fixed [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/ReaderBase.h:121:42: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:1132:36: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable arrayValue [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:1154:42: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:1269:42: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:126:26: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable ReadOnly [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:1309:22: style: Variable 'clip_id' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:1324:30: style: Consider using std::find_if algorithm instead of a raw loop. [useStlAlgorithm] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:1352:32: style: Consider using std::find_if algorithm instead of a raw loop. [useStlAlgorithm] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:1453:32: style: Consider using std::find_if algorithm instead of a raw loop. [useStlAlgorithm] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:147:73: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:1704:46: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable KeepAspectRatio [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:202:2: warning: Class 'Timeline' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:202:2: warning: Class 'Timeline' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:266:22: style: Consider using std::transform algorithm instead of a raw loop. [useStlAlgorithm] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:412:25: style: Consider using std::find_if algorithm instead of a raw loop. [useStlAlgorithm] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:424:27: style: Consider using std::find_if algorithm instead of a raw loop. [useStlAlgorithm] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:463:18: style:inconclusive: Technically the member function 'openshot::Timeline::GetMaxTime' can be const. [functionConst] libopenshot-0.3.3/src/Timeline.h:285:10: note: Technically the member function 'openshot::Timeline::GetMaxTime' can be const. libopenshot-0.3.3/src/Timeline.cpp:463:18: note: Technically the member function 'openshot::Timeline::GetMaxTime' can be const. 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:471:7: style: Local variable 'max_time' shadows outer variable [shadowVariable] libopenshot-0.3.3/src/Timeline.h:163:10: note: Shadowed declaration libopenshot-0.3.3/src/Timeline.cpp:471:7: note: Shadow variable 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:483:17: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:486:37: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:494:17: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:516:18: performance:inconclusive: Technically the member function 'openshot::Timeline::calculate_time' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/Timeline.h:183:10: note: Technically the member function 'openshot::Timeline::calculate_time' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/Timeline.cpp:516:18: note: Technically the member function 'openshot::Timeline::calculate_time' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:516:18: style: Unused private function: 'Timeline::calculate_time' [unusedPrivateFunction] libopenshot-0.3.3/src/Timeline.h:183:10: note: Unused private function: 'Timeline::calculate_time' libopenshot-0.3.3/src/Timeline.cpp:516:18: note: Unused private function: 'Timeline::calculate_time' 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:526:140: style: Parameter 'options' can be declared as pointer to const [constParameterPointer] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:74:37: performance: Function parameter 'info' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:882:16: performance:inconclusive: Technically the member function 'openshot::Timeline::isEqual' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/Timeline.h:197:8: note: Technically the member function 'openshot::Timeline::isEqual' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/Timeline.cpp:882:16: note: Technically the member function 'openshot::Timeline::isEqual' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/Timeline.cpp:912:26: style: Local variable 'frame' shadows outer variable [shadowVariable] libopenshot-0.3.3/src/Timeline.cpp:895:25: note: Shadowed declaration libopenshot-0.3.3/src/Timeline.cpp:912:26: note: Shadow variable 2.18.0 libopenshot-0.3.3/src/Timeline.h:221:3: style: Class 'Timeline' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/Timeline.h:232:12: style: The destructor '~Timeline' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] libopenshot-0.3.3/src/TimelineBase.h:55:12: note: Virtual destructor in base class libopenshot-0.3.3/src/Timeline.h:232:12: note: Destructor in derived class 2.18.0 libopenshot-0.3.3/src/Timeline.h:260:8: style:inconclusive: Technically the member function 'openshot::Timeline::AutoMapClips' can be const. [functionConst] 2.18.0 libopenshot-0.3.3/src/Timeline.h:290:8: style: Virtual function 'Close' is called from destructor '~Timeline()' at line 209. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/Timeline.cpp:209:3: note: Calling Close libopenshot-0.3.3/src/Timeline.h:290:8: note: Close is a virtual function 2.18.0 libopenshot-0.3.3/src/Timeline.h:293:36: performance: Function 'Effects()' should return member 'effects' by const reference. [returnByReference] 2.18.0 libopenshot-0.3.3/src/Timeline.h:327:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/Timeline.h:327:8: style: Virtual function 'SetJson' is called from constructor 'Timeline(const std::string&projectPath,bool convert_absolute_paths)' at line 172. Dynamic binding is not used. [virtualCallInConstructor] libopenshot-0.3.3/src/Timeline.cpp:172:2: note: Calling SetJson libopenshot-0.3.3/src/Timeline.h:327:8: note: SetJson is a virtual function 2.18.0 libopenshot-0.3.3/src/Timeline.h:50:36: style: Parameter 'lhs' can be declared as pointer to const [constParameterPointer] 2.18.0 libopenshot-0.3.3/src/Timeline.h:50:57: style: Parameter 'rhs' can be declared as pointer to const [constParameterPointer] 2.18.0 libopenshot-0.3.3/src/Timeline.h:50:8: style:inconclusive: Technically the member function 'openshot::CompareClips::operator()' can be const. [functionConst] 2.18.0 libopenshot-0.3.3/src/Timeline.h:60:42: style: Parameter 'lhs' can be declared as pointer to const [constParameterPointer] 2.18.0 libopenshot-0.3.3/src/Timeline.h:60:69: style: Parameter 'rhs' can be declared as pointer to const [constParameterPointer] 2.18.0 libopenshot-0.3.3/src/Timeline.h:60:8: style:inconclusive: Technically the member function 'openshot::CompareEffects::operator()' can be const. [functionConst] 2.18.0 libopenshot-0.3.3/src/Timeline.h:70:8: style:inconclusive: Technically the member function 'openshot::CompareClipEndFrames::operator()' can be const. [functionConst] 2.18.0 libopenshot-0.3.3/src/Timeline.h:76:8: style:inconclusive: Technically the member function 'openshot::CompareEffectEndFrames::operator()' can be const. [functionConst] 2.18.0 libopenshot-0.3.3/src/TrackedObjectBBox.cpp:178:25: performance:inconclusive: Technically the member function 'openshot::TrackedObjectBBox::InterpolateBoxes' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/TrackedObjectBBox.h:194:8: note: Technically the member function 'openshot::TrackedObjectBBox::InterpolateBoxes' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/TrackedObjectBBox.cpp:178:25: note: Technically the member function 'openshot::TrackedObjectBBox::InterpolateBoxes' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/TrackedObjectBBox.cpp:178:69: performance: Function parameter 'left' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/TrackedObjectBBox.cpp:178:80: performance: Function parameter 'right' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/TrackedObjectBBox.cpp:218:29: style:inconclusive: Technically the member function 'openshot::TrackedObjectBBox::GetBaseFPS' can be const. [functionConst] libopenshot-0.3.3/src/TrackedObjectBBox.h:163:12: note: Technically the member function 'openshot::TrackedObjectBBox::GetBaseFPS' can be const. libopenshot-0.3.3/src/TrackedObjectBBox.cpp:218:29: note: Technically the member function 'openshot::TrackedObjectBBox::GetBaseFPS' can be const. 2.18.0 libopenshot-0.3.3/src/TrackedObjectBBox.cpp:230:44: style:inconclusive: Function 'ScalePoints' argument 1 names different: declaration 'scale' definition 'time_scale'. [funcArgNamesDifferent] libopenshot-0.3.3/src/TrackedObjectBBox.h:166:27: note: Function 'ScalePoints' argument 1 names different: declaration 'scale' definition 'time_scale'. libopenshot-0.3.3/src/TrackedObjectBBox.cpp:230:44: note: Function 'ScalePoints' argument 1 names different: declaration 'scale' definition 'time_scale'. 2.18.0 libopenshot-0.3.3/src/TrackedObjectBBox.cpp:243:41: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable in [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/TrackedObjectBBox.cpp:270:9: style: The scope of the variable 'angle' can be reduced. [variableScope] 2.18.0 libopenshot-0.3.3/src/TrackedObjectBBox.cpp:338:51: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/TrackedObjectBBox.cpp:461:111: performance: Function parameter 'memo' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/TrackedObjectBBox.cpp:461:32: performance:inconclusive: Technically the member function 'openshot::TrackedObjectBBox::add_property_json' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/TrackedObjectBBox.h:210:15: note: Technically the member function 'openshot::TrackedObjectBBox::add_property_json' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/TrackedObjectBBox.cpp:461:32: note: Technically the member function 'openshot::TrackedObjectBBox::add_property_json' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/TrackedObjectBBox.cpp:461:62: performance: Function parameter 'name' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/TrackedObjectBBox.cpp:461:93: performance: Function parameter 'type' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/TrackedObjectBBox.cpp:467:39: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable objectValue [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/TrackedObjectBBox.cpp:82:42: style:inconclusive: Function 'Contains' argument 1 names different: declaration 'frame_number' definition 'frame_num'. [funcArgNamesDifferent] libopenshot-0.3.3/src/TrackedObjectBBox.h:169:25: note: Function 'Contains' argument 1 names different: declaration 'frame_number' definition 'frame_num'. libopenshot-0.3.3/src/TrackedObjectBBox.cpp:82:42: note: Function 'Contains' argument 1 names different: declaration 'frame_number' definition 'frame_num'. 2.18.0 libopenshot-0.3.3/src/TrackedObjectBBox.h:84:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/WriterBase.cpp:152:34: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable objectValue [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/WriterBase.cpp:187:44: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/WriterBase.cpp:91:18: style:inconclusive: Technically the member function 'openshot::WriterBase::DisplayInfo' can be const. [functionConst] libopenshot-0.3.3/src/WriterBase.h:98:8: note: Technically the member function 'openshot::WriterBase::DisplayInfo' can be const. libopenshot-0.3.3/src/WriterBase.cpp:91:18: note: Technically the member function 'openshot::WriterBase::DisplayInfo' can be const. 2.18.0 libopenshot-0.3.3/src/WriterBase.cpp:92:15: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable fixed [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/WriterBase.h:94:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:103:33: performance: Function parameter 'message' should be passed by const reference. [passedByValue] head libopenshot-0.3.3/src/ZmqLogger.cpp:116:0: error: failed to evaluate #if condition, undefined function-like macro invocation: ZMQ_MAKE_VERSION( ... ) [preprocessorErrorDirective] head libopenshot-0.3.3/src/ZmqLogger.cpp:116:2: error: failed to evaluate #if condition, undefined function-like macro invocation: ZMQ_MAKE_VERSION( ... ) [preprocessorErrorDirective] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:128:39: performance: Function parameter 'message' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:132:31: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable flush [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:135:34: performance: Function parameter 'new_path' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:145:46: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable out [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:149:11: style: Variable 'localtm' can be declared as pointer to const [constVariablePointer] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:149:26: portability: Non reentrant function 'localtime' called. For threadsafe applications it is recommended to use the reentrant replacement function 'localtime_r'. [prohibitedlocaltimeCalled] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:151:46: style: Obsolete function 'std::asctime' called. It is recommended to use 'strftime' instead. [prohibitedasctimeCalled] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:178:47: performance: Function parameter 'method_name' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:179:19: performance: Function parameter 'arg1_name' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:180:19: performance: Function parameter 'arg2_name' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:181:19: performance: Function parameter 'arg3_name' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:182:19: performance: Function parameter 'arg4_name' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:183:19: performance: Function parameter 'arg5_name' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:184:19: performance: Function parameter 'arg6_name' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:195:19: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable fixed [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:63:40: performance: Function parameter 'new_connection' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/ZmqLogger.cpp:87:42: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable ZMQ_PUB [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/audio_effects/Compressor.h:81:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/audio_effects/Delay.h:67:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/audio_effects/Distortion.h:64:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/audio_effects/Echo.h:69:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/audio_effects/Expander.h:82:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/audio_effects/Noise.cpp:24:23: performance: Function parameter 'level' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/audio_effects/Noise.cpp:87:39: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/audio_effects/Noise.h:50:3: style: Class 'Noise' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/audio_effects/Noise.h:61:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/audio_effects/ParametricEQ.h:66:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/audio_effects/Robotization.h:70:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/audio_effects/Whisperization.cpp:114:48: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/audio_effects/Whisperization.cpp:54:12: style: Variable 'num_input_channels' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/audio_effects/Whisperization.cpp:54:31: style: Variable 'num_input_channels' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/audio_effects/Whisperization.cpp:56:12: style: Variable 'num_samples' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/audio_effects/Whisperization.cpp:56:24: style: Variable 'num_samples' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/audio_effects/Whisperization.cpp:77:54: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable RAND_MAX [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/audio_effects/Whisperization.h:69:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/audio_effects/Whisperization.h:79:4: style: Class 'WhisperizationEffect' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/effects/Bars.cpp:124:38: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Bars.cpp:25:79: performance: Function parameter 'bottom' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Bars.cpp:55:36: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Format_RGBA8888_Premultiplied [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/effects/Bars.cpp:65:26: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/effects/Bars.cpp:66:17: style: Variable 'color_pixels' can be declared as pointer to const [constVariablePointer] 2.18.0 libopenshot-0.3.3/src/effects/Bars.cpp:66:32: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/effects/Bars.h:83:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Blur.cpp:122:12: performance:inconclusive: Technically the member function 'openshot::Blur::boxBlurT' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/effects/Blur.h:44:8: note: Technically the member function 'openshot::Blur::boxBlurT' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/effects/Blur.cpp:122:12: note: Technically the member function 'openshot::Blur::boxBlurT' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/effects/Blur.cpp:122:36: style: Parameter 'scl' can be declared as pointer to const [constParameterPointer] 2.18.0 libopenshot-0.3.3/src/effects/Blur.cpp:177:38: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Blur.cpp:25:103: performance: Function parameter 'new_iterations' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Blur.cpp:57:20: style: Variable 'sigma_value' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/Blur.cpp:57:8: style: Variable 'sigma_value' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/Blur.cpp:95:12: performance:inconclusive: Technically the member function 'openshot::Blur::boxBlurH' can be static (but you may consider moving to unnamed namespace). [functionStatic] libopenshot-0.3.3/src/effects/Blur.h:43:8: note: Technically the member function 'openshot::Blur::boxBlurH' can be static (but you may consider moving to unnamed namespace). libopenshot-0.3.3/src/effects/Blur.cpp:95:12: note: Technically the member function 'openshot::Blur::boxBlurH' can be static (but you may consider moving to unnamed namespace). 2.18.0 libopenshot-0.3.3/src/effects/Blur.cpp:95:36: style: Parameter 'scl' can be declared as pointer to const [constParameterPointer] 2.18.0 libopenshot-0.3.3/src/effects/Blur.h:86:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Brightness.cpp:116:44: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Brightness.cpp:25:58: performance: Function parameter 'new_contrast' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Brightness.cpp:57:26: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/effects/Brightness.h:75:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Caption.cpp:114:15: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/effects/Caption.cpp:120:14: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/effects/Caption.cpp:122:14: warning: Potentially invalid type conversion in old-style C cast, clarify/fix with C++ cast [dangerousTypeCast] 2.18.0 libopenshot-0.3.3/src/effects/Caption.cpp:148:35: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Antialiasing [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/effects/Caption.cpp:155:8: style: Local variable 'font' shadows outer variable [shadowVariable] libopenshot-0.3.3/src/effects/Caption.h:44:9: note: Shadowed declaration libopenshot-0.3.3/src/effects/Caption.cpp:155:8: note: Shadow variable 2.18.0 libopenshot-0.3.3/src/effects/Caption.cpp:157:16: style: Local variable 'metrics' shadows outer variable [shadowVariable] libopenshot-0.3.3/src/effects/Caption.h:43:16: note: Shadowed declaration libopenshot-0.3.3/src/effects/Caption.cpp:157:16: note: Shadow variable 2.18.0 libopenshot-0.3.3/src/effects/Caption.cpp:234:16: style: Local variable 'fitting_line' shadows outer variable [shadowVariable] libopenshot-0.3.3/src/effects/Caption.cpp:224:15: note: Shadowed declaration libopenshot-0.3.3/src/effects/Caption.cpp:234:16: note: Shadow variable 2.18.0 libopenshot-0.3.3/src/effects/Caption.cpp:391:41: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Caption.cpp:39:30: performance: Function parameter 'captions' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Caption.cpp:69:22: style:inconclusive: Technically the member function 'openshot::Caption::CaptionText' can be const. [functionConst] libopenshot-0.3.3/src/effects/Caption.h:100:14: note: Technically the member function 'openshot::Caption::CaptionText' can be const. libopenshot-0.3.3/src/effects/Caption.cpp:69:22: note: Technically the member function 'openshot::Caption::CaptionText' can be const. 2.18.0 libopenshot-0.3.3/src/effects/Caption.cpp:74:39: performance: Function parameter 'new_caption_text' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Caption.cpp:94:205: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable MultilineOption [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/effects/Caption.h:100:14: performance: Function 'CaptionText()' should return member 'caption_text' by const reference. [returnByReference] 2.18.0 libopenshot-0.3.3/src/effects/Caption.h:105:33: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Caption.h:42:14: style:inconclusive: Member variable 'Caption::caption_text' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/effects/Caption.cpp:43:3: note: Member variable 'Caption::caption_text' is in the wrong place in the initializer list. libopenshot-0.3.3/src/effects/Caption.h:42:14: note: Member variable 'Caption::caption_text' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/effects/Caption.h:43:16: style:inconclusive: Member variable 'Caption::metrics' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/effects/Caption.cpp:31:107: note: Member variable 'Caption::metrics' is in the wrong place in the initializer list. libopenshot-0.3.3/src/effects/Caption.h:43:16: note: Member variable 'Caption::metrics' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/effects/Caption.h:43:16: style:inconclusive: Member variable 'Caption::metrics' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/effects/Caption.cpp:41:103: note: Member variable 'Caption::metrics' is in the wrong place in the initializer list. libopenshot-0.3.3/src/effects/Caption.h:43:16: note: Member variable 'Caption::metrics' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/effects/Caption.h:44:9: style:inconclusive: Member variable 'Caption::font' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/effects/Caption.cpp:31:95: note: Member variable 'Caption::font' is in the wrong place in the initializer list. libopenshot-0.3.3/src/effects/Caption.h:44:9: note: Member variable 'Caption::font' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/effects/Caption.h:44:9: style:inconclusive: Member variable 'Caption::font' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/effects/Caption.cpp:41:91: note: Member variable 'Caption::font' is in the wrong place in the initializer list. libopenshot-0.3.3/src/effects/Caption.h:44:9: note: Member variable 'Caption::font' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/effects/Caption.h:45:7: style:inconclusive: Member variable 'Caption::is_dirty' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/effects/Caption.cpp:31:60: note: Member variable 'Caption::is_dirty' is in the wrong place in the initializer list. libopenshot-0.3.3/src/effects/Caption.h:45:7: note: Member variable 'Caption::is_dirty' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/effects/Caption.h:45:7: style:inconclusive: Member variable 'Caption::is_dirty' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/effects/Caption.cpp:41:56: note: Member variable 'Caption::is_dirty' is in the wrong place in the initializer list. libopenshot-0.3.3/src/effects/Caption.h:45:7: note: Member variable 'Caption::is_dirty' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/effects/Caption.h:59:11: style:inconclusive: Member variable 'Caption::background_corner' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/effects/Caption.cpp:32:38: note: Member variable 'Caption::background_corner' is in the wrong place in the initializer list. libopenshot-0.3.3/src/effects/Caption.h:59:11: note: Member variable 'Caption::background_corner' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/effects/Caption.h:59:11: style:inconclusive: Member variable 'Caption::background_corner' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/effects/Caption.cpp:42:34: note: Member variable 'Caption::background_corner' is in the wrong place in the initializer list. libopenshot-0.3.3/src/effects/Caption.h:59:11: note: Member variable 'Caption::background_corner' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/effects/Caption.h:61:11: style:inconclusive: Member variable 'Caption::stroke_width' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/effects/Caption.cpp:31:7: note: Member variable 'Caption::stroke_width' is in the wrong place in the initializer list. libopenshot-0.3.3/src/effects/Caption.h:61:11: note: Member variable 'Caption::stroke_width' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/effects/Caption.h:61:11: style:inconclusive: Member variable 'Caption::stroke_width' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/effects/Caption.cpp:41:3: note: Member variable 'Caption::stroke_width' is in the wrong place in the initializer list. libopenshot-0.3.3/src/effects/Caption.h:61:11: note: Member variable 'Caption::stroke_width' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/effects/Caption.h:78:2: style: Class 'Caption' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/effects/ChromaKey.cpp:103:20: style: Variable 'halothreshold' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ChromaKey.cpp:103:6: style: Variable 'halothreshold' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ChromaKey.cpp:114:17: style: Variable 'pixelcount' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ChromaKey.cpp:114:6: style: Variable 'pixelcount' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ChromaKey.cpp:27:2: performance: Variable 'color' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] 2.18.0 libopenshot-0.3.3/src/effects/ChromaKey.cpp:539:43: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/ChromaKey.h:102:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.cpp:100:25: style: Variable 'green_pixel_offset' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.cpp:101:24: style: Variable 'blue_pixel_offset' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.cpp:102:25: style: Variable 'alpha_pixel_offset' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.cpp:204:44: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.cpp:25:153: performance: Function parameter 'alpha_y' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.cpp:52:26: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.cpp:85:25: style: Variable 'starting_row_index' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.cpp:86:17: style: Variable 'byte_index' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.cpp:89:18: style: Variable 'R' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.cpp:90:18: style: Variable 'G' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.cpp:91:18: style: Variable 'B' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.cpp:92:18: style: Variable 'A' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.cpp:94:29: style: Variable 'red_starting_row_index' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.cpp:95:31: style: Variable 'green_starting_row_index' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.cpp:96:30: style: Variable 'blue_starting_row_index' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.cpp:97:31: style: Variable 'alpha_starting_row_index' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.cpp:99:23: style: Variable 'red_pixel_offset' is assigned a value that is never used. [unreadVariable] 2.18.0 libopenshot-0.3.3/src/effects/ColorShift.h:87:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Crop.cpp:144:38: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Crop.cpp:99:29: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Format_RGBA8888_Premultiplied [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/effects/Crop.h:92:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Deinterlace.cpp:103:45: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Deinterlace.cpp:26:31: style:inconclusive: Function 'Deinterlace' argument 1 names different: declaration 'isOdd' definition 'UseOddLines'. [funcArgNamesDifferent] libopenshot-0.3.3/src/effects/Deinterlace.h:48:20: note: Function 'Deinterlace' argument 1 names different: declaration 'isOdd' definition 'UseOddLines'. libopenshot-0.3.3/src/effects/Deinterlace.cpp:26:31: note: Function 'Deinterlace' argument 1 names different: declaration 'isOdd' definition 'UseOddLines'. 2.18.0 libopenshot-0.3.3/src/effects/Deinterlace.cpp:59:73: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Format_RGBA8888_Premultiplied [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/effects/Deinterlace.cpp:67:10: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/effects/Deinterlace.h:48:3: style: Class 'Deinterlace' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/effects/Deinterlace.h:71:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Hue.cpp:116:37: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Hue.cpp:25:19: performance: Function parameter 'hue' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Hue.cpp:60:8: style: Variable 'matrix' can be declared as const array [constVariable] 2.18.0 libopenshot-0.3.3/src/effects/Hue.cpp:67:26: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/effects/Hue.h:51:3: style: Class 'Hue' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] 2.18.0 libopenshot-0.3.3/src/effects/Hue.h:74:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Mask.cpp:163:26: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable objectValue [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/effects/Mask.cpp:171:38: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Mask.cpp:87:33: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable SmoothTransformation [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/effects/Mask.cpp:95:26: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/effects/Mask.cpp:96:17: style: Variable 'mask_pixels' can be declared as pointer to const [constVariablePointer] 2.18.0 libopenshot-0.3.3/src/effects/Mask.cpp:96:31: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/effects/Mask.h:41:8: style:inconclusive: Member variable 'Mask::needs_refresh' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/effects/Mask.cpp:29:52: note: Member variable 'Mask::needs_refresh' is in the wrong place in the initializer list. libopenshot-0.3.3/src/effects/Mask.h:41:8: note: Member variable 'Mask::needs_refresh' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/effects/Mask.h:41:8: style:inconclusive: Member variable 'Mask::needs_refresh' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/effects/Mask.cpp:36:100: note: Member variable 'Mask::needs_refresh' is in the wrong place in the initializer list. libopenshot-0.3.3/src/effects/Mask.h:41:8: note: Member variable 'Mask::needs_refresh' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/effects/Mask.h:47:8: style:inconclusive: Member variable 'Mask::replace_image' is in the wrong place in the initializer list. [initializerList] libopenshot-0.3.3/src/effects/Mask.cpp:36:78: note: Member variable 'Mask::replace_image' is in the wrong place in the initializer list. libopenshot-0.3.3/src/effects/Mask.h:47:8: note: Member variable 'Mask::replace_image' is in the wrong place in the initializer list. 2.18.0 libopenshot-0.3.3/src/effects/Mask.h:84:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Negate.cpp:19:9: warning: Member variable 'EffectBase::clip' is not initialized in the constructor. Maybe it should be initialized directly in the class EffectBase? [uninitDerivedMemberVar] 2.18.0 libopenshot-0.3.3/src/effects/Negate.cpp:19:9: warning: Member variable 'EffectBase::order' is not initialized in the constructor. Maybe it should be initialized directly in the class EffectBase? [uninitDerivedMemberVar] 2.18.0 libopenshot-0.3.3/src/effects/Negate.cpp:19:9: warning: Member variable 'EffectBase::parentEffect' is not initialized in the constructor. Maybe it should be initialized directly in the class EffectBase? [uninitDerivedMemberVar] 2.18.0 libopenshot-0.3.3/src/effects/Negate.cpp:62:40: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Negate.h:62:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Negate.h:64:8: style: The function 'SetJsonValue' overrides a function in a base class but just delegates back to the base class. [uselessOverride] libopenshot-0.3.3/src/EffectBase.h:100:16: note: Virtual function in base class libopenshot-0.3.3/src/effects/Negate.h:64:8: note: Function in derived class 2.18.0 libopenshot-0.3.3/src/effects/Pixelate.cpp:116:42: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Pixelate.cpp:31:97: performance: Function parameter 'bottom' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Pixelate.cpp:62:9: style: The scope of the variable 'left_value' can be reduced. [variableScope] 2.18.0 libopenshot-0.3.3/src/effects/Pixelate.cpp:63:9: style: The scope of the variable 'top_value' can be reduced. [variableScope] 2.18.0 libopenshot-0.3.3/src/effects/Pixelate.cpp:64:9: style: The scope of the variable 'right_value' can be reduced. [variableScope] 2.18.0 libopenshot-0.3.3/src/effects/Pixelate.cpp:65:9: style: The scope of the variable 'bottom_value' can be reduced. [variableScope] 2.18.0 libopenshot-0.3.3/src/effects/Pixelate.cpp:80:75: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable SmoothTransformation [valueFlowBailoutIncompleteVar] 2.18.0 libopenshot-0.3.3/src/effects/Pixelate.h:86:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Saturation.cpp:173:44: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Saturation.cpp:25:100: performance: Function parameter 'saturation_B' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Saturation.cpp:70:26: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/effects/Saturation.h:86:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Shift.cpp:139:39: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Shift.cpp:25:35: performance: Function parameter 'y' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Shift.cpp:51:26: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/effects/Shift.h:77:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Wave.cpp:118:38: performance: Function parameter 'value' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Wave.cpp:25:101: performance: Function parameter 'speed_y' should be passed by const reference. [passedByValue] 2.18.0 libopenshot-0.3.3/src/effects/Wave.cpp:55:41: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/effects/Wave.cpp:56:26: style: C-style pointer casting [cstyleCast] 2.18.0 libopenshot-0.3.3/src/effects/Wave.cpp:81:17: style: Checking if unsigned expression 'source_px' is less than zero. [unsignedLessThanZero] 2.18.0 libopenshot-0.3.3/src/effects/Wave.h:83:34: performance: Function parameter 'value' should be passed by const reference. [passedByValue] DONE