AI review of 2026-11-11_03-42-47-afb5cbae18f5-results.txt PR: #9014 https://github.com/cppcheck-opensource/cppcheck/pull/9014 Tested: afb5cbae18f524ec6c3d83836d8bdf5262727a2c Merge base: 4b66976df1dea47746a82c7bee12be7a322305d1 Reviewed: 2026-11-15 04:13:23 UTC Model: claude-opus-5-5 (effort high) Results: 1 reviewed of 1 in the report, the test is still running (900 analysed of 1000 requested packages) Verdicts: 1 improvement Tokens: 1786 input, 0 cache read, 1465 cache write, 167 output The verdicts are written by AI and can be wrong. ---- 1 / 1 ---- Verdict: IMPROVEMENT (high confidence) Package: https://ftp.debian.org/debian/pool/main/p/packetq/packetq_1.8.0+dfsg.orig.tar.xz Result: main packetq-1.8.0+dfsg/src/server.cpp:157:13: error:inconclusive: Non-local variable 'm_stream' will use object that points to local variable 'ptr'. [danglingLifetime] packetq-1.8.0+dfsg/src/server.cpp:258:44: note: Address of variable taken here. packetq-1.8.0+dfsg/src/server.cpp:258:44: note: Calling function 'push_front', 1st argument '&ptr[res]' value is lifetime=ptr packetq-1.8.0+dfsg/src/server.cpp:152:40: note: Passed to constructor of 'Buffer'. packetq-1.8.0+dfsg/src/server.cpp:152:39: note: Added to container 'm_stream'. packetq-1.8.0+dfsg/src/server.cpp:238:31: note: Variable created here. packetq-1.8.0+dfsg/src/server.cpp:157:13: note: Non-local variable 'm_stream' will use object that points to local variable 'ptr'. Explanation: Buffer's constructor deep-copies the data with new[] and memcpy, so m_stream never holds a pointer to the caller's local 'ptr'. On top of that, the warning at line 157 (inside write()) was reached by carrying the lifetime of m_stream from push_front() at line 152 into a different member function. That cross-function forwarding is exactly what the PR stops for non-local variables. The removed warning was a false positive.