Replace `if` with `assert`. #19

pvc1989 · 2020-09-18T15:00:59Z

The only case for marker[qid] == 0 to be true is when elen == 0, which means the qid element has 0 node. So, the if clause will never be entered and can be safely replace by an assert statement, which can be turned off by setting the NDEBUG compiler flag.

pvc1989 force-pushed the use-assert branch from dd56032 to dd4922d Compare September 23, 2020 04:47

pvc1989 force-pushed the use-assert branch from dd4922d to 693728b Compare December 16, 2020 07:54

Replace if with assert.

c611bed

pvc1989 force-pushed the use-assert branch from 693728b to c611bed Compare March 24, 2021 03:02

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Replace `if` with `assert`. #19

Replace `if` with `assert`. #19

pvc1989 commented Sep 18, 2020 •

edited

Loading

Replace if with assert. #19

Are you sure you want to change the base?

Replace if with assert. #19

Conversation

pvc1989 commented Sep 18, 2020 • edited Loading

Replace `if` with `assert`. #19

Replace `if` with `assert`. #19

pvc1989 commented Sep 18, 2020 •

edited

Loading