Re: PVS-Studio report

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



What kind of dal? Lentil?

On Wed, 27 Nov 2024 at 2:32 PM, Xisco Fauli <xiscofauli@xxxxxxxxxxxxxxx> wrote:

Hello,

Unfortunately now it's flagging a lot of issues like

const SfxBoolItem* pStateItem = dynamic_cast<const SfxBoolItem*>( pItem  );
assert(pStateItem); // SfxBoolItem expected
if (pStateItem)
↑ V547 _expression_ 'pStateItem' is always true.

which is not ideal either, so I updated the report again without //V_ASSERT_CONTRACT

On 25/11/24 10:09, Xisco Fauli wrote:

Hello,

On 21/11/24 13:41, Caolán McNamara wrote:
On Thu, 2024-11-21 at 12:51 +0100, Xisco Fauli wrote:
-> it can be fixed with 
https://pvs-studio.com/en/docs/manual/0040/#IDBD1CC960F7
Do I read it right, that it just has to see the comment somewhere, e.g.
if added to include/sal/config.h to treat all asserts like that ?

Yes, that's one way. The other one is to pass a config file to pvs-studio-analyzer with the -R flag. https://report.dev.libreoffice.org/ has been updated using //V_ASSERT_CONTRACT


-- 
Xisco Faulí
LibreOffice QA Team
IRC: x1sc0
-- 
Xisco Faulí
LibreOffice QA Team
IRC: x1sc0

[Index of Archives]     [LARTC]     [Bugtraq]     [Yosemite Forum]     [Photo]

  Powered by Linux