The constants used here are unsigned so it makes more sense that the parameters are as well.tags/v1.4.90
@@ -38,13 +38,13 @@ void CMsgHandler::setDesktopSize(int width, int height) | |||
cp.height = height; | |||
} | |||
void CMsgHandler::setExtendedDesktopSize(int reason, int result, | |||
void CMsgHandler::setExtendedDesktopSize(unsigned reason, unsigned result, | |||
int width, int height, | |||
const ScreenSet& layout) | |||
{ | |||
cp.supportsSetDesktopSize = true; | |||
if ((reason == (signed)reasonClient) && (result != (signed)resultSuccess)) | |||
if ((reason == reasonClient) && (result != resultSuccess)) | |||
return; | |||
if (!layout.validate(width, height)) |
@@ -46,7 +46,7 @@ namespace rfb { | |||
// methods to set the members of cp appropriately. | |||
virtual void setDesktopSize(int w, int h); | |||
virtual void setExtendedDesktopSize(int reason, int result, | |||
virtual void setExtendedDesktopSize(unsigned reason, unsigned result, | |||
int w, int h, | |||
const ScreenSet& layout); | |||
virtual void setCursor(int width, int height, const Point& hotspot, |
@@ -313,8 +313,8 @@ void CConn::setDesktopSize(int w, int h) | |||
} | |||
// setExtendedDesktopSize() is a more advanced version of setDesktopSize() | |||
void CConn::setExtendedDesktopSize(int reason, int result, int w, int h, | |||
const rfb::ScreenSet& layout) | |||
void CConn::setExtendedDesktopSize(unsigned reason, unsigned result, | |||
int w, int h, const rfb::ScreenSet& layout) | |||
{ | |||
CConnection::setExtendedDesktopSize(reason, result, w, h, layout); | |||
@@ -51,8 +51,8 @@ public: | |||
void serverInit(); | |||
void setDesktopSize(int w, int h); | |||
void setExtendedDesktopSize(int reason, int result, int w, int h, | |||
const rfb::ScreenSet& layout); | |||
void setExtendedDesktopSize(unsigned reason, unsigned result, | |||
int w, int h, const rfb::ScreenSet& layout); | |||
void setName(const char* name); | |||