fb_generic: use const int by ref for more safety

This commit is contained in:
2019-10-27 16:24:47 +01:00
parent 8cca903780
commit 15a310be47
2 changed files with 5 additions and 5 deletions

View File

@@ -1929,7 +1929,7 @@ void CFrameBuffer::setFbArea(int element, int _x, int _y, int _dx, int _dy)
} }
} }
int CFrameBuffer::checkFbAreaElement(int _x, int _y, int _dx, int _dy, fb_area_t *area) int CFrameBuffer::checkFbAreaElement(const int& _x, const int& _y, const int& _dx, const int& _dy, const fb_area_t *area)
{ {
if (fb_no_check) if (fb_no_check)
return FB_PAINTAREA_MATCH_NO; return FB_PAINTAREA_MATCH_NO;
@@ -1945,7 +1945,7 @@ int CFrameBuffer::checkFbAreaElement(int _x, int _y, int _dx, int _dy, fb_area_t
return FB_PAINTAREA_MATCH_OK; return FB_PAINTAREA_MATCH_OK;
} }
bool CFrameBuffer::_checkFbArea(int _x, int _y, int _dx, int _dy, bool prev) bool CFrameBuffer::_checkFbArea(const int& _x, const int& _y, const int& _dx, const int& _dy, const bool& prev)
{ {
if (v_fbarea.empty()) if (v_fbarea.empty())
return true; return true;

View File

@@ -329,8 +329,8 @@ class CFrameBuffer : public sigc::trackable
v_fbarea_t v_fbarea; v_fbarea_t v_fbarea;
bool do_paint_mute_icon; bool do_paint_mute_icon;
bool _checkFbArea(int _x, int _y, int _dx, int _dy, bool prev); bool _checkFbArea(const int& _x, const int& _y, const int& _dx, const int& _dy, const bool& prev);
int checkFbAreaElement(int _x, int _y, int _dx, int _dy, fb_area_t *area); int checkFbAreaElement(const int& _x, const int& _y, const int& _dx, const int& _dy, const fb_area_t *area);
public: public:
enum { enum {
@@ -341,7 +341,7 @@ class CFrameBuffer : public sigc::trackable
FB_PAINTAREA_MAX FB_PAINTAREA_MAX
}; };
inline bool checkFbArea(int _x, int _y, int _dx, int _dy, bool prev) { return (fbAreaActiv && !fb_no_check) ? _checkFbArea(_x, _y, _dx, _dy, prev) : true; } inline bool checkFbArea(const int& _x, const int& _y, const int& _dx, const int& _dy, const bool& prev) { return (fbAreaActiv && !fb_no_check) ? _checkFbArea(_x, _y, _dx, _dy, prev) : true; }
void setFbArea(int element, int _x=0, int _y=0, int _dx=0, int _dy=0); void setFbArea(int element, int _x=0, int _y=0, int _dx=0, int _dy=0);
void fbNoCheck(bool noCheck) { fb_no_check = noCheck; } void fbNoCheck(bool noCheck) { fb_no_check = noCheck; }
void doPaintMuteIcon(bool mode) { do_paint_mute_icon = mode; } void doPaintMuteIcon(bool mode) { do_paint_mute_icon = mode; }