Develop/eagle 1070u1 desktop · linuxdeepin/dde-file-manager@06d2b53

Parameter 'surfaces' is passed by value. It could be passed as a const reference which is usually faster and recommended in C++.

The function 'organizeAction' is never used.

Parameter 'surfaces' is passed by value. It could be passed as a const reference which is usually faster and recommended in C++.

The function 'hookCanvasMenu' is never used.

The function 'eventContextMenu' is never used.

Variable 'lay' is assigned a value that is never used.

Consecutive return, break, continue, goto or throw statements are unnecessary. The second statement can never be executed, and so should be removed.

The function 'setPositionIndicatorRect' is never used.

Suspicious calculation. Please use parentheses to clarify the code. The code ''a%b?c:d'' should be written as either ''(a%b)?c:d'' or ''a%(b?c:d)''.

Redundant initialization for 'pos'. The initialized value is overwritten before it is read.

Provide feedback