Date: | 12/21/18 19:17:45 |
---|---|
PVS-Studio Version: | 6.27.28929.1381 |
Total Warnings (GA): | 611 |
Total Warnings (OP): | 211 |
Total Warnings (64): | 81 |
Group | Projects | Location | Level | Code | Message |
---|---|---|---|---|---|
64-bit errors | core |
defs.h:917 | High | V205 | Explicit conversion of pointer type to 32-bit integer type: reinterpret_cast< wxUIntPtr > (p) |
64-bit errors | core |
calctrl.cpp:221 | High | V220 | Suspicious sequence of types castings: memsize -> 32-bit integer -> memsize. |
64-bit errors | core |
listbox.cpp:721 | High | V220 | Suspicious sequence of types castings: memsize -> 32-bit integer -> memsize. The value being cast: '(n)'. |
64-bit errors | core |
notebook.cpp:577 | High | V220 | Suspicious sequence of types castings: memsize -> 32-bit integer -> memsize. The value being cast: '(nPage)'. |
64-bit errors | core |
notebook.cpp:397 | High | V220 | Suspicious sequence of types castings: memsize -> 32-bit integer -> memsize. The value being cast: '(nPage)'. |
64-bit errors | core |
notebook.cpp:678 | High | V220 | Suspicious sequence of types castings: memsize -> 32-bit integer -> memsize. The value being cast: '(nPage)'. |
64-bit errors | core |
notebook.cpp:450 | High | V220 | Suspicious sequence of types castings: memsize -> 32-bit integer -> memsize. The value being cast: '(nPage)'. |
64-bit errors | core |
notebook.cpp:438 | High | V220 | Suspicious sequence of types castings: memsize -> 32-bit integer -> memsize. The value being cast: '(nPage)'. |
64-bit errors | core |
notebook.cpp:425 | High | V220 | Suspicious sequence of types castings: memsize -> 32-bit integer -> memsize. The value being cast: '(nPage)'. |
64-bit errors | core |
notebook.cpp:401 | High | V220 | Suspicious sequence of types castings: memsize -> 32-bit integer -> memsize. The value being cast: '(nPage)'. |
64-bit errors | core |
treectrl.cpp:2606 | High | V220 | Suspicious sequence of types castings: memsize -> 32-bit integer -> memsize. |
64-bit errors | core |
treectrl.cpp:2552 | High | V220 | Suspicious sequence of types castings: memsize -> 32-bit integer -> memsize. |
64-bit errors | core |
window.cpp:6887 | High | V220 | Suspicious sequence of types castings: memsize -> 32-bit integer -> memsize. |
64-bit errors | core |
window.cpp:3535 | High | V220 | Suspicious sequence of types castings: memsize -> 32-bit integer -> memsize. The value being cast: 'lParam'. |
64-bit errors | core |
arrstr.h:513 | Medium | V302 | Member operator[] of 'wxArrayStringsAdapter' class has a 32-bit type argument. Use memsize-type here. |
64-bit errors | core |
cmdargs.h:97 | Medium | V302 | Member operator[] of 'wxCmdLineArgsArray' class has a 32-bit type argument. Use memsize-type here. |
64-bit errors | core |
cshelp.h:197 | Medium | V302 | Member operator[] of 'wxSimpleHelpProviderHashMap' class has a 32-bit type argument. Use memsize-type here. |
64-bit errors | core |
grid.h:909 | Medium | V302 | Member operator[] of 'wxUnsignedToIntHashMap' class has a 32-bit type argument. Use memsize-type here. |
64-bit errors | core |
hashmap.h:742 | Medium | V302 | Member operator[] of 'wxLongToLongHashMap' class has a 32-bit type argument. Use memsize-type here. |
64-bit errors | core |
image.h:189 | Medium | V302 | Member operator[] of 'wxImageHistogramBase' class has a 32-bit type argument. Use memsize-type here. |
64-bit errors | core |
string.h:1449 | Medium | V302 | Member operator[] of 'wxString' class has a 32-bit type argument. Use memsize-type here. |
64-bit errors | core |
string.h:191 | Medium | V302 | Member operator[] of 'wxCStrData' class has a 32-bit type argument. Use memsize-type here. |
64-bit errors | core |
string.h:1461 | Medium | V302 | Member operator[] of 'wxString' class has a 32-bit type argument. Use memsize-type here. |
64-bit errors | core |
string.h:1459 | Medium | V302 | Member operator[] of 'wxString' class has a 32-bit type argument. Use memsize-type here. |
64-bit errors | core |
string.h:192 | Medium | V302 | Member operator[] of 'wxCStrData' class has a 32-bit type argument. Use memsize-type here. |
64-bit errors | core |
string.h:1447 | Medium | V302 | Member operator[] of 'wxString' class has a 32-bit type argument. Use memsize-type here. |
64-bit errors | core |
dcgraph.cpp:1136 | Medium | V113 | Implicit type conversion from memsize to double type or vice versa. |
64-bit errors | core |
dcgraph.cpp:1138 | Medium | V113 | Implicit type conversion from memsize to double type or vice versa. |
64-bit errors | core |
dcsvg.cpp:567 | Medium | V113 | Implicit type conversion from memsize to double type or vice versa. |
64-bit errors | core |
dcsvg.cpp:566 | Medium | V113 | Implicit type conversion from memsize to double type or vice versa. |
64-bit errors | core |
dcsvg.cpp:896 | Medium | V128 | The 'm_clipNestingLevel' variable of the memsize type is written to a stream. Consider verifying the compatibility of 32 and 64 bit versions of the application in the context of a stored data. |
64-bit errors | core |
imagtiff.cpp:350 | Medium | V113 | Implicit type conversion from memsize to double type or vice versa. |
64-bit errors | core |
quantize.cpp:1347 | Medium | V123 | Allocation of memory by the pattern "(X *)malloc(sizeof(Y))" where the sizes of X and Y types are not equal. |
64-bit errors | core |
sizer.cpp:2224 | Medium | V127 | An overflow of the 32-bit 'n' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
sizer.cpp:2241 | Medium | V127 | An overflow of the 32-bit 'n' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
sizer.cpp:2269 | Medium | V127 | An overflow of the 32-bit 'n' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
sizer.cpp:1857 | Medium | V127 | An overflow of the 32-bit 'num' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
sizer.cpp:2341 | Medium | V127 | An overflow of the 32-bit 'n' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
sizer.cpp:2425 | Medium | V127 | An overflow of the 32-bit 'n' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
sizer.cpp:2400 | Medium | V127 | An overflow of the 32-bit 'n' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
sizer.cpp:1708 | Medium | V127 | An overflow of the 32-bit 'total' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
calctrlg.cpp:938 | Medium | V127 | An overflow of the 32-bit 'y' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
filectrlg.cpp:525 | Medium | V127 | An overflow of the 32-bit 'item.m_itemId' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
grid.cpp:6188 | Medium | V127 | An overflow of the 32-bit 'h' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
grid.cpp:6132 | Medium | V127 | An overflow of the 32-bit 'x' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
grid.cpp:6194 | Medium | V127 | An overflow of the 32-bit 'h' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
grid.cpp:6091 | Medium | V127 | An overflow of the 32-bit variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
grid.cpp:810 | Medium | V127 | An overflow of the 32-bit 'rowOrCol' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
grid.cpp:804 | Medium | V127 | An overflow of the 32-bit 'rowOrCol' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
grid.cpp:6127 | Medium | V127 | An overflow of the 32-bit 'y' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
gridctrl.cpp:487 | Medium | V127 | An overflow of the 32-bit 'width' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
tipwin.cpp:349 | Medium | V127 | An overflow of the 32-bit 'pt.y' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
vlbox.cpp:346 | Medium | V127 | An overflow of the 32-bit 'itemrect.y' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
vlbox.cpp:489 | Medium | V127 | An overflow of the 32-bit 'rectRow.y' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
vscroll.cpp:289 | Medium | V127 | An overflow of the 32-bit 'size' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
vscroll.cpp:397 | Medium | V127 | An overflow of the 32-bit 's' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
vscroll.cpp:537 | Medium | V127 | An overflow of the 32-bit 'orient_pos' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
vscroll.cpp:542 | Medium | V127 | An overflow of the 32-bit 'orient_size' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
vscroll.cpp:915 | Medium | V127 | An overflow of the 32-bit 'v_rect.y' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
vscroll.cpp:920 | Medium | V127 | An overflow of the 32-bit 'v_rect.height' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
vscroll.cpp:932 | Medium | V127 | An overflow of the 32-bit 'h_rect.width' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
vscroll.cpp:927 | Medium | V127 | An overflow of the 32-bit 'h_rect.x' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
vscroll.cpp:730 | Medium | V127 | An overflow of the 32-bit 's' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
vscroll.cpp:860 | Medium | V127 | An overflow of the 32-bit 'v_rect.y' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
vscroll.cpp:865 | Medium | V127 | An overflow of the 32-bit 'h_rect.x' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
dc.cpp:1479 | Medium | V113 | Implicit type conversion from memsize to double type or vice versa. |
64-bit errors | core |
dc.cpp:1478 | Medium | V113 | Implicit type conversion from memsize to double type or vice versa. |
64-bit errors | core |
dib.cpp:547 | Medium | V119 | More than one sizeof() operator is used in one expression. |
64-bit errors | core |
dib.cpp:392 | Medium | V119 | More than one sizeof() operator is used in one expression. |
64-bit errors | core |
dib.cpp:338 | Medium | V119 | More than one sizeof() operator is used in one expression. |
64-bit errors | core |
menu.cpp:1106 | Medium | V127 | An overflow of the 32-bit 'mswExtra' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
msgdlg.cpp:53 | Medium | V302 | Member operator[] of 'wxMessageDialogMap' class has a 32-bit type argument. Use memsize-type here. |
64-bit errors | core |
palette.cpp:93 | Medium | V119 | More than one sizeof() operator is used in one expression. |
64-bit errors | core |
printdlg.cpp:146 | Medium | V119 | More than one sizeof() operator is used in one expression. |
64-bit errors | core |
statusbar.cpp:486 | Medium | V127 | An overflow of the 32-bit 'width' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
statusbar.cpp:482 | Medium | V127 | An overflow of the 32-bit 'width' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
statusbar.cpp:476 | Medium | V127 | An overflow of the 32-bit 'width' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
64-bit errors | core |
taskbar.cpp:100 | Medium | V105 | Third operand of '?:' operation: implicit type conversion to memsize type. |
64-bit errors | core |
toplevel.cpp:438 | Medium | V119 | More than one sizeof() operator is used in one expression. |
64-bit errors | core |
window.cpp:167 | Medium | V302 | Member operator[] of 'MSWMessageHandlers' class has a 32-bit type argument. Use memsize-type here. |
64-bit errors | core |
datavgen.cpp:2863 | Low | V127 | An overflow of the 32-bit 'itemPosInNode' variable is possible inside a long cycle which utilizes a memsize-type loop counter. |
General Analysis | core |
any.h:730 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_buffer. |
General Analysis | core |
buffer.h:84 | High | V595 | The 'str' pointer was utilized before it was verified against nullptr. Check lines: 84, 87. |
General Analysis | core |
buffer.h:99 | High | V595 | The 'str' pointer was utilized before it was verified against nullptr. Check lines: 99, 102. |
General Analysis | core |
dvrenderers.h:309 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_enabled. |
General Analysis | core |
filectrl.h:94 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_filterIndex. |
General Analysis | core |
gauge.h:54 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_nDirection. |
General Analysis | core |
numdlgg.h:33 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_spinctrl. |
General Analysis | core |
sashwin.h:202 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_dragStatus. |
General Analysis | core |
textdlgg.h:39 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_dialogStyle. |
General Analysis | core |
list.h:335 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_key. |
General Analysis | core |
mimetype.h:157 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_iconIndex. |
General Analysis | core |
markupparser.h:28 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_fontSize. |
General Analysis | core |
string.h:3362 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_len. |
General Analysis | core |
textwrapper.h:82 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_sizer. |
General Analysis | core |
datavcmn.cpp:48 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_focusOnIdle. |
General Analysis | core |
datavcmn.cpp:1700 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_data. |
General Analysis | core |
dcbufcmn.cpp:117 | High | V595 | The 'm_dc' pointer was utilized before it was verified against nullptr. Check lines: 117, 130. |
General Analysis | core |
graphcmn.cpp:570 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_width, m_height. |
General Analysis | core |
imaggif.cpp:655 | High | V501 | There are identical sub-expressions 'wxGIFHandler_WriteWord(stream, 0)' to the left and to the right of the '&&' operator. |
General Analysis | core |
imaggif.cpp:771 | High | V616 | The 'loopcount' named constant with the value of 0 is used in the bitwise operation. |
General Analysis | core |
imagjpeg.cpp:262 | High | V614 | Uninitialized pointer 'cinfo.src' used. |
General Analysis | core |
stockitem.cpp:172 | High | V523 | The 'then' statement is equivalent to the 'else' statement. |
General Analysis | core |
stockitem.cpp:170 | High | V523 | The 'then' statement is equivalent to the 'else' statement. |
General Analysis | core |
stockitem.cpp:171 | High | V523 | The 'then' statement is equivalent to the 'else' statement. |
General Analysis | core |
stockitem.cpp:165 | High | V523 | The 'then' statement is equivalent to the 'else' statement. |
General Analysis | core |
stockitem.cpp:140 | High | V523 | The 'then' statement is equivalent to the 'else' statement. |
General Analysis | core |
stockitem.cpp:201 | High | V523 | The 'then' statement is equivalent to the 'else' statement. |
General Analysis | core |
stockitem.cpp:186 | High | V523 | The 'then' statement is equivalent to the 'else' statement. |
General Analysis | core |
stockitem.cpp:187 | High | V523 | The 'then' statement is equivalent to the 'else' statement. |
General Analysis | core |
stockitem.cpp:189 | High | V523 | The 'then' statement is equivalent to the 'else' statement. |
General Analysis | core |
stockitem.cpp:173 | High | V523 | The 'then' statement is equivalent to the 'else' statement. |
General Analysis | core |
toplvcmn.cpp:55 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_modified. |
General Analysis | core |
datavgen.cpp:4606 | High | V595 | The 'm_owner' pointer was utilized before it was verified against nullptr. Check lines: 4606, 4634. |
General Analysis | core |
datavgen.cpp:1535 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_dist_x, m_dist_y. |
General Analysis | core |
filectrlg.cpp:534 | High | V501 | There are identical sub-expressions '!(m_dirName.empty())' to the left and to the right of the '&&' operator. |
General Analysis | core |
grid.cpp:2220 | High | V678 | An object is used as an argument to its own method. Consider checking the first actual argument of the 'SetDefAttr' function. |
General Analysis | core |
grideditors.cpp:902 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_value. |
General Analysis | core |
grideditors.cpp:672 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_value. |
General Analysis | core |
propdlg.cpp:137 | High | V773 | The 'bookCtrl' pointer was assigned values twice without releasing the memory. A memory leak is possible. |
General Analysis | core |
propdlg.cpp:133 | High | V773 | The 'bookCtrl' pointer was assigned values twice without releasing the memory. A memory leak is possible. |
General Analysis | core |
propdlg.cpp:141 | High | V773 | The 'bookCtrl' pointer was assigned values twice without releasing the memory. A memory leak is possible. |
General Analysis | core |
propdlg.cpp:124 | High | V773 | The 'bookCtrl' pointer was assigned values twice without releasing the memory. A memory leak is possible. |
General Analysis | core |
richtooltipg.cpp:688 | High | V773 | Visibility scope of the 'popup' pointer was exited without releasing the memory. A memory leak is possible. |
General Analysis | core |
srchctlg.cpp:1141 | High | V547 | Expression 'multiplier != 1' is always true. |
General Analysis | core |
srchctlg.cpp:1053 | High | V547 | Expression 'multiplier != 1' is always true. |
General Analysis | core |
app.cpp:317 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_pfnGetConsoleCommandHistory, m_pfnGetConsoleCommandHistoryLength. |
General Analysis | core |
artmsw.cpp:220 | High | V547 | Expression 'stockIconId != ((SHSTOCKICONID) - 1)' is always true. |
General Analysis | core |
bitmap.cpp:1816 | High | V773 | The function was exited without releasing the 'mask' pointer. A memory leak is possible. |
General Analysis | core |
dcclient.cpp:72 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_ps. |
General Analysis | core |
imaglist.cpp:539 | High | V773 | The function was exited without releasing the 'pMask' pointer. A memory leak is possible. |
General Analysis | core |
joystick.cpp:119 | High | V564 | The '&' operator is applied to bool type value. You've probably forgotten to include parentheses or intended to use the '&&' operator. |
General Analysis | core |
listctrl.cpp:146 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_item. |
General Analysis | core |
listctrl.cpp:177 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_item. |
General Analysis | core |
menu.cpp:738 | High | V501 | There are identical sub-expressions '!RemoveMenu(hMenu, 0, 0x00000400L)' to the left and to the right of the '||' operator. |
General Analysis | core |
access.cpp:96 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_nCurrent. |
General Analysis | core |
access.cpp:1090 | High | V547 | Expression 'status == wxACC_NOT_SUPPORTED' is always false. |
General Analysis | core |
access.cpp:1088 | High | V547 | Expression 'status == wxACC_FAIL' is always false. |
General Analysis | core |
access.cpp:1093 | High | V547 | Expression 'status == wxACC_NOT_IMPLEMENTED' is always true. |
General Analysis | core |
activex.cpp:686 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_customId. |
General Analysis | core |
droptgt.cpp:171 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_hwnd. |
General Analysis | core |
printdlg.cpp:1115 | High | V547 | Expression '(pd->Flags & 0x00000100) != 0x00000100' is always true. |
General Analysis | core |
printdlg.cpp:1117 | High | V547 | Expression '(pd->Flags & 0x00000020) != 0x00000020' is always true. |
General Analysis | core |
printdlg.cpp:1118 | High | V547 | Expression '(pd->Flags & 0x00000400) == 0x00000400' is always false. |
General Analysis | core |
printdlg.cpp:1119 | High | V547 | Expression '(pd->Flags & 0x00000800) == 0x00000800' is always false. |
General Analysis | core |
printdlg.cpp:1113 | High | V547 | Expression '(pd->Flags & 0x00000000) == 0x00000000' is always true. |
General Analysis | core |
printdlg.cpp:1116 | High | V547 | Expression '(pd->Flags & 0x00000200) != 0x00000200' is always true. |
General Analysis | core |
printdlg.cpp:1114 | High | V547 | Expression '(pd->Flags & 0x00000010) != 0x00000010' is always true. |
General Analysis | core |
progdlg.cpp:77 | High | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_style, m_range, m_timeStop, m_state. |
General Analysis | core |
statbmp.cpp:116 | High | V773 | The function was exited without releasing the 'image' pointer. A memory leak is possible. |
General Analysis | core |
taskbarbutton.cpp:469 | High | V773 | The function was exited without releasing the 'item' pointer. A memory leak is possible. |
General Analysis | core |
taskbarbutton.cpp:1131 | High | V595 | The 'item' pointer was utilized before it was verified against nullptr. Check lines: 1131, 1134. |
General Analysis | core |
textctrl.cpp:451 | High | V547 | Expression 'm_verRichEdit == 2' is always true. |
General Analysis | core |
toplevel.cpp:439 | High | V641 | The size of the allocated memory buffer is not a multiple of the element size. |
General Analysis | core |
toplevel.cpp:1142 | High | V522 | Dereferencing of the null pointer 'm_menuSystem' might take place. |
General Analysis | core |
window.cpp:1201 | High | V549 | The fourth argument of 'ScrollWindow' function is equal to the fifth argument. |
General Analysis | core |
anystr.h:29 | Medium | V690 | The '=' operator is declared as private in the 'wxAnyStrPtr' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
collpane.h:77 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_bCollapsed. |
General Analysis | core |
combo.h:150 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_toplevEvtHandler, m_popupWinEvtHandler. |
General Analysis | core |
dynarray.h:231 | Medium | V690 | The 'wxBaseSortedArray' class implements the '=' operator, but lacks a copy constructor. It is dangerous to use such a class. |
General Analysis | core |
event.h:1345 | Medium | V690 | The 'wxAsyncMethodCallEvent0' class implements a copy constructor, but lacks the '=' operator. It is dangerous to use such a class. |
General Analysis | core |
event.h:1383 | Medium | V690 | The 'wxAsyncMethodCallEvent1' class implements a copy constructor, but lacks the '=' operator. It is dangerous to use such a class. |
General Analysis | core |
event.h:1426 | Medium | V690 | The 'wxAsyncMethodCallEvent2' class implements a copy constructor, but lacks the '=' operator. It is dangerous to use such a class. |
General Analysis | core |
event.h:4000 | Medium | V690 | The '=' operator is declared as private in the 'wxEventConnectionRef' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
filectrl.h:93 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_filterIndex. |
General Analysis | core |
filefn.h:58 | Medium | V677 | Custom declaration of a standard 'off_t' type. The system header file should be used: #include <WCHAR.H> or <SYS\TYPES.H>. |
General Analysis | core |
filepicker.h:150 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_pickerIface. |
General Analysis | core |
flags.h:35 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_data. |
General Analysis | core |
calctrlg.h:30 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_rowOffset. |
General Analysis | core |
choicdgg.h:36 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_listbox. |
General Analysis | core |
grid.h:2605 | Medium | V690 | The '=' operator is declared as private in the 'wxGridEditorCreatedEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
grid.h:962 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_numRows, m_numCols, m_defaultRowLabelValues, m_defaultColLabelValues, m_cellEditCtrlEnabled. |
General Analysis | core |
grid.h:915 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_sizeDefault. |
General Analysis | core |
grideditors.h:241 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_value. |
General Analysis | core |
propdlg.h:79 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_selectedPage. |
General Analysis | core |
splitter.h:59 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_sashPositionCurrent. |
General Analysis | core |
statbmpg.h:18 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_scaleMode. |
General Analysis | core |
stattextg.h:24 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_mnemonic. |
General Analysis | core |
mimetype.h:185 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_iconIndex. |
General Analysis | core |
module.h:34 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_state. |
General Analysis | core |
bitmap.h:251 | Medium | V690 | The 'wxMask' class implements a copy constructor, but lacks the '=' operator. It is dangerous to use such a class. |
General Analysis | core |
checkbox.h:20 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_state. |
General Analysis | core |
combo.h:39 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_animFlags. |
General Analysis | core |
dc.h:171 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_canvas. |
General Analysis | core |
uuid.h:58 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_uuid. |
General Analysis | core |
pen.h:31 | Medium | V773 | The 'm_refData' pointer was not released in destructor. A memory leak is possible. |
General Analysis | core |
uxtheme.h:177 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_lfW. |
General Analysis | core |
object.h:266 | Medium | V690 | The 'wxObjectDataPtr' class implements the '=' operator, but lacks a copy constructor. It is dangerous to use such a class. |
General Analysis | core |
sizer.h:312 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_id. |
General Analysis | core |
sizer.h:296 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_id, m_ratio. |
General Analysis | core |
sizer.h:281 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_id. |
General Analysis | core |
strconv.h:312 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: accum, bit. |
General Analysis | core |
strconv.h:283 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: accum, bit, msb, isLSB. |
General Analysis | core |
stream.h:378 | Medium | V690 | The 'wxFilterClassFactory' class implements the '=' operator, but lacks a copy constructor. It is dangerous to use such a class. |
General Analysis | core |
string.h:136 | Medium | V690 | The 'wxCStrData' class implements a copy constructor, but lacks the '=' operator. It is dangerous to use such a class. |
General Analysis | core |
strvararg.h:133 | Medium | V690 | The '=' operator is declared as private in the 'wxFormatString' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
taskbarbutton.h:45 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_id, m_enable, m_dismissOnClick, m_hasBackground, m_shown, m_interactive. |
General Analysis | core |
textcompleter.h:45 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_index. |
General Analysis | core |
timer.h:159 | Medium | V690 | The '=' operator is declared as private in the 'wxTimerEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
tokenzr.h:45 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_hasMoreTokens, m_delimsLen, m_lastDelim. |
General Analysis | core |
valnum.h:141 | Medium | V690 | The '=' operator is declared as private in the 'wxNumValidator' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
valnum.h:372 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_precision, m_min, m_max. |
General Analysis | core |
vector.h:175 | Medium | V690 | The 'reverse_iterator' class implements a copy constructor, but lacks the '=' operator. It is dangerous to use such a class. |
General Analysis | core |
vector.h:237 | Medium | V690 | The 'const_reverse_iterator' class implements a copy constructor, but lacks the '=' operator. It is dangerous to use such a class. |
General Analysis | core |
weakref.h:175 | Medium | V522 | There might be dereferencing of a potential null pointer 'pt'. |
General Analysis | core |
wizard.h:266 | Medium | V690 | The '=' operator is declared as private in the 'wxWizardEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
dcbase.cpp:1066 | Medium | V636 | The 'rect.GetHeight() / 2' expression was implicitly cast from 'int' type to 'double' type. Consider utilizing an explicit type cast to avoid the loss of a fractional part. An example: double A = (double)(X) / Y;. |
General Analysis | core |
dcbase.cpp:1065 | Medium | V636 | The 'rect.GetWidth() / 2' expression was implicitly cast from 'int' type to 'double' type. Consider utilizing an explicit type cast to avoid the loss of a fractional part. An example: double A = (double)(X) / Y;. |
General Analysis | core |
dcsvg.cpp:139 | Medium | V719 | The switch statement does not cover all values of the 'wxPenStyle' enum: wxPENSTYLE_FIRST_HATCH, wxPENSTYLE_LAST_HATCH. |
General Analysis | core |
dcsvg.cpp:194 | Medium | V719 | The switch statement does not cover all values of the 'wxBrushStyle' enum: wxBRUSHSTYLE_FIRST_HATCH, wxBRUSHSTYLE_LAST_HATCH. |
General Analysis | core |
docview.cpp:398 | Medium | V547 | Expression '!file' is always false. |
General Analysis | core |
event.cpp:756 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_rawCode, m_rawFlags. |
General Analysis | core |
fontcmn.cpp:546 | Medium | V547 | Expression 'static_cast<int> (style) == wxDEFAULT' is always false. |
General Analysis | core |
fontmap.cpp:412 | Medium | V547 | Expression '!title' is always false. |
General Analysis | core |
fontmap.cpp:198 | Medium | V547 | Expression '!title' is always false. |
General Analysis | core |
gifdecod.cpp:96 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_restbits, m_restbyte, m_lastbyte, m_buffer, m_bufp. |
General Analysis | core |
imagbmp.cpp:527 | Medium | V668 | There is no sense in testing the 'cmap' pointer against null, as the memory was allocated using the 'new' operator. The exception will be generated in the case of memory allocation error. |
General Analysis | core |
image.cpp:1539 | Medium | V1004 | The 'src_alpha' pointer was used unsafely after it was verified against nullptr. Check lines: 1526, 1539. |
General Analysis | core |
image.cpp:255 | Medium | V575 | The potential null pointer is passed into 'memcpy' function. Inspect the first argument. Check lines: 255, 254. |
General Analysis | core |
image.cpp:251 | Medium | V575 | The potential null pointer is passed into 'memcpy' function. Inspect the first argument. Check lines: 251, 250. |
General Analysis | core |
imaggif.cpp:476 | Medium | V668 | There is no sense in testing the 'm_hashTable' pointer against null, as the memory was allocated using the 'new' operator. The exception will be generated in the case of memory allocation error. |
General Analysis | core |
imaggif.cpp:293 | Medium | V560 | A part of conditional expression is always true: ok. |
General Analysis | core |
imagjpeg.cpp:491 | Medium | V665 | Possibly, the usage of '#pragma warning(default: X)' is incorrect in this context. The '#pragma warning(push/pop)' should be used instead. Check lines: 231, 491. |
General Analysis | core |
imagpng.cpp:873 | Medium | V665 | Possibly, the usage of '#pragma warning(default: X)' is incorrect in this context. The '#pragma warning(push/pop)' should be used instead. Check lines: 311, 873. |
General Analysis | core |
pickerbase.cpp:78 | Medium | V668 | There is no sense in testing the 'm_text' pointer against null, as the memory was allocated using the 'new' operator. The exception will be generated in the case of memory allocation error. |
General Analysis | core |
prntbase.cpp:2079 | Medium | V668 | There is no sense in testing the 'm_previewBitmap' pointer against null, as the memory was allocated using the 'new' operator. The exception will be generated in the case of memory allocation error. |
General Analysis | core |
quantize.cpp:1398 | Medium | V769 | The 'table' pointer in the 'table += (255 + 1)' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. Check lines: 1398, 1396. |
General Analysis | core |
quantize.cpp:1221 | Medium | V769 | The 'table' pointer in the 'table += 255' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. Check lines: 1221, 1220. |
General Analysis | core |
quantize.cpp:1347 | Medium | V522 | There might be dereferencing of a potential null pointer 'cquantize->histogram'. Check lines: 1347, 1345. |
General Analysis | core |
quantize.cpp:1360 | Medium | V522 | There might be dereferencing of a potential null pointer 'cquantize->sv_colormap'. Check lines: 1360, 1359. |
General Analysis | core |
quantize.cpp:1338 | Medium | V522 | There might be dereferencing of a potential null pointer 'cquantize'. Check lines: 1338, 1336. |
General Analysis | core |
quantize.cpp:635 | Medium | V522 | There might be dereferencing of a potential null pointer 'boxlist'. Check lines: 635, 632. |
General Analysis | core |
sizer.cpp:169 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_ratio. |
General Analysis | core |
xpmdecod.cpp:553 | Medium | V575 | The potential null pointer is passed into 'strchr' function. Inspect the first argument. Check lines: 553, 547. |
General Analysis | core |
xpmdecod.cpp:563 | Medium | V522 | There might be dereferencing of a potential null pointer 'p'. Check lines: 563, 547. |
General Analysis | core |
calctrlg.cpp:1498 | Medium | V560 | A part of conditional expression is always false: wday >= 7. |
General Analysis | core |
datavgen.cpp:6654 | Medium | V547 | Expression is always false. |
General Analysis | core |
dirctrlg.cpp:348 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_styleEx. |
General Analysis | core |
grid.cpp:855 | Medium | V547 | Expression '!attr' is always true. |
General Analysis | core |
grid.cpp:6054 | Medium | V764 | Possible incorrect order of arguments passed to 'GetTextBoxSize' function: 'textHeight' and 'textWidth'. |
General Analysis | core |
gridctrl.cpp:823 | Medium | V547 | Expression '!params' is always false. |
General Analysis | core |
gridctrl.cpp:750 | Medium | V547 | Expression '!m_format' is always false. |
General Analysis | core |
gridctrl.cpp:373 | Medium | V519 | The 'lineWidth' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 369, 373. |
General Analysis | core |
gridctrl.cpp:244 | Medium | V547 | Expression '!params' is always false. |
General Analysis | core |
grideditors.cpp:1026 | Medium | V547 | Expression '!params' is always false. |
General Analysis | core |
grideditors.cpp:624 | Medium | V547 | Expression '!params' is always false. |
General Analysis | core |
grideditors.cpp:852 | Medium | V547 | Expression '!params' is always false. |
General Analysis | core |
grideditors.cpp:1106 | Medium | V547 | Expression '!m_format' is always false. |
General Analysis | core |
grideditors.cpp:1569 | Medium | V547 | Expression '!params' is always false. |
General Analysis | core |
logg.cpp:1003 | Medium | V547 | Expression '!filename' is always false. |
General Analysis | core |
progdlgg.cpp:116 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_factor. |
General Analysis | core |
treectlg.cpp:2891 | Medium | V560 | A part of conditional expression is always true: count > 0. |
General Analysis | core |
treectlg.cpp:2747 | Medium | V560 | A part of conditional expression is always true: count > 0. |
General Analysis | core |
jmorecfg.h:207 | Medium | V677 | Custom declaration of a standard 'INT16' type. The system header file should be used: #include <BaseTsd.h>. |
General Analysis | core |
jmorecfg.h:199 | Medium | V677 | Custom declaration of a standard 'UINT16' type. The system header file should be used: #include <BaseTsd.h>. |
General Analysis | core |
jmorecfg.h:187 | Medium | V677 | Custom declaration of a standard 'UINT8' type. The system header file should be used: #include <BaseTsd.h>. |
General Analysis | core |
artmsw.cpp:220 | Medium | V1016 | The value '-1' is out of range of enum values. This causes unspecified or undefined behavior. |
General Analysis | core |
artmsw.cpp:92 | Medium | V1016 | The value '-1' is out of range of enum values. This causes unspecified or undefined behavior. |
General Analysis | core |
bitmap.cpp:690 | Medium | V769 | The 'dst' pointer in the 'dst ++' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. Check lines: 690, 673. |
General Analysis | core |
brush.cpp:239 | Medium | V773 | The 'm_refData' pointer was not released in destructor. A memory leak is possible. |
General Analysis | core |
combo.cpp:512 | Medium | V592 | The expression was enclosed by parentheses twice: '((!m_text || m_widthCustomPaint))'. One pair of parentheses is unnecessary or misprint is present. |
General Analysis | core |
dc.cpp:1519 | Medium | V547 | Expression '!m_oldPalette' is always true. |
General Analysis | core |
dc.cpp:476 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_canvas. |
General Analysis | core |
dialup.cpp:505 | Medium | V701 | realloc() possible leak: when realloc() fails in allocating memory, original pointer 'lpRasConn' is lost. Consider assigning realloc() to a temporary pointer. |
General Analysis | core |
dialup.cpp:784 | Medium | V547 | Expression '!username' is always false. |
General Analysis | core |
dialup.cpp:734 | Medium | V547 | Expression '!entryName' is always false. |
General Analysis | core |
dialup.cpp:770 | Medium | V547 | Expression '!entryName' is always false. |
General Analysis | core |
dialup.cpp:665 | Medium | V522 | There might be dereferencing of a potential null pointer 'rasEntries'. Check lines: 665, 664. |
General Analysis | core |
dialup.cpp:1173 | Medium | V720 | It is advised to utilize the 'SuspendThread' function only when developing a debugger (see documentation for details). |
General Analysis | core |
enhmeta.cpp:452 | Medium | V575 | The potential null pointer is passed into 'SetMetaFileBitsEx' function. Inspect the second argument. Check lines: 452, 439. |
General Analysis | core |
enhmeta.cpp:61 | Medium | V547 | Expression '!fn' is always false. |
General Analysis | core |
enhmeta.cpp:512 | Medium | V575 | The potential null pointer is passed into 'SetWinMetaFileBits' function. Inspect the second argument. Check lines: 512, 499. |
General Analysis | core |
filedlg.cpp:170 | Medium | V658 | A value is being subtracted from the unsigned variable. This can result in an overflow. In such a case, the '>' comparison operation can potentially behave unexpectedly. Consider inspecting the 'pNM->code > (0U - 699U)' expression. |
General Analysis | core |
filedlg.cpp:170 | Medium | V658 | A value is being subtracted from the unsigned variable. This can result in an overflow. In such a case, the '<=' comparison operation can potentially behave unexpectedly. Consider inspecting the 'pNM->code <= (0U - 601U)' expression. |
General Analysis | core |
filedlg.cpp:660 | Medium | V728 | An excessive check can be simplified. The '||' operator is surrounded by opposite expressions '!of.nFileExtension' and 'of.nFileExtension'. |
General Analysis | core |
font.cpp:799 | Medium | V773 | The 'm_refData' pointer was not released in destructor. A memory leak is possible. |
General Analysis | core |
font.cpp:275 | Medium | V522 | There might be dereferencing of a potential null pointer 'otm'. Check lines: 275, 272. |
General Analysis | core |
frame.cpp:294 | Medium | V572 | It is odd that the object which was created using 'new' operator is immediately cast to another type. |
General Analysis | core |
gdiimage.cpp:581 | Medium | V560 | A part of conditional expression is always true: !hicon. |
General Analysis | core |
graphics.cpp:1675 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_fontScaleRatio, m_state1, m_state2. |
General Analysis | core |
graphicsd2d.cpp:700 | Medium | V719 | The switch statement does not cover all values of the 'wxPenStyle' enum: wxPENSTYLE_FIRST_HATCH, wxPENSTYLE_LAST_HATCH. |
General Analysis | core |
headerctrl.cpp:636 | Medium | V658 | A value is being subtracted from the unsigned variable. This can result in an overflow. In such a case, the '<=' comparison operation can potentially behave unexpectedly. Consider inspecting the 'code <= (0U - 300U)' expression. |
General Analysis | core |
headerctrl.cpp:636 | Medium | V658 | A value is being subtracted from the unsigned variable. This can result in an overflow. In such a case, the '>' comparison operation can potentially behave unexpectedly. Consider inspecting the 'code > (0U - 399U)' expression. |
General Analysis | core |
listctrl.cpp:3459 | Medium | V547 | Expression 'lvItem.pszText' is always true. |
General Analysis | core |
listctrl.cpp:180 | Medium | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_pItem. |
General Analysis | core |
mdi.cpp:895 | Medium | V1004 | The 'parent' pointer was used unsafely after it was verified against nullptr. Check lines: 833, 895. |
General Analysis | core |
mdi.cpp:1511 | Medium | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_buf. |
General Analysis | core |
menu.cpp:1141 | Medium | V1004 | The 'menu' pointer was used unsafely after it was verified against nullptr. Check lines: 1135, 1141. |
General Analysis | core |
access.cpp:802 | Medium | V547 | Expression 'parent' is always true. |
General Analysis | core |
activex.cpp:1112 | Medium | V519 | The 'hret' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 1101, 1112. |
General Analysis | core |
activex.cpp:399 | Medium | V547 | Expression 'lprcPosRect' is always true. |
General Analysis | core |
activex.cpp:405 | Medium | V547 | Expression 'lprcClipRect' is always true. |
General Analysis | core |
dataobj.cpp:1353 | Medium | V547 | Expression 'pDrop->fWide' is always true. |
General Analysis | core |
dataobj.cpp:1036 | Medium | V665 | Possibly, the usage of '#pragma warning(default: X)' is incorrect in this context. The '#pragma warning(push/pop)' should be used instead. Check lines: 1003, 1036. |
General Analysis | core |
uuid.cpp:63 | Medium | V794 | The assignment operator should be protected from the case of 'this == &uuid'. |
General Analysis | core |
printdlg.cpp:814 | Medium | V547 | Expression 'pd->hDevNames' is always false. |
General Analysis | core |
printdlg.cpp:400 | Medium | V560 | A part of conditional expression is always true: !m_devMode. |
General Analysis | core |
printdlg.cpp:809 | Medium | V547 | Expression 'pd->hDevMode' is always false. |
General Analysis | core |
printwin.cpp:113 | Medium | V668 | There is no sense in testing the 'dc' pointer against null, as the memory was allocated using the 'new' operator. The exception will be generated in the case of memory allocation error. |
General Analysis | core |
printwin.cpp:111 | Medium | V668 | There is no sense in testing the 'dc' pointer against null, as the memory was allocated using the 'new' operator. The exception will be generated in the case of memory allocation error. |
General Analysis | core |
region.cpp:147 | Medium | V773 | The 'm_refData' pointer was not released in destructor. A memory leak is possible. |
General Analysis | core |
region.cpp:361 | Medium | V794 | The assignment operator should be protected from the case of 'this == &ri'. |
General Analysis | core |
settings.cpp:262 | Medium | V560 | A part of conditional expression is always true: index > 0. |
General Analysis | core |
taskbarbutton.cpp:936 | Medium | V1016 | The value 'THB_ICON | THB_FLAGS' is out of range of enum values. This causes unspecified or undefined behavior. |
General Analysis | core |
textctrl.cpp:1209 | Medium | V547 | Expression '!done' is always true. |
General Analysis | core |
tooltip.cpp:163 | Medium | V665 | Possibly, the usage of '#pragma warning(default: X)' is incorrect in this context. The '#pragma warning(push/pop)' should be used instead. Check lines: 88, 163. |
General Analysis | core |
toplevel.cpp:442 | Medium | V575 | The potential null pointer is passed into 'memset' function. Inspect the first argument. Check lines: 442, 439. |
General Analysis | core |
treectrl.cpp:529 | Medium | V665 | Possibly, the usage of '#pragma warning(default: X)' is incorrect in this context. The '#pragma warning(push/pop)' should be used instead. Check lines: 385, 529. |
General Analysis | core |
treectrl.cpp:1047 | Medium | V1016 | Expression 'which >= 0' is always true. |
General Analysis | core |
volume.cpp:595 | Medium | V1016 | Expression 'type >= 0' is always true. |
General Analysis | core |
aboutdlg.h:65 | Low | V524 | It is odd that the body of 'SetLicense' function is fully equivalent to the body of 'SetLicence' function. |
General Analysis | core |
any.h:919 | Low | V550 | An odd precise comparison. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
any.h:909 | Low | V550 | An odd precise comparison. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
any.h:635 | Low | V522 | Dereferencing of the null pointer 'value' might take place. The potential null pointer is passed into 'SetValue' function. Inspect the first argument. Check lines: 635, 1048. |
General Analysis | core |
any.h:1116 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
anybutton.h:85 | Low | V524 | It is odd that the body of 'GetBitmapLabel' function is fully equivalent to the body of 'GetBitmap' function. |
General Analysis | core |
arrstr.h:475 | Low | V690 | The '=' operator is declared as private in the 'wxArrayStringsAdapter' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
clrpicker.h:163 | Low | V690 | The '=' operator is declared as private in the 'wxColourPickerEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
cmdline.h:127 | Low | V690 | The '=' operator is declared as private in the 'wxCmdLineArgs' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
collpane.h:74 | Low | V690 | The '=' operator is declared as private in the 'wxCollapsiblePaneEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
compiler.h:105 | Low | V1003 | The macro 'wxVISUALC_VERSION' is a dangerous expression. The parameter 'major' must be surrounded by parentheses. |
General Analysis | core |
dataobj.h:257 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
dateevt.h:22 | Low | V690 | The '=' operator is declared as private in the 'wxDateEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
datetime.h:1861 | Low | V659 | Declarations of functions with 'wxDateTime::Add' name differ in the 'const' keyword only, but the bodies of these functions have different composition. This is suspicious and can possibly be an error. Check lines: 1854, 1861. |
General Analysis | core |
datetime.h:1882 | Low | V659 | Declarations of functions with 'wxDateTime::Subtract' name differ in the 'const' keyword only, but the bodies of these functions have different composition. This is suspicious and can possibly be an error. Check lines: 1875, 1882. |
General Analysis | core |
datetime.h:1918 | Low | V659 | Declarations of functions with 'wxDateTime::Subtract' name differ in the 'const' keyword only, but the bodies of these functions have different composition. This is suspicious and can possibly be an error. Check lines: 1913, 1918. |
General Analysis | core |
datetime.h:2044 | Low | V524 | It is odd that the body of 'IsEqualTo' function is fully equivalent to the body of '==' function. |
General Analysis | core |
datetime.h:2008 | Low | V659 | Declarations of functions with 'wxTimeSpan::Add' name differ in the 'const' keyword only, but the bodies of these functions have different composition. This is suspicious and can possibly be an error. Check lines: 2003, 2008. |
General Analysis | core |
datetime.h:2020 | Low | V659 | Declarations of functions with 'wxTimeSpan::Subtract' name differ in the 'const' keyword only, but the bodies of these functions have different composition. This is suspicious and can possibly be an error. Check lines: 2015, 2020. |
General Analysis | core |
datetime.h:2127 | Low | V659 | Declarations of functions with 'wxDateSpan::Subtract' name differ in the 'const' keyword only, but the bodies of these functions have different composition. This is suspicious and can possibly be an error. Check lines: 2122, 2127. |
General Analysis | core |
datetime.h:2095 | Low | V659 | Declarations of functions with 'wxDateSpan::Multiply' name differ in the 'const' keyword only, but the bodies of these functions have different composition. This is suspicious and can possibly be an error. Check lines: 2085, 2095. |
General Analysis | core |
datetime.h:2034 | Low | V659 | Declarations of functions with 'wxTimeSpan::Multiply' name differ in the 'const' keyword only, but the bodies of these functions have different composition. This is suspicious and can possibly be an error. Check lines: 2027, 2034. |
General Analysis | core |
datetime.h:2078 | Low | V659 | Declarations of functions with 'wxDateSpan::Add' name differ in the 'const' keyword only, but the bodies of these functions have different composition. This is suspicious and can possibly be an error. Check lines: 2073, 2078. |
General Analysis | core |
debug.h:460 | Low | V1003 | The macro 'wxASSERT_MIN_BITSIZE' is a dangerous expression. The parameter 'size' must be surrounded by parentheses. |
General Analysis | core |
dialog.h:366 | Low | V690 | The '=' operator is declared as private in the 'wxWindowModalDialogEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
dvrenderers.h:36 | Low | V690 | The 'wxDataViewIconText' class implements a copy constructor, but lacks the '=' operator. It is dangerous to use such a class. |
General Analysis | core |
dynarray.h:405 | Low | V524 | It is odd that the body of 'Clear' function is fully equivalent to the body of 'Empty' function. |
General Analysis | core |
dynarray.h:314 | Low | V524 | It is odd that the body of 'size' function is fully equivalent to the body of 'GetCount' function. |
General Analysis | core |
dynarray.h:202 | Low | V659 | Declarations of functions with 'Sort' name differ in the 'const' keyword only, but the bodies of these functions have different composition. This is suspicious and can possibly be an error. Check lines: 196, 202. |
General Analysis | core |
event.h:3029 | Low | V690 | The '=' operator is declared as private in the 'wxDisplayChangedEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
event.h:1474 | Low | V690 | The 'wxAsyncMethodCallEventFunctor' class implements a copy constructor, but lacks the '=' operator. It is dangerous to use such a class. |
General Analysis | core |
event.h:2384 | Low | V690 | The '=' operator is declared as private in the 'wxNcPaintEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
event.h:1176 | Low | V690 | The '=' operator is declared as private in the 'wxEventBasicPayloadMixin' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
event.h:2691 | Low | V690 | The '=' operator is declared as private in the 'wxMaximizeEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
event.h:2961 | Low | V690 | The '=' operator is declared as private in the 'wxSysColourChangedEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
event.h:2460 | Low | V690 | The '=' operator is declared as private in the 'wxChildFocusEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
event.h:3178 | Low | V690 | The '=' operator is declared as private in the 'wxWindowCreateEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
event.h:3191 | Low | V690 | The '=' operator is declared as private in the 'wxWindowDestroyEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
event.h:3401 | Low | V690 | The '=' operator is declared as private in the 'wxEventTableEntry' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
event.h:3422 | Low | V690 | The '=' operator is declared as private in the 'wxDynamicEventTableEntry' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
event.h:2523 | Low | V690 | The '=' operator is declared as private in the 'wxInitDialogEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
event.h:1636 | Low | V690 | The '=' operator is declared as private in the 'wxScrollEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
filectrl.h:90 | Low | V690 | The '=' operator is declared as private in the 'wxFileCtrlEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
filefn.h:433 | Low | V524 | It is odd that the body of 'wxLstat' function is fully equivalent to the body of 'wxStat' function. |
General Analysis | core |
filename.h:148 | Low | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_relative, m_hasExt. |
General Analysis | core |
filepicker.h:38 | Low | V690 | The '=' operator is declared as private in the 'wxFileDirPickerEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
fontpicker.h:201 | Low | V690 | The '=' operator is declared as private in the 'wxFontPickerEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
gdicmn.h:713 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
gdicmn.h:802 | Low | V659 | Declarations of functions with 'Deflate' name differ in the 'const' keyword only, but the bodies of these functions have different composition. This is suspicious and can possibly be an error. Check lines: 799, 802. |
General Analysis | core |
animate.h:20 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
filectrlg.h:189 | Low | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_style, m_choice, m_text, m_list, m_check, m_static, ... |
General Analysis | core |
grid.h:2420 | Low | V690 | The '=' operator is declared as private in the 'wxGridEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
grid.h:2540 | Low | V690 | The '=' operator is declared as private in the 'wxGridRangeSelectEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
grid.h:2483 | Low | V690 | The '=' operator is declared as private in the 'wxGridSizeEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
grid.h:612 | Low | V690 | The 'wxGridCellCoords' class implements the '=' operator, but lacks a copy constructor. It is dangerous to use such a class. |
General Analysis | core |
laywin.h:60 | Low | V690 | The '=' operator is declared as private in the 'wxQueryLayoutInfoEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
laywin.h:115 | Low | V690 | The '=' operator is declared as private in the 'wxCalculateLayoutEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
geometry.h:680 | Low | V524 | It is odd that the body of 'GetLeftTop' function is fully equivalent to the body of 'GetPosition' function. |
General Analysis | core |
geometry.h:158 | Low | V636 | The '(pt.m_x - m_x) * (pt.m_x - m_x)' expression was implicitly cast from 'int' type to 'double' type. Consider utilizing an explicit type cast to avoid overflow. An example: double A = (double)(X) * Y;. |
General Analysis | core |
geometry.h:518 | Low | V690 | The 'wxRect2DDouble' class implements the '=' operator, but lacks a copy constructor. It is dangerous to use such a class. |
General Analysis | core |
geometry.h:553 | Low | V524 | It is odd that the body of 'GetLeftTop' function is fully equivalent to the body of 'GetPosition' function. |
General Analysis | core |
hyperlink.h:99 | Low | V690 | The '=' operator is declared as private in the 'wxHyperlinkEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
imaggif.h:34 | Low | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_crntShiftDWord, m_pixelCount, m_EOFCode, m_clearCode, m_runningCode, m_runningBits, ... |
General Analysis | core |
kbdstate.h:94 | Low | V524 | It is odd that the body of 'SetRawControlDown' function is fully equivalent to the body of 'SetControlDown' function. |
General Analysis | core |
list.h:1216 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
list.h:1171 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
listbase.h:469 | Low | V524 | It is odd that the body of 'GetText' function is fully equivalent to the body of 'GetLabel' function. |
General Analysis | core |
longlong.h:170 | Low | V659 | Declarations of functions with 'Abs' name differ in the 'const' keyword only, but the bodies of these functions have different composition. This is suspicious and can possibly be an error. Check lines: 169, 170. |
General Analysis | core |
math.h:130 | Low | V550 | An odd precise comparison: x == y. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
menu.h:36 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
menu.h:37 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
module.h:21 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
gdiimage.h:26 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
dataform.h:18 | Low | V690 | The 'wxDataFormat' class implements the '=' operator, but lacks a copy constructor. It is dangerous to use such a class. |
General Analysis | core |
object.h:300 | Low | V522 | There might be dereferencing of a potential null pointer 'm_ptr'. |
General Analysis | core |
odcombo.h:76 | Low | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_value, m_itemHover, m_itemHeight, m_clientDataItemsType, m_widestWidth, m_widestItem, ... |
General Analysis | core |
power.h:48 | Low | V690 | The '=' operator is declared as private in the 'wxPowerEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
flagscheck.h:108 | Low | V1003 | The macro 'wxADD_FLAG' is a dangerous expression. The parameters 'f', 'others' must be surrounded by parentheses. |
General Analysis | core |
process.h:158 | Low | V690 | The '=' operator is declared as private in the 'wxProcessEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
scopedptr.h:80 | Low | V522 | There might be dereferencing of a potential null pointer 'm_ptr'. |
General Analysis | core |
scopeguard.h:163 | Low | V690 | The 'wxScopeGuardImpl2' class implements the '=' operator, but lacks a copy constructor. It is dangerous to use such a class. |
General Analysis | core |
scopeguard.h:100 | Low | V690 | The 'wxScopeGuardImpl0' class implements the '=' operator, but lacks a copy constructor. It is dangerous to use such a class. |
General Analysis | core |
scopeguard.h:131 | Low | V690 | The 'wxScopeGuardImpl1' class implements the '=' operator, but lacks a copy constructor. It is dangerous to use such a class. |
General Analysis | core |
scopeguard.h:196 | Low | V690 | The 'wxScopeGuardImpl3' class implements the '=' operator, but lacks a copy constructor. It is dangerous to use such a class. |
General Analysis | core |
sharedptr.h:86 | Low | V522 | There might be dereferencing of a potential null pointer 'm_ref'. |
General Analysis | core |
sharedptr.h:79 | Low | V522 | There might be dereferencing of a potential null pointer 'm_ref'. |
General Analysis | core |
sharedptr.h:80 | Low | V522 | There might be dereferencing of a potential null pointer 'm_ref->m_ptr'. |
General Analysis | core |
sizer.h:500 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
sizer.h:347 | Low | V524 | It is odd that the body of 'SetInitSize' function is fully equivalent to the body of 'SetMinSize' function. |
General Analysis | core |
stream.h:477 | Low | V524 | It is odd that the body of 'GetLastAccess' function is fully equivalent to the body of 'GetBufferSize' function. |
General Analysis | core |
string.h:941 | Low | V690 | The 'const_iterator' class implements a copy constructor, but lacks the '=' operator. It is dangerous to use such a class. |
General Analysis | core |
string.h:908 | Low | V690 | The 'iterator' class implements a copy constructor, but lacks the '=' operator. It is dangerous to use such a class. |
General Analysis | core |
string.h:1005 | Low | V690 | The 'reverse_iterator_impl' class implements a copy constructor, but lacks the '=' operator. It is dangerous to use such a class. |
General Analysis | core |
string.h:1947 | Low | V524 | It is odd that the body of 'Append' function is fully equivalent to the body of '<<' function. |
General Analysis | core |
string.h:281 | Low | V553 | The length of 'wxString' class's declaration is more than 2000 lines long. You should consider refactoring the code. |
General Analysis | core |
string.h:1949 | Low | V524 | It is odd that the body of 'Append' function is fully equivalent to the body of '<<' function. |
General Analysis | core |
string.h:2274 | Low | V524 | It is odd that the body of 'sprintf' function is fully equivalent to the body of 'Printf' function. |
General Analysis | core |
tbarbase.h:272 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
textctrl.h:318 | Low | V524 | It is odd that the body of 'SetFontPointSize' function is fully equivalent to the body of 'SetFontSize' function. |
General Analysis | core |
tokenzr.h:106 | Low | V730 | It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: m_hasMoreTokens, m_delimsLen, m_mode, m_lastDelim. |
General Analysis | core |
translation.h:64 | Low | V522 | There might be dereferencing of a potential null pointer 'm_ptr'. |
General Analysis | core |
treelist.h:457 | Low | V690 | The '=' operator is declared as private in the 'wxTreeListEvent' class, but the default copy constructor will still be generated by compiler. It is dangerous to use such a class. |
General Analysis | core |
unichar.h:29 | Low | V690 | The 'wxUniChar' class implements the '=' operator, but lacks a copy constructor. It is dangerous to use such a class. |
General Analysis | core |
unichar.h:221 | Low | V690 | The 'wxUniCharRef' class implements the '=' operator, but lacks a copy constructor. It is dangerous to use such a class. |
General Analysis | core |
variant.h:101 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
vector.h:439 | Low | V550 | Instantiation of wxVector < REAL >: An odd precise comparison: vb.m_values[i] != m_values[i]. It's probably better to use a comparison with defined precision: fabs(A - B) > Epsilon. |
General Analysis | core |
weakref.h:156 | Low | V522 | There might be dereferencing of a potential null pointer 'm_pobj'. |
General Analysis | core |
window.h:144 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
window.h:729 | Low | V524 | It is odd that the body of 'AcceptsFocusFromKeyboard' function is fully equivalent to the body of 'AcceptsFocusRecursively' function. |
General Analysis | core |
affinematrix2d.cpp:99 | Low | V550 | An odd precise comparison: m_11 == mat.m_11. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
affinematrix2d.cpp:181 | Low | V550 | An odd precise comparison: m_12 == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
affinematrix2d.cpp:182 | Low | V550 | An odd precise comparison: m_21 == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
affinematrix2d.cpp:182 | Low | V550 | An odd precise comparison: m_22 == 1. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
affinematrix2d.cpp:183 | Low | V550 | An odd precise comparison: m_tx == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
affinematrix2d.cpp:183 | Low | V550 | An odd precise comparison: m_ty == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
affinematrix2d.cpp:101 | Low | V550 | An odd precise comparison: m_tx == tr.m_x. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
affinematrix2d.cpp:100 | Low | V550 | An odd precise comparison: m_22 == mat.m_22. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
affinematrix2d.cpp:100 | Low | V550 | An odd precise comparison: m_21 == mat.m_21. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
affinematrix2d.cpp:99 | Low | V550 | An odd precise comparison: m_12 == mat.m_12. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
affinematrix2d.cpp:77 | Low | V550 | An odd precise comparison. It's probably better to use a comparison with defined precision: fabs(det) < Epsilon. |
General Analysis | core |
affinematrix2d.cpp:101 | Low | V550 | An odd precise comparison: m_ty == tr.m_y. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
affinematrix2d.cpp:181 | Low | V550 | An odd precise comparison: m_11 == 1. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
artprov.cpp:37 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
bookctrl.cpp:282 | Low | V537 | Consider reviewing the correctness of 'x' item's usage. |
General Analysis | core |
bookctrl.cpp:286 | Low | V537 | Consider reviewing the correctness of 'y' item's usage. |
General Analysis | core |
clrpickercmn.cpp:98 | Low | V522 | There might be dereferencing of a potential null pointer 'm_text'. |
General Analysis | core |
cmndata.cpp:386 | Low | V522 | There might be dereferencing of a potential null pointer 'wxThePrintPaperDatabase'. |
General Analysis | core |
cmndata.cpp:373 | Low | V522 | There might be dereferencing of a potential null pointer 'wxThePrintPaperDatabase'. |
General Analysis | core |
datavcmn.cpp:1620 | Low | V524 | It is odd that the body of 'PrependColumn' function is fully equivalent to the body of 'AppendColumn' function. |
General Analysis | core |
dcbase.cpp:787 | Low | V1003 | The macro 'half' is a dangerous expression. The parameters 'z1', 'z2' must be surrounded by parentheses. |
General Analysis | core |
dcgraph.cpp:1112 | Low | V550 | An odd precise comparison: angle == 0.0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
dcgraph.cpp:685 | Low | V550 | An odd precise comparison: ea == sa. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
dcgraph.cpp:636 | Low | V550 | An odd precise comparison: radius == 0.0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
dcgraph.cpp:297 | Low | V550 | An odd precise comparison: h != 0.. It's probably better to use a comparison with defined precision: fabs(A - B) > Epsilon. |
General Analysis | core |
dcgraph.cpp:297 | Low | V550 | An odd precise comparison: w != 0.. It's probably better to use a comparison with defined precision: fabs(A - B) > Epsilon. |
General Analysis | core |
dcsvg.cpp:142 | Low | V525 | The code contains the collection of similar blocks. Check items '5', '8', '8' in lines 142, 145, 148. |
General Analysis | core |
dcsvg.cpp:852 | Low | V550 | An odd precise comparison: angle == 360. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
dobjcmn.cpp:506 | Low | V769 | The 'ptr' pointer in the 'ptr + 10' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. |
General Analysis | core |
dobjcmn.cpp:518 | Low | V769 | The 'strstr(buffer, "<!--EndFrag")' pointer in the 'strstr(buffer, "<!--EndFrag") - buffer' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. |
General Analysis | core |
dobjcmn.cpp:514 | Low | V769 | The 'strstr(buffer, "<!--StartFrag")' pointer in the 'strstr(buffer, "<!--StartFrag") - buffer' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. |
General Analysis | core |
dobjcmn.cpp:506 | Low | V769 | The 'strstr(buffer, "<html>")' pointer in the 'strstr(buffer, "<html>") - buffer' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. |
General Analysis | core |
docview.cpp:1324 | Low | V524 | It is odd that the body of 'OnUpdateFileNew' function is fully equivalent to the body of 'OnUpdateFileOpen' function. |
General Analysis | core |
event.cpp:491 | Low | V522 | There might be dereferencing of a potential null pointer 'win'. |
General Analysis | core |
filectrlcmn.cpp:93 | Low | V522 | There might be dereferencing of a potential null pointer. |
General Analysis | core |
fontpickercmn.cpp:133 | Low | V522 | There might be dereferencing of a potential null pointer 'm_text'. |
General Analysis | core |
gbsizer.cpp:227 | Low | V522 | There might be dereferencing of a potential null pointer 'item->GetWindow()'. |
General Analysis | core |
gdicmn.cpp:651 | Low | V522 | There might be dereferencing of a potential null pointer 'GetColour(COLOUR_BLUE)'. |
General Analysis | core |
gdicmn.cpp:645 | Low | V522 | There might be dereferencing of a potential null pointer 'GetColour(COLOUR_BLACK)'. |
General Analysis | core |
gdicmn.cpp:672 | Low | V522 | There might be dereferencing of a potential null pointer 'GetColour(COLOUR_RED)'. |
General Analysis | core |
gdicmn.cpp:678 | Low | V522 | There might be dereferencing of a potential null pointer 'GetColour(COLOUR_WHITE)'. |
General Analysis | core |
gdicmn.cpp:666 | Low | V522 | There might be dereferencing of a potential null pointer 'GetColour(COLOUR_LIGHTGREY)'. |
General Analysis | core |
gdicmn.cpp:660 | Low | V522 | There might be dereferencing of a potential null pointer 'GetColour(COLOUR_YELLOW)'. |
General Analysis | core |
gdicmn.cpp:657 | Low | V522 | There might be dereferencing of a potential null pointer 'GetColour(COLOUR_GREEN)'. |
General Analysis | core |
gdicmn.cpp:654 | Low | V522 | There might be dereferencing of a potential null pointer 'GetColour(COLOUR_CYAN)'. |
General Analysis | core |
gifdecod.cpp:71 | Low | V522 | There might be dereferencing of a potential null pointer 'm_ptr'. |
General Analysis | core |
graphcmn.cpp:548 | Low | V550 | An odd precise comparison: stop.GetPosition() == 1.. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
graphcmn.cpp:487 | Low | V550 | An odd precise comparison: r == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
graphcmn.cpp:487 | Low | V550 | An odd precise comparison: alpha == 180. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
graphcmn.cpp:487 | Low | V550 | An odd precise comparison: alpha == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
graphcmn.cpp:486 | Low | V550 | An odd precise comparison: v2Length == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
graphcmn.cpp:486 | Low | V550 | An odd precise comparison: v1Length == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
graphcmn.cpp:442 | Low | V550 | An odd precise comparison: radius == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
graphcmn.cpp:835 | Low | V522 | There might be dereferencing of a potential null pointer 'GetRenderer()'. |
General Analysis | core |
graphcmn.cpp:935 | Low | V522 | There might be dereferencing of a potential null pointer 'GetRenderer()'. |
General Analysis | core |
graphcmn.cpp:840 | Low | V522 | There might be dereferencing of a potential null pointer 'GetRenderer()'. |
General Analysis | core |
graphcmn.cpp:874 | Low | V522 | There might be dereferencing of a potential null pointer 'GetRenderer()'. |
General Analysis | core |
graphcmn.cpp:879 | Low | V522 | There might be dereferencing of a potential null pointer 'GetRenderer()'. |
General Analysis | core |
graphcmn.cpp:888 | Low | V522 | There might be dereferencing of a potential null pointer 'GetRenderer()'. |
General Analysis | core |
graphcmn.cpp:902 | Low | V522 | There might be dereferencing of a potential null pointer 'GetRenderer()'. |
General Analysis | core |
graphcmn.cpp:955 | Low | V522 | There might be dereferencing of a potential null pointer 'GetRenderer()'. |
General Analysis | core |
graphcmn.cpp:949 | Low | V522 | There might be dereferencing of a potential null pointer 'GetRenderer()'. |
General Analysis | core |
graphcmn.cpp:911 | Low | V522 | There might be dereferencing of a potential null pointer 'GetRenderer()'. |
General Analysis | core |
graphcmn.cpp:944 | Low | V522 | There might be dereferencing of a potential null pointer 'GetRenderer()'. |
General Analysis | core |
graphcmn.cpp:961 | Low | V522 | There might be dereferencing of a potential null pointer 'GetRenderer()'. |
General Analysis | core |
graphcmn.cpp:925 | Low | V522 | There might be dereferencing of a potential null pointer 'GetRenderer()'. |
General Analysis | core |
imagbmp.cpp:387 | Low | V522 | There might be dereferencing of a potential null pointer 'alpha'. |
General Analysis | core |
imagbmp.cpp:666 | Low | V547 | Expression 'bpp == 32' is always true. |
General Analysis | core |
imagbmp.cpp:468 | Low | V547 | Expression 'format == wxBMP_1BPP_BW' is always true. |
General Analysis | core |
imagbmp.cpp:1297 | Low | V1008 | Consider inspecting the 'for' operator. No more than one iteration of the loop will be performed. |
General Analysis | core |
imagbmp.cpp:600 | Low | V522 | There might be dereferencing of a potential null pointer 'cmap'. |
General Analysis | core |
image.cpp:669 | Low | V636 | The expression was implicitly cast from 'int' type to 'double' type. Consider utilizing an explicit type cast to avoid overflow. An example: double A = (double)(X) * Y;. |
General Analysis | core |
image.cpp:671 | Low | V636 | The expression was implicitly cast from 'int' type to 'double' type. Consider utilizing an explicit type cast to avoid overflow. An example: double A = (double)(X) * Y;. |
General Analysis | core |
image.cpp:688 | Low | V550 | An odd precise comparison. It's probably better to use a comparison with defined precision: fabs(sum_a) > Epsilon. |
General Analysis | core |
image.cpp:1034 | Low | V550 | An odd precise comparison. It's probably better to use a comparison with defined precision: fabs(sum_a) > Epsilon. |
General Analysis | core |
image.cpp:670 | Low | V636 | The expression was implicitly cast from 'int' type to 'double' type. Consider utilizing an explicit type cast to avoid overflow. An example: double A = (double)(X) * Y;. |
General Analysis | core |
image.cpp:2355 | Low | V522 | There might be dereferencing of a potential null pointer 'imgdata'. |
General Analysis | core |
image.cpp:3684 | Low | V522 | There might be dereferencing of a potential null pointer 'alpha'. |
General Analysis | core |
image.cpp:3285 | Low | V601 | The 'false' value is implicitly cast to the integer type. |
General Analysis | core |
image.cpp:1046 | Low | V769 | The 'dst_alpha' pointer in the 'dst_alpha ++' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. |
General Analysis | core |
image.cpp:2743 | Low | V601 | The 'false' value is implicitly cast to the integer type. |
General Analysis | core |
image.cpp:546 | Low | V522 | There might be dereferencing of a potential null pointer 'src_alpha_line'. |
General Analysis | core |
image.cpp:552 | Low | V522 | There might be dereferencing of a potential null pointer 'src_alpha_pixel'. |
General Analysis | core |
image.cpp:3489 | Low | V769 | The 'p' pointer in the 'p ++' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. |
General Analysis | core |
image.cpp:2082 | Low | V522 | There might be dereferencing of a potential null pointer 'data'. |
General Analysis | core |
image.cpp:2298 | Low | V522 | There might be dereferencing of a potential null pointer 'maskdata'. |
General Analysis | core |
image.cpp:3548 | Low | V769 | The 'data[i - 1]' pointer in the 'data[i - 1] + (3 * w)' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. |
General Analysis | core |
image.cpp:2300 | Low | V522 | There might be dereferencing of a potential null pointer 'imgdata'. |
General Analysis | core |
image.cpp:3461 | Low | V769 | The 'p' pointer in the 'p ++' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. |
General Analysis | core |
image.cpp:847 | Low | V769 | The 'dst_alpha' pointer in the 'dst_alpha ++' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. |
General Analysis | core |
image.cpp:700 | Low | V769 | The 'dst_alpha' pointer in the 'dst_alpha ++' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. |
General Analysis | core |
image.cpp:552 | Low | V769 | The 'target_alpha' pointer in the 'target_alpha ++' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. |
General Analysis | core |
image.cpp:414 | Low | V769 | The 'target_alpha' pointer in the 'target_alpha ++' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. |
General Analysis | core |
image.cpp:3679 | Low | V769 | The 'dst' pointer in the 'dst ++' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. |
General Analysis | core |
image.cpp:3684 | Low | V769 | The 'alpha_dst' pointer in the 'alpha_dst ++' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. |
General Analysis | core |
image.cpp:3296 | Low | V601 | The 'false' value is implicitly cast to the integer type. |
General Analysis | core |
imaggif.cpp:57 | Low | V1003 | The macro 'HT_PUT_KEY' is a dangerous expression. The parameter 'l' must be surrounded by parentheses. |
General Analysis | core |
imaggif.cpp:56 | Low | V1003 | The macro 'HT_GET_CODE' is a dangerous expression. The parameter 'l' must be surrounded by parentheses. |
General Analysis | core |
imaggif.cpp:55 | Low | V1003 | The macro 'HT_GET_KEY' is a dangerous expression. The parameter 'l' must be surrounded by parentheses. |
General Analysis | core |
imaggif.cpp:58 | Low | V1003 | The macro 'HT_PUT_CODE' is a dangerous expression. The parameter 'l' must be surrounded by parentheses. |
General Analysis | core |
imagpng.cpp:845 | Low | V769 | The 'pAlpha' pointer in the 'pAlpha ++' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. |
General Analysis | core |
imagtga.cpp:341 | Low | V769 | The 'alpha' pointer in the 'alpha ++' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. |
General Analysis | core |
matrix.cpp:102 | Low | V659 | Declarations of functions with 'wxTransformMatrix::operator ()' name differ in the 'const' keyword only, but the bodies of these functions have different composition. This is suspicious and can possibly be an error. Check lines: 94, 102. |
General Analysis | core |
paper.cpp:76 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
pickerbase.cpp:121 | Low | V537 | Consider reviewing the correctness of 'y' item's usage. |
General Analysis | core |
prntbase.cpp:516 | Low | V524 | It is odd that the body of 'GetPageSetupData' function is fully equivalent to the body of 'GetPageSetupDialogData' function. |
General Analysis | core |
quantize.cpp:905 | Low | V525 | The code contains the collection of similar blocks. Check items '1 << (8 - 5)', '1 << (8 - 6)', '1 << (8 - 5)' in lines 905, 906, 907. |
General Analysis | core |
quantize.cpp:1293 | Low | V575 | The potential null pointer is passed into 'memset' function. Inspect the first argument. |
General Analysis | core |
quantize.cpp:754 | Low | V525 | The code contains the collection of similar blocks. Check items '(1 << ((8 - 5) + (5 - 3))) - (1 << (8 - 5))', '(1 << ((8 - 6) + (6 - 3))) - (1 << (8 - 6))', '(1 << ((8 - 5) + (5 - 3))) - (1 << (8 - 5))' in lines 754, 756, 758. |
General Analysis | core |
sizer.cpp:410 | Low | V522 | There might be dereferencing of a potential null pointer 'GetWindow()'. |
General Analysis | core |
sizer.cpp:676 | Low | V522 | There might be dereferencing of a potential null pointer 'item->GetWindow()'. |
General Analysis | core |
sizer.cpp:1708 | Low | V1026 | The 'total' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
sizer.cpp:826 | Low | V522 | There might be dereferencing of a potential null pointer 'item->GetSizer()'. |
General Analysis | core |
sizer.cpp:876 | Low | V522 | There might be dereferencing of a potential null pointer 'item->GetWindow()'. |
General Analysis | core |
sizer.cpp:892 | Low | V522 | There might be dereferencing of a potential null pointer 'item->GetWindow()'. |
General Analysis | core |
sizer.cpp:1123 | Low | V522 | There might be dereferencing of a potential null pointer 'item->GetSizer()'. |
General Analysis | core |
sizer.cpp:1159 | Low | V522 | There might be dereferencing of a potential null pointer 'item->GetSizer()'. |
General Analysis | core |
sizer.cpp:1185 | Low | V522 | There might be dereferencing of a potential null pointer 'item->GetSizer()'. |
General Analysis | core |
sizer.cpp:404 | Low | V522 | There might be dereferencing of a potential null pointer 'GetSizer()'. |
General Analysis | core |
sizer.cpp:1211 | Low | V522 | There might be dereferencing of a potential null pointer 'item->GetSizer()'. |
General Analysis | core |
sizer.cpp:1247 | Low | V522 | There might be dereferencing of a potential null pointer 'item->GetSizer()'. |
General Analysis | core |
sizer.cpp:679 | Low | V522 | There might be dereferencing of a potential null pointer 'item->GetSizer()'. |
General Analysis | core |
sizer.cpp:853 | Low | V522 | There might be dereferencing of a potential null pointer 'item->GetSizer()'. |
General Analysis | core |
uiactioncmn.cpp:77 | Low | V760 | Two identical blocks of text were found. The second block begins from line 79. |
General Analysis | core |
utilscmn.cpp:1135 | Low | V547 | Expression 'hasValidScheme' is always true. |
General Analysis | core |
valnum.cpp:197 | Low | V522 | There might be dereferencing of a potential null pointer 'text'. |
General Analysis | core |
wincmn.cpp:1463 | Low | V522 | There might be dereferencing of a potential null pointer 'pLast'. |
General Analysis | core |
wrapsizer.cpp:578 | Low | V522 | There might be dereferencing of a potential null pointer 'sizer'. |
General Analysis | core |
collpaneg.cpp:222 | Low | V522 | There might be dereferencing of a potential null pointer 'm_pButton'. |
General Analysis | core |
datavgen.cpp:6502 | Low | V522 | There might be dereferencing of a potential null pointer 'dvWnd'. |
General Analysis | core |
datavgen.cpp:609 | Low | V522 | There might be dereferencing of a potential null pointer 'm_branchData'. |
General Analysis | core |
datavgen.cpp:2541 | Low | V522 | There might be dereferencing of a potential null pointer 'node'. |
General Analysis | core |
datavgen.cpp:3831 | Low | V522 | There might be dereferencing of a potential null pointer 'node'. |
General Analysis | core |
datavgen.cpp:6471 | Low | V522 | There might be dereferencing of a potential null pointer 'dvWnd'. |
General Analysis | core |
datavgen.cpp:4659 | Low | V522 | There might be dereferencing of a potential null pointer 'node'. |
General Analysis | core |
datavgen.cpp:6294 | Low | V522 | There might be dereferencing of a potential null pointer 'dvWnd'. |
General Analysis | core |
datavgen.cpp:6518 | Low | V522 | There might be dereferencing of a potential null pointer 'dvWnd'. |
General Analysis | core |
datavgen.cpp:6363 | Low | V522 | There might be dereferencing of a potential null pointer 'dvWnd'. |
General Analysis | core |
datavgen.cpp:4800 | Low | V522 | There might be dereferencing of a potential null pointer 'node'. |
General Analysis | core |
datavgen.cpp:5483 | Low | V522 | There might be dereferencing of a potential null pointer 'node'. |
General Analysis | core |
datavgen.cpp:6094 | Low | V522 | There might be dereferencing of a potential null pointer 'dvWnd'. |
General Analysis | core |
datavgen.cpp:6124 | Low | V522 | There might be dereferencing of a potential null pointer 'dvWnd'. |
General Analysis | core |
datavgen.cpp:6268 | Low | V522 | There might be dereferencing of a potential null pointer 'dvWnd'. |
General Analysis | core |
datavgen.cpp:6445 | Low | V522 | There might be dereferencing of a potential null pointer 'dvWnd'. |
General Analysis | core |
datavgen.cpp:569 | Low | V547 | The 'A = !m_branchData->open' expression is equivalent to the 'A = true' expression. |
General Analysis | core |
datavgen.cpp:6331 | Low | V522 | There might be dereferencing of a potential null pointer 'dvWnd'. |
General Analysis | core |
datavgen.cpp:6743 | Low | V522 | There might be dereferencing of a potential null pointer 'dvWnd'. |
General Analysis | core |
datavgen.cpp:6605 | Low | V522 | There might be dereferencing of a potential null pointer 'dvWnd'. |
General Analysis | core |
datavgen.cpp:6577 | Low | V522 | There might be dereferencing of a potential null pointer 'dvWnd'. |
General Analysis | core |
filectrlg.cpp:44 | Low | V1003 | The macro 'IsTopMostDir' is a dangerous expression. The parameter 'dir' must be surrounded by parentheses. |
General Analysis | core |
grid.cpp:2639 | Low | V522 | There might be dereferencing of a potential null pointer 'editor'. |
General Analysis | core |
grid.cpp:6091 | Low | V1026 | The variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
grid.cpp:6127 | Low | V1026 | The 'y' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
grid.cpp:6132 | Low | V1026 | The 'x' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
grid.cpp:6188 | Low | V1026 | The 'h' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
grid.cpp:6194 | Low | V1026 | The 'h' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
grid.cpp:6272 | Low | V547 | The 'A = enable' expression is equivalent to the 'A = true' expression. |
General Analysis | core |
grid.cpp:5593 | Low | V524 | It is odd that the body of 'GetColGridLinePen' function is fully equivalent to the body of 'GetRowGridLinePen' function. |
General Analysis | core |
grid.cpp:8059 | Low | V522 | There might be dereferencing of a potential null pointer 'attr'. |
General Analysis | core |
grid.cpp:8049 | Low | V522 | There might be dereferencing of a potential null pointer 'attr'. |
General Analysis | core |
grid.cpp:8039 | Low | V522 | There might be dereferencing of a potential null pointer 'attr'. |
General Analysis | core |
grid.cpp:6284 | Low | V547 | The 'A = enable' expression is equivalent to the 'A = false' expression. |
General Analysis | core |
grid.cpp:8006 | Low | V522 | There might be dereferencing of a potential null pointer 'attr_stub'. |
General Analysis | core |
grid.cpp:4168 | Low | V522 | There might be dereferencing of a potential null pointer 'editor'. |
General Analysis | core |
grid.cpp:5059 | Low | V522 | There might be dereferencing of a potential null pointer 'editor'. |
General Analysis | core |
grid.cpp:5524 | Low | V522 | There might be dereferencing of a potential null pointer 'editor'. |
General Analysis | core |
grid.cpp:5532 | Low | V522 | There might be dereferencing of a potential null pointer 'renderer'. |
General Analysis | core |
grid.cpp:6396 | Low | V522 | There might be dereferencing of a potential null pointer 'editor'. |
General Analysis | core |
grid.cpp:6478 | Low | V522 | There might be dereferencing of a potential null pointer 'editor'. |
General Analysis | core |
grid.cpp:6518 | Low | V522 | There might be dereferencing of a potential null pointer 'editor'. |
General Analysis | core |
grid.cpp:7930 | Low | V522 | There might be dereferencing of a potential null pointer 'attr'. |
General Analysis | core |
grid.cpp:8025 | Low | V522 | There might be dereferencing of a potential null pointer 'attr_stub'. |
General Analysis | core |
grid.cpp:5568 | Low | V537 | Consider reviewing the correctness of 'penWidth' item's usage. |
General Analysis | core |
grid.cpp:7940 | Low | V522 | There might be dereferencing of a potential null pointer 'attr'. |
General Analysis | core |
grid.cpp:7960 | Low | V522 | There might be dereferencing of a potential null pointer 'attr'. |
General Analysis | core |
grid.cpp:7970 | Low | V522 | There might be dereferencing of a potential null pointer 'attr'. |
General Analysis | core |
grid.cpp:7982 | Low | V522 | There might be dereferencing of a potential null pointer 'attr'. |
General Analysis | core |
grid.cpp:7950 | Low | V522 | There might be dereferencing of a potential null pointer 'attr'. |
General Analysis | core |
gridctrl.cpp:487 | Low | V1026 | The 'width' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
grideditors.cpp:331 | Low | V522 | There might be dereferencing of a potential null pointer 'm_control'. |
General Analysis | core |
grideditors.cpp:278 | Low | V522 | There might be dereferencing of a potential null pointer 'm_control'. |
General Analysis | core |
grideditors.cpp:1461 | Low | V522 | There might be dereferencing of a potential null pointer 'm_control'. |
General Analysis | core |
infobar.cpp:339 | Low | V522 | There might be dereferencing of a potential null pointer 'item->GetWindow()'. |
General Analysis | core |
infobar.cpp:377 | Low | V522 | There might be dereferencing of a potential null pointer 'item->GetWindow()'. |
General Analysis | core |
infobar.cpp:409 | Low | V522 | There might be dereferencing of a potential null pointer 'item->GetWindow()'. |
General Analysis | core |
spinctlg.cpp:533 | Low | V550 | An odd precise comparison: val != m_value. It's probably better to use a comparison with defined precision: fabs(A - B) > Epsilon. |
General Analysis | core |
spinctlg.cpp:518 | Low | V550 | An odd precise comparison: m_increment != 0. It's probably better to use a comparison with defined precision: fabs(A - B) > Epsilon. |
General Analysis | core |
spinctlg.cpp:250 | Low | V522 | There might be dereferencing of a potential null pointer. |
General Analysis | core |
srchctlg.cpp:1034 | Low | V537 | Consider reviewing the correctness of 'x' item's usage. |
General Analysis | core |
tipwin.cpp:349 | Low | V1026 | The 'pt.y' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
treectlg.cpp:143 | Low | V730 | Not all members of a class are initialized inside the constructor. Consider inspecting: m_state, m_parent, m_attr, m_images, m_x, m_y, ... |
General Analysis | core |
treelist.cpp:45 | Low | V566 | The integer constant is converted to pointer. Possibly an error or a bad coding style. |
General Analysis | core |
vlbox.cpp:346 | Low | V1026 | The 'itemrect.y' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
vlbox.cpp:489 | Low | V1026 | The 'rectRow.y' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
vscroll.cpp:932 | Low | V1026 | The 'h_rect.width' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
vscroll.cpp:927 | Low | V1026 | The 'h_rect.x' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
vscroll.cpp:920 | Low | V1026 | The 'v_rect.height' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
vscroll.cpp:915 | Low | V1026 | The 'v_rect.y' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
vscroll.cpp:865 | Low | V1026 | The 'h_rect.x' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
vscroll.cpp:730 | Low | V1026 | The 's' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
vscroll.cpp:860 | Low | V1026 | The 'v_rect.y' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
vscroll.cpp:537 | Low | V1026 | The 'orient_pos' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
vscroll.cpp:397 | Low | V1026 | The 's' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
vscroll.cpp:289 | Low | V1026 | The 'size' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
vscroll.cpp:542 | Low | V1026 | The 'orient_size' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
wizard.cpp:957 | Low | V550 | An odd precise comparison. It's probably better to use a comparison with defined precision: fabs(A - B) > Epsilon. |
General Analysis | core |
wizard.cpp:183 | Low | V522 | There might be dereferencing of a potential null pointer 'item->GetWindow()'. |
General Analysis | core |
wizard.cpp:197 | Low | V522 | There might be dereferencing of a potential null pointer 'item->GetWindow()'. |
General Analysis | core |
app.cpp:977 | Low | V667 | The 'throw' operator does not possess any arguments and is not situated within the 'catch' block. |
General Analysis | core |
bitmap.cpp:1539 | Low | V522 | There might be dereferencing of a potential null pointer 'bitmap.GetPalette()'. |
General Analysis | core |
colordlg.cpp:247 | Low | V524 | It is odd that the body of 'DoGetClientSize' function is fully equivalent to the body of 'DoGetSize' function. |
General Analysis | core |
datecontrols.cpp:55 | Low | V601 | The 'false' value is implicitly cast to the integer type. |
General Analysis | core |
dc.cpp:1426 | Low | V550 | An odd precise comparison: angle == 0.0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
dc.cpp:1949 | Low | V550 | An odd precise comparison: x == m_userScaleX. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
dc.cpp:1949 | Low | V550 | An odd precise comparison: y == m_userScaleY. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
dc.cpp:2202 | Low | V522 | There might be dereferencing of a potential null pointer 'mask'. |
General Analysis | core |
dc.cpp:995 | Low | V1027 | Pointer to an object of the 'wxPoint' class is cast to unrelated 'tagPOINT' class. |
General Analysis | core |
dc.cpp:927 | Low | V1027 | Pointer to an object of the 'wxPoint' class is cast to unrelated 'tagPOINT' class. |
General Analysis | core |
dc.cpp:967 | Low | V1027 | Pointer to an object of the 'wxPoint' class is cast to unrelated 'tagPOINT' class. |
General Analysis | core |
dcmemory.cpp:90 | Low | V522 | There might be dereferencing of a potential null pointer 'msw_impl'. |
General Analysis | core |
dialog.cpp:74 | Low | V522 | There might be dereferencing of a potential null pointer 'm_ptr'. |
General Analysis | core |
dialup.cpp:941 | Low | V601 | The 'false' value is implicitly cast to the integer type. |
General Analysis | core |
dialup.cpp:1106 | Low | V513 | Use _beginthreadex/_endthreadex functions instead of CreateThread/ExitThread functions. |
General Analysis | core |
dialup.cpp:644 | Low | V601 | The 'true' value is implicitly cast to the integer type. |
General Analysis | core |
dialup.cpp:881 | Low | V601 | The 'true' value is implicitly cast to the integer type. |
General Analysis | core |
dialup.cpp:665 | Low | V574 | The 'rasEntries' pointer is used simultaneously as an array and as a pointer to single object. Check lines: 665, 708. |
General Analysis | core |
dib.cpp:733 | Low | V769 | The 'alpha' pointer in the 'alpha ++' expression could be nullptr. In such case, resulting value will be senseless and it should not be used. |
General Analysis | core |
dragimag.cpp:320 | Low | V575 | The potential null pointer is passed into 'ImageList_BeginDrag' function. Inspect the first argument. |
General Analysis | core |
dragimag.cpp:172 | Low | V575 | The potential null pointer is passed into 'ImageList_Add' function. Inspect the second argument. |
General Analysis | core |
dragimag.cpp:180 | Low | V575 | The potential null pointer is passed into 'ImageList_Add' function. Inspect the second argument. |
General Analysis | core |
evtloop.cpp:41 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
font.cpp:654 | Low | V550 | An odd precise comparison: static_cast<double> (pointSize) != d. It's probably better to use a comparison with defined precision: fabs(A - B) > Epsilon. |
General Analysis | core |
font.cpp:405 | Low | V550 | An odd precise comparison: pointSize != 0.0f. It's probably better to use a comparison with defined precision: fabs(A - B) > Epsilon. |
General Analysis | core |
graphics.cpp:1653 | Low | V522 | There might be dereferencing of a potential null pointer 'msw'. |
General Analysis | core |
graphics.cpp:1566 | Low | V676 | It is incorrect to compare the variable of BOOL type with TRUE. Correct expression is: 'm_matrix->IsIdentity() != FALSE'. |
General Analysis | core |
graphics.cpp:1560 | Low | V676 | It is incorrect to compare the variable of BOOL type with TRUE. |
General Analysis | core |
graphics.cpp:1498 | Low | V676 | It is incorrect to compare the variable of BOOL type with TRUE. Correct expression is: 'm_path->IsVisible((FLOAT) x, (FLOAT) y) != FALSE'. |
General Analysis | core |
graphics.cpp:1697 | Low | V656 | Variables 'm_state1', 'm_state2' are initialized through the call to the same function. It's probably an error or un-optimized code. Consider inspecting the 'm_context->Save()' expression. Check lines: 1696, 1697. |
General Analysis | core |
graphics.cpp:1393 | Low | V550 | An odd precise comparison: angle == 0. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
graphicsd2d.cpp:965 | Low | V550 | An odd precise comparison: lhs._11 == rhs._11. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
graphicsd2d.cpp:667 | Low | V525 | The code contains the collection of similar blocks. Check items 'D2D1_CAP_STYLE_ROUND', 'D2D1_CAP_STYLE_SQUARE', 'D2D1_CAP_STYLE_FLAT', 'D2D1_CAP_STYLE_FLAT' in lines 667, 669, 671, 673. |
General Analysis | core |
graphicsd2d.cpp:967 | Low | V550 | An odd precise comparison: lhs._31 == rhs._31. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
graphicsd2d.cpp:966 | Low | V550 | An odd precise comparison: lhs._22 == rhs._22. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
graphicsd2d.cpp:966 | Low | V550 | An odd precise comparison: lhs._21 == rhs._21. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
graphicsd2d.cpp:965 | Low | V550 | An odd precise comparison: lhs._12 == rhs._12. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
graphicsd2d.cpp:685 | Low | V525 | The code contains the collection of similar blocks. Check items 'D2D1_LINE_JOIN_BEVEL', 'D2D1_LINE_JOIN_MITER', 'D2D1_LINE_JOIN_ROUND', 'D2D1_LINE_JOIN_MITER' in lines 685, 687, 689, 691. |
General Analysis | core |
graphicsd2d.cpp:2206 | Low | V519 | The 'hr' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 2205, 2206. |
General Analysis | core |
graphicsd2d.cpp:967 | Low | V550 | An odd precise comparison: lhs._32 == rhs._32. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
graphicsd2d.cpp:2925 | Low | V522 | There might be dereferencing of a potential null pointer 'wxWICImagingFactory()'. |
General Analysis | core |
graphicsd2d.cpp:2806 | Low | V522 | There might be dereferencing of a potential null pointer 'wxDWriteFactory()'. |
General Analysis | core |
graphicsd2d.cpp:2734 | Low | V522 | There might be dereferencing of a potential null pointer 'wxDWriteFactory()'. |
General Analysis | core |
graphicsd2d.cpp:2195 | Low | V522 | There might be dereferencing of a potential null pointer 'wxWICImagingFactory()'. |
General Analysis | core |
graphicsd2d.cpp:1975 | Low | V522 | There might be dereferencing of a potential null pointer 'wxWICImagingFactory()'. |
General Analysis | core |
graphicsd2d.cpp:1131 | Low | V550 | An odd precise comparison: lhs.x == rhs.x. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
graphicsd2d.cpp:493 | Low | V522 | There might be dereferencing of a potential null pointer 'm_node'. |
General Analysis | core |
graphicsd2d.cpp:1131 | Low | V550 | An odd precise comparison: lhs.y == rhs.y. It's probably better to use a comparison with defined precision: fabs(A - B) < Epsilon. |
General Analysis | core |
imaglist.cpp:130 | Low | V575 | The potential null pointer is passed into 'ImageList_GetIconSize' function. Inspect the first argument. |
General Analysis | core |
imaglist.cpp:184 | Low | V575 | The potential null pointer is passed into 'ImageList_Add' function. Inspect the second argument. |
General Analysis | core |
listbox.cpp:742 | Low | V676 | It is incorrect to compare the variable of BOOL type with TRUE. |
General Analysis | core |
mdi.cpp:1628 | Low | V601 | The 'true' value is implicitly cast to the integer type. |
General Analysis | core |
menu.cpp:547 | Low | V547 | Expression '!updateAllMargins' is always true. |
General Analysis | core |
msgdlg.cpp:124 | Low | V601 | The 'false' value is implicitly cast to the integer type. |
General Analysis | core |
activex.cpp:1204 | Low | V522 | There might be dereferencing of a potential null pointer 'msw'. |
General Analysis | core |
safearray.cpp:69 | Low | V575 | The potential null pointer is passed into 'SafeArrayGetDim' function. Inspect the first argument. |
General Analysis | core |
printdlg.cpp:403 | Low | V676 | It is incorrect to compare the variable of BOOL type with TRUE. Correct expression is: 'printer->Open(printerName) != FALSE'. |
General Analysis | core |
radiobut.cpp:229 | Low | V601 | The 'true' value is implicitly cast to the integer type. Inspect the first argument. |
General Analysis | core |
region.cpp:138 | Low | V1027 | Pointer to an object of the 'wxPoint' class is cast to unrelated 'tagPOINT' class. |
General Analysis | core |
statusbar.cpp:486 | Low | V1026 | The 'width' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
statusbar.cpp:482 | Low | V1026 | The 'width' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
statusbar.cpp:476 | Low | V1026 | The 'width' variable is incremented in the loop. Undefined behavior will occur in case of signed integer overflow. |
General Analysis | core |
textctrl.cpp:88 | Low | V566 | The integer constant is converted to pointer. Possibly an error or a bad coding style: reinterpret_cast< wxDropTarget * > (1) |
General Analysis | core |
window.cpp:3746 | Low | V522 | There might be dereferencing of a potential null pointer 'csparam'. |
General Analysis | core |
window.cpp:5922 | Low | V624 | The constant 3.14159265 is being utilized. The resulting value could be inaccurate. Consider using the M_PI constant from <math.h>. |
Micro-optimizations | core |
docview.cpp:1182 | High | V808 | 'errMsg' object of 'wxString' type was created but was not utilized. |
Micro-optimizations | core |
dirctrlg.cpp:697 | High | V808 | 'search' object of 'wxString' type was created but was not utilized. |
Micro-optimizations | core |
dirctrlg.cpp:697 | High | V808 | 'filename' object of 'wxString' type was created but was not utilized. |
Micro-optimizations | core |
anystr.h:91 | Medium | V806 | Decreased performance. The expression of strlen(MyStr.c_str()) kind can be rewritten as MyStr.length(). |
Micro-optimizations | core |
datetime.h:1953 | Medium | V818 | It is more efficient to use an initialization list 'm_diff(hours)' rather than an assignment operator. |
Micro-optimizations | core |
datetime.h:1346 | Medium | V818 | It is more efficient to use an initialization list 'm_diff(diff)' rather than an assignment operator. |
Micro-optimizations | core |
datetime.h:1117 | Medium | V818 | It is more efficient to use an initialization list 'm_time(time)' rather than an assignment operator. |
Micro-optimizations | core |
dcsvg.h:213 | Medium | V807 | Decreased performance. Consider creating a reference to avoid using the 'region.GetBox()' expression repeatedly. |
Micro-optimizations | core |
filesys.h:43 | Medium | V818 | It is more efficient to use an initialization list 'm_Location(loc), m_Anchor(anchor), m_Modif(modif)' rather than an assignment operator. |
Micro-optimizations | core |
graphics.h:429 | Medium | V813 | Decreased performance. The 'col' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
graphics.h:442 | Medium | V813 | Decreased performance. The 'col' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
graphics.h:438 | Medium | V813 | Decreased performance. The 'col' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
log.h:409 | Medium | V813 | Decreased performance. The 'component' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:2943 | Medium | V813 | Decreased performance. The 'first', 'last' arguments should probably be rendered as constant references. |
Micro-optimizations | core |
string.h:3515 | Medium | V813 | Decreased performance. The 'ch' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:3509 | Medium | V813 | Decreased performance. The 'ch' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:3489 | Medium | V813 | Decreased performance. The 'i' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:3487 | Medium | V813 | Decreased performance. The 'i' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:3304 | Medium | V813 | Decreased performance. The 'ch' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:3196 | Medium | V813 | Decreased performance. The 'ch' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:3188 | Medium | V813 | Decreased performance. The 'ch' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:3180 | Medium | V813 | Decreased performance. The 'ch' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:3038 | Medium | V813 | Decreased performance. The 'ch' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:2996 | Medium | V813 | Decreased performance. The 'ch' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:2946 | Medium | V813 | Decreased performance. The 'first', 'last' arguments should probably be rendered as constant references. |
Micro-optimizations | core |
string.h:2933 | Medium | V813 | Decreased performance. The 'first', 'last', 'first1', 'last1' arguments should probably be rendered as constant references. |
Micro-optimizations | core |
string.h:3172 | Medium | V813 | Decreased performance. The 'ch' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:2911 | Medium | V813 | Decreased performance. The 'first', 'last' arguments should probably be rendered as constant references. |
Micro-optimizations | core |
string.h:1926 | Medium | V813 | Decreased performance. The 'ch' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:1835 | Medium | V813 | Decreased performance. The 'ch' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:976 | Medium | V813 | Decreased performance. The 'i' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:2920 | Medium | V813 | Decreased performance. The 'first', 'last' arguments should probably be rendered as constant references. |
Micro-optimizations | core |
string.h:2153 | Medium | V813 | Decreased performance. The 'ch' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:2304 | Medium | V813 | Decreased performance. The 'ch' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:2432 | Medium | V813 | Decreased performance. The 'ch' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:1970 | Medium | V813 | Decreased performance. The 'ch' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:2442 | Medium | V813 | Decreased performance. The 'first', 'last' arguments should probably be rendered as constant references. |
Micro-optimizations | core |
string.h:2575 | Medium | V813 | Decreased performance. The 'ch' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:2585 | Medium | V813 | Decreased performance. The 'first', 'last' arguments should probably be rendered as constant references. |
Micro-optimizations | core |
string.h:2688 | Medium | V813 | Decreased performance. The 'it' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:2076 | Medium | V813 | Decreased performance. The 'c' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:2710 | Medium | V813 | Decreased performance. The 'it' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:2702 | Medium | V813 | Decreased performance. The 'it', 'first', 'last' arguments should probably be rendered as constant references. |
Micro-optimizations | core |
string.h:2741 | Medium | V813 | Decreased performance. The 'first', 'last' arguments should probably be rendered as constant references. |
Micro-optimizations | core |
string.h:2748 | Medium | V813 | Decreased performance. The 'first' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:2873 | Medium | V813 | Decreased performance. The 'first', 'last' arguments should probably be rendered as constant references. |
Micro-optimizations | core |
string.h:2882 | Medium | V813 | Decreased performance. The 'first', 'last' arguments should probably be rendered as constant references. |
Micro-optimizations | core |
string.h:2891 | Medium | V813 | Decreased performance. The 'first', 'last' arguments should probably be rendered as constant references. |
Micro-optimizations | core |
string.h:2718 | Medium | V813 | Decreased performance. The 'it' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:2712 | Medium | V813 | Decreased performance. The 'it' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:2714 | Medium | V813 | Decreased performance. The 'it' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
string.h:2901 | Medium | V813 | Decreased performance. The 'first', 'last' arguments should probably be rendered as constant references. |
Micro-optimizations | core |
tbarbase.h:63 | Medium | V818 | It is more efficient to use an initialization list 'm_bmpNormal(bmpNormal), m_bmpDisabled(bmpDisabled)' rather than an assignment operator. |
Micro-optimizations | core |
treelist.h:326 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.h:268 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.h:211 | Medium | V813 | Decreased performance. The 'parent', 'previous' arguments should probably be rendered as constant references. |
Micro-optimizations | core |
treelist.h:222 | Medium | V813 | Decreased performance. The 'parent' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.h:201 | Medium | V813 | Decreased performance. The 'parent' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
utils.h:219 | Medium | V818 | It is more efficient to use an initialization list 'm_stringValue(defValue)' rather than an assignment operator. |
Micro-optimizations | core |
versioninfo.h:22 | Medium | V818 | It is more efficient to use an initialization list 'm_name(name), m_description(description), m_copyright(copyright)' rather than an assignment operator. |
Micro-optimizations | core |
bmpbase.cpp:36 | Medium | V818 | It is more efficient to use an initialization list 'm_value(value)' rather than an assignment operator. |
Micro-optimizations | core |
bmpbase.cpp:37 | Medium | V818 | It is more efficient to use an initialization list 'm_value(value)' rather than an assignment operator. |
Micro-optimizations | core |
cmdproc.cpp:46 | Medium | V818 | It is more efficient to use an initialization list 'm_commandName(name)' rather than an assignment operator. |
Micro-optimizations | core |
cmndata.cpp:285 | Medium | V818 | It is more efficient to use an initialization list 'm_printData(printData)' rather than an assignment operator. |
Micro-optimizations | core |
cmndata.cpp:204 | Medium | V818 | It is more efficient to use an initialization list 'm_printData(printData)' rather than an assignment operator. |
Micro-optimizations | core |
colourcmn.cpp:29 | Medium | V818 | It is more efficient to use an initialization list 'm_value(value)' rather than an assignment operator. |
Micro-optimizations | core |
datavcmn.cpp:673 | Medium | V818 | It is more efficient to use an initialization list 'm_value(value)' rather than an assignment operator. |
Micro-optimizations | core |
datavcmn.cpp:976 | Medium | V813 | Decreased performance. The 'rectCell' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
datavcmn.cpp:2064 | Medium | V813 | Decreased performance. The 'cell' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
datavcmn.cpp:1957 | Medium | V813 | Decreased performance. The 'cell' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
datavcmn.cpp:1919 | Medium | V813 | Decreased performance. The 'labelRect' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
datavcmn.cpp:1860 | Medium | V813 | Decreased performance. The 'labelRect' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
datavcmn.cpp:1808 | Medium | V813 | Decreased performance. The 'rect' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
datavcmn.cpp:1785 | Medium | V813 | Decreased performance. The 'labelRect' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
datavcmn.cpp:1975 | Medium | V818 | It is more efficient to use an initialization list 'm_value(value)' rather than an assignment operator. |
Micro-optimizations | core |
datavcmn.cpp:1059 | Medium | V813 | Decreased performance. The 'rect' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
datavcmn.cpp:1779 | Medium | V818 | It is more efficient to use an initialization list 'm_choices(choices)' rather than an assignment operator. |
Micro-optimizations | core |
datavcmn.cpp:1731 | Medium | V813 | Decreased performance. The 'rect' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
dcsvg.cpp:74 | Medium | V813 | Decreased performance. The 'c' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
dcsvg.cpp:99 | Medium | V813 | Decreased performance. The 'c' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
docview.cpp:817 | Medium | V818 | It is more efficient to use an initialization list 'm_description(descr), m_directory(dir), m_defaultExt(ext), m_fileFilter(filter), m_docTypeName(docTypeName), ...' rather than an assignment operator. |
Micro-optimizations | core |
fs_mem.cpp:34 | Medium | V818 | It is more efficient to use an initialization list 'm_MimeType(mime)' rather than an assignment operator. |
Micro-optimizations | core |
fs_mem.cpp:43 | Medium | V818 | It is more efficient to use an initialization list 'm_MimeType(mime)' rather than an assignment operator. |
Micro-optimizations | core |
image.cpp:51 | Medium | V818 | It is more efficient to use an initialization list 'm_value(value)' rather than an assignment operator. |
Micro-optimizations | core |
lboxcmn.cpp:304 | Medium | V820 | The 'selections' variable is not used after copying. Copying can be replaced with move/swap for optimization. |
Micro-optimizations | core |
markupparser.cpp:62 | Medium | V813 | Decreased performance. The 'end' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
paper.cpp:57 | Medium | V818 | It is more efficient to use an initialization list 'm_paperName(name)' rather than an assignment operator. |
Micro-optimizations | core |
persist.cpp:170 | Medium | V813 | Decreased performance. The 'value' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
prntbase.cpp:591 | Medium | V818 | It is more efficient to use an initialization list 'm_printoutTitle(title)' rather than an assignment operator. |
Micro-optimizations | core |
quantize.cpp:267 | Medium | V802 | On 32-bit platform, structure size can be reduced from 44 to 40 bytes by rearranging the fields according to their sizes in decreasing order. |
Micro-optimizations | core |
xpmdecod.cpp:815 | Medium | V803 | Decreased performance. In case 'it' is iterator it's more effective to use prefix form of increment. Replace iterator++ with ++iterator. |
Micro-optimizations | core |
datavgen.cpp:1441 | Medium | V813 | Decreased performance. The 'rect' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
datavgen.cpp:1377 | Medium | V813 | Decreased performance. The 'rect' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
datavgen.cpp:1233 | Medium | V813 | Decreased performance. The 'cell' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
datavgen.cpp:1139 | Medium | V813 | Decreased performance. The 'labelRect' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
datavgen.cpp:1345 | Medium | V818 | It is more efficient to use an initialization list 'm_label(label)' rather than an assignment operator. |
Micro-optimizations | core |
datavgen.cpp:1516 | Medium | V818 | It is more efficient to use an initialization list 'm_bitmap(bitmap)' rather than an assignment operator. |
Micro-optimizations | core |
dirctrlg.cpp:284 | Medium | V818 | It is more efficient to use an initialization list 'm_path(path), m_name(name)' rather than an assignment operator. |
Micro-optimizations | core |
dirctrlg.cpp:563 | Medium | V807 | Decreased performance. Consider creating a reference to avoid using the 'event.GetLabel()' expression repeatedly. |
Micro-optimizations | core |
dirctrlg.cpp:663 | Medium | V813 | Decreased performance. The 'parentId' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
dirctrlg.cpp:994 | Medium | V813 | Decreased performance. The 'itemId' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
dirctrlg.cpp:989 | Medium | V813 | Decreased performance. The 'itemId' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
dirctrlg.cpp:827 | Medium | V813 | Decreased performance. The 'parentId' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
filectrlg.cpp:139 | Medium | V818 | It is more efficient to use an initialization list 'm_fileName(fileName), m_filePath(filePath)' rather than an assignment operator. |
Micro-optimizations | core |
grid.cpp:1943 | Medium | V810 | Decreased performance. The 'wxMin(scaleX, scaleY)' function was called several times with identical arguments. The result should possibly be saved to a temporary variable, which then could be used while calling the 'SetUserScale' function. |
Micro-optimizations | core |
gridctrl.cpp:82 | Medium | V818 | It is more efficient to use an initialization list 'm_iformat(informat), m_oformat(outformat)' rather than an assignment operator. |
Micro-optimizations | core |
gridsel.cpp:813 | Medium | V810 | Decreased performance. The 'wxGridCellCoords(row, col)' function was called several times with identical arguments. The result should possibly be saved to a temporary variable, which then could be used while calling the 'BlockToDeviceRect' function. |
Micro-optimizations | core |
helpext.cpp:136 | Medium | V818 | It is more efficient to use an initialization list 'url(iurl), doc(idoc)' rather than an assignment operator. |
Micro-optimizations | core |
splash.cpp:143 | Medium | V818 | It is more efficient to use an initialization list 'm_bitmap(bitmap)' rather than an assignment operator. |
Micro-optimizations | core |
treebkg.cpp:439 | Medium | V813 | Decreased performance. The 'pageId' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treebkg.cpp:320 | Medium | V813 | Decreased performance. The 'pageId' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1222 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:320 | Medium | V813 | Decreased performance. The 'dvi' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1243 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1208 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1263 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1304 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1312 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1403 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1172 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1165 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1158 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1215 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1147 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1122 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1112 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1086 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1074 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1067 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1059 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1052 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1023 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1010 | Medium | V813 | Decreased performance. The 'parent', 'previous' arguments should probably be rendered as constant references. |
Micro-optimizations | core |
treelist.cpp:1140 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
treelist.cpp:1255 | Medium | V813 | Decreased performance. The 'item' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
app.cpp:156 | Medium | V818 | It is more efficient to use an initialization list 'focused(focused_)' rather than an assignment operator. |
Micro-optimizations | core |
dcprint.cpp:138 | Medium | V818 | It is more efficient to use an initialization list 'm_printData(printData)' rather than an assignment operator. |
Micro-optimizations | core |
graphicsd2d.cpp:745 | Medium | V813 | Decreased performance. The 'colour' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
graphicsd2d.cpp:1211 | Medium | V802 | On 32-bit platform, structure size can be reduced from 36 to 28 bytes by rearranging the fields according to their sizes in decreasing order. |
Micro-optimizations | core |
graphicsd2d.cpp:2503 | Medium | V801 | Decreased performance. It is better to redefine the second function argument as a reference. Consider replacing 'const .. brush' with 'const .. &brush'. |
Micro-optimizations | core |
access.cpp:96 | Medium | V818 | It is more efficient to use an initialization list 'm_variant(variant)' rather than an assignment operator. |
Micro-optimizations | core |
slider.cpp:599 | Medium | V813 | Decreased performance. The 'colBg' argument should probably be rendered as a constant reference. |
Micro-optimizations | core |
taskbarbutton.cpp:1281 | Medium | V818 | It is more efficient to use an initialization list 'm_appID(appID)' rather than an assignment operator. |
Micro-optimizations | core |
png.h:653 | Medium | V802 | On 32-bit platform, structure size can be reduced from 20 to 16 bytes by rearranging the fields according to their sizes in decreasing order. |
Micro-optimizations | core |
dataview.h:1211 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (m_data)' check can be removed. |
Micro-optimizations | core |
dnd.h:147 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (m_dataObject)' check can be removed. |
Micro-optimizations | core |
encconv.h:56 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (m_Table)' check can be removed. |
Micro-optimizations | core |
cmndata.cpp:110 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (m_privData)' check can be removed. |
Micro-optimizations | core |
combocmn.cpp:1297 | Low | V821 | Decreased performance. The 'sz' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
ctrlsub.cpp:173 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (clientDataOld)' check can be removed. |
Micro-optimizations | core |
datavcmn.cpp:2452 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (m_data)' check can be removed. |
Micro-optimizations | core |
datavcmn.cpp:342 | Low | V820 | The 'temp' variable is not used after copying. Copying can be replaced with move/swap for optimization. |
Micro-optimizations | core |
datavcmn.cpp:2800 | Low | V807 | Decreased performance. Consider creating a reference to avoid using the 'node->GetChildren()' expression repeatedly. |
Micro-optimizations | core |
datavcmn.cpp:341 | Low | V820 | The 'value2' variable is not used after copying. Copying can be replaced with move/swap for optimization. |
Micro-optimizations | core |
datavcmn.cpp:340 | Low | V820 | The 'value1' variable is not used after copying. Copying can be replaced with move/swap for optimization. |
Micro-optimizations | core |
datavcmn.cpp:1032 | Low | V821 | Decreased performance. The 'changeFont' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
imagpnm.cpp:38 | Low | V821 | Decreased performance. The 'text_stream' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
imagxpm.cpp:101 | Low | V820 | The 'img' variable is not used after copying. Copying can be replaced with move/swap for optimization. |
Micro-optimizations | core |
matrix.cpp:473 | Low | V820 | The 'result' variable is not used after copying. Copying can be replaced with move/swap for optimization. |
Micro-optimizations | core |
prntbase.cpp:539 | Low | V807 | Decreased performance. Consider creating a reference to avoid using the 'wxSizerFlags().Expand()' expression repeatedly. |
Micro-optimizations | core |
prntbase.cpp:1871 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (m_previewPrintout)' check can be removed. |
Micro-optimizations | core |
prntbase.cpp:1873 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (m_previewBitmap)' check can be removed. |
Micro-optimizations | core |
prntbase.cpp:1875 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (m_printPrintout)' check can be removed. |
Micro-optimizations | core |
prntbase.cpp:88 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (wxPrintFactory::m_factory)' check can be removed. |
Micro-optimizations | core |
wincmn.cpp:1764 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (m_caret)' check can be removed. |
Micro-optimizations | core |
wincmn.cpp:2303 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (m_tooltip)' check can be removed. |
Micro-optimizations | core |
wincmn.cpp:1786 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (m_windowValidator)' check can be removed. |
Micro-optimizations | core |
xpmdecod.cpp:817 | Low | V807 | Decreased performance. Consider creating a reference to avoid using the 'it->second' expression repeatedly. |
Micro-optimizations | core |
xpmdecod.cpp:805 | Low | V807 | Decreased performance. Consider creating a reference to avoid using the 'entry->second' expression repeatedly. |
Micro-optimizations | core |
choicdgg.cpp:356 | Low | V807 | Decreased performance. Consider creating a reference to avoid using the 'wxSizerFlags().Expand()' expression repeatedly. |
Micro-optimizations | core |
datavgen.cpp:6813 | Low | V820 | The 'v' variable is not used after copying. Copying can be replaced with move/swap for optimization. |
Micro-optimizations | core |
dirctrlg.cpp:1457 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (m_smallImageList)' check can be removed. |
Micro-optimizations | core |
dirctrlg.cpp:1218 | Low | V821 | Decreased performance. The 'sz' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
dragimgg.cpp:61 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (m_windowDC)' check can be removed. |
Micro-optimizations | core |
filectrlg.cpp:944 | Low | V807 | Decreased performance. Consider creating a reference to avoid using the 'wxSizerFlags().Expand()' expression repeatedly. |
Micro-optimizations | core |
grid.cpp:6451 | Low | V807 | Decreased performance. Consider creating a pointer to avoid using the 'editor->GetControl()' expression repeatedly. |
Micro-optimizations | core |
grid.cpp:5551 | Low | V821 | Decreased performance. The 'rect' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
laywin.cpp:271 | Low | V807 | Decreased performance. Consider creating a reference to avoid using the 'parent->GetChildren()' expression repeatedly. |
Micro-optimizations | core |
sashwin.cpp:428 | Low | V821 | Decreased performance. The 'mediumShadowPen' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
sashwin.cpp:431 | Low | V821 | Decreased performance. The 'hilightPen' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
sashwin.cpp:430 | Low | V821 | Decreased performance. The 'lightShadowPen' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
sashwin.cpp:429 | Low | V821 | Decreased performance. The 'darkShadowPen' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
splash.cpp:206 | Low | V821 | Decreased performance. The 'dc' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
splash.cpp:193 | Low | V821 | Decreased performance. The 'dc' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
treectlg.cpp:1790 | Low | V807 | Decreased performance. Consider creating a reference to avoid using the 'parent->GetChildren()' expression repeatedly. |
Micro-optimizations | core |
anybutton.cpp:1067 | Low | V821 | Decreased performance. The 'hpenWhite' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
anybutton.cpp:1066 | Low | V821 | Decreased performance. The 'hpenLightGr' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
bmpcbox.cpp:131 | Low | V821 | Decreased performance. The 'value' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
calctrl.cpp:399 | Low | V821 | Decreased performance. The 'states' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
combo.cpp:190 | Low | V821 | Decreased performance. The 'sz' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
combo.cpp:328 | Low | V821 | Decreased performance. The 'rectTextField' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
dc.cpp:2639 | Low | V821 | Decreased performance. The 'r' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
dialog.cpp:183 | Low | V821 | Decreased performance. The 'modalData' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
dragimag.cpp:343 | Low | V821 | Decreased performance. The 'curHotSpot' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
graphics.cpp:2113 | Low | V821 | Decreased performance. The 's' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
graphicsd2d.cpp:1356 | Low | V820 | The 'newPathGeometry' variable is not used after copying. Copying can be replaced with move/swap for optimization. |
Micro-optimizations | core |
graphicsd2d.cpp:2219 | Low | V821 | Decreased performance. The 'color' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
listctrl.cpp:3385 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (lvItem.pszText)' check can be removed. |
Micro-optimizations | core |
listctrl.cpp:872 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (lvItem.pszText)' check can be removed. |
Micro-optimizations | core |
activex.cpp:1188 | Low | V821 | Decreased performance. The 'dc' variable can be constructed in a lower level scope. |
Micro-optimizations | core |
uuid.cpp:105 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (m_pszCForm)' check can be removed. |
Micro-optimizations | core |
printdlg.cpp:957 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (pd)' check can be removed. |
Micro-optimizations | core |
printdlg.cpp:735 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (pd)' check can be removed. |
Micro-optimizations | core |
printdlg.cpp:1080 | Low | V807 | Decreased performance. Consider creating a reference to avoid using the 'data.GetPrintData()' expression repeatedly. |
Micro-optimizations | core |
printwin.cpp:113 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (dc)' check can be removed. |
Micro-optimizations | core |
progdlg.cpp:815 | Low | V820 | The 'iconSmall' variable is not used after copying. Copying can be replaced with move/swap for optimization. |
Micro-optimizations | core |
progdlg.cpp:816 | Low | V820 | The 'iconBig' variable is not used after copying. Copying can be replaced with move/swap for optimization. |
Micro-optimizations | core |
taskbar.cpp:130 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (m_win)' check can be removed. |
Micro-optimizations | core |
taskbarbutton.cpp:1134 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (item)' check can be removed. |
Micro-optimizations | core |
taskbarbutton.cpp:1407 | Low | V809 | Verifying that a pointer value is not NULL is not required. The 'if (category)' check can be removed. |
Micro-optimizations | core |
treectrl.cpp:3656 | Low | V821 | Decreased performance. The 'item' variable can be constructed in a lower level scope. |