Editing PVS static analyzer - 5.0 Release
Jump to navigation
Jump to search
The edit can be undone. Please check the comparison below to verify that this is what you want to do, and then publish the changes below to finish undoing the edit.
Latest revision | Your text | ||
Line 10,618: | Line 10,618: | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30% | <td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-regexp.cc">lo-regexp.cc (127)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v555/ V555]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v555/ V555]</td> | ||
<td style='width: 65%;'>The expression 'new_pos - pos > 0' will work as 'new_pos != pos'.</td> | <td style='width: 65%;'>The expression 'new_pos - pos > 0' will work as 'new_pos != pos'.</td> | ||
Line 10,633: | Line 10,633: | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30% | <td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (44)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | ||
<td style='width: 65%;'>An odd precise comparison: octave::math::x_nint(x) == x. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | <td style='width: 65%;'>An odd precise comparison: octave::math::x_nint(x) == x. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30% | <td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (47)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | ||
<td style='width: 65%;'>An odd precise comparison: x == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | <td style='width: 65%;'>An odd precise comparison: x == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30% | <td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (47)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | ||
<td style='width: 65%;'>An odd precise comparison: x == 1. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | <td style='width: 65%;'>An odd precise comparison: x == 1. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30% | <td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (50)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | ||
<td style='width: 65%;'>An odd precise comparison: x == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | <td style='width: 65%;'>An odd precise comparison: x == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30% | <td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (65)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | ||
<td style='width: 65%;'>An odd precise comparison: octave::math::x_nint(x) == x. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | <td style='width: 65%;'>An odd precise comparison: octave::math::x_nint(x) == x. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30% | <td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (68)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | ||
<td style='width: 65%;'>An odd precise comparison: x == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | <td style='width: 65%;'>An odd precise comparison: x == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30% | <td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (68)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | ||
<td style='width: 65%;'>An odd precise comparison: x == 1. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | <td style='width: 65%;'>An odd precise comparison: x == 1. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30% | <td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (71)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v550/ V550]</td> | ||
<td style='width: 65%;'>An odd precise comparison: x == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | <td style='width: 65%;'>An odd precise comparison: x == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30% | <td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (113)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v575/ V575]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v575/ V575]</td> | ||
<td style='width: 65%;'>The potential null pointer is passed into 'fgets' function. Inspect the first argument. Check lines: 113, 104.</td> | <td style='width: 65%;'>The potential null pointer is passed into 'fgets' function. Inspect the first argument. Check lines: 113, 104.</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30% | <td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/lo-utils.cc">lo-utils.cc (129)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v769/ V769]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v769/ V769]</td> | ||
<td style='width: 65%;'>The 'buf' pointer in the 'buf + tmp' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. Check lines: 129, 122.</td> | <td style='width: 65%;'>The 'buf' pointer in the 'buf + tmp' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. Check lines: 129, 122.</td> | ||
Line 10,753: | Line 10,753: | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30% | <td style='width: 30%;'><div title="/home/pavel/octave/liboctave/util/oct-inttypes.cc">oct-inttypes.cc (784)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v524/ V524]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v524/ V524]</td> | ||
<td style='width: 65%;'>It is odd that the body of 'powf' function is fully equivalent to the body of 'pow' function.</td> | <td style='width: 65%;'>It is odd that the body of 'powf' function is fully equivalent to the body of 'pow' function.</td> | ||
Line 10,833: | Line 10,833: | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td style='width: 30% | <td style='width: 30%;'><div title="/home/pavel/octave/libgui/qterminal/libqterminal/unix/CharacterColor.h">CharacterColor.h (44)</div></td> | ||
<td style='width: 5%;'>[https://www.viva64.com/en/w/v690/ V690]</td> | <td style='width: 5%;'>[https://www.viva64.com/en/w/v690/ V690]</td> | ||
<td style='width: 65%;'>The 'ColorEntry' class implements the '=' operator, but lacks a copy constructor. It is dangerous to use such a class.</td> | <td style='width: 65%;'>The 'ColorEntry' class implements the '=' operator, but lacks a copy constructor. It is dangerous to use such a class.</td> |