summaryrefslogtreecommitdiff
path: root/src/gui/widgets
diff options
context:
space:
mode:
authorAndrei Karas <akaras@inbox.ru>2011-03-28 19:00:43 +0300
committerAndrei Karas <akaras@inbox.ru>2011-03-28 20:36:29 +0300
commitf0ccfc8b1c210202f0eab10d9bf2381a27f8750a (patch)
tree911ad5dbb48748b549e352979bd4450ff9dc34f1 /src/gui/widgets
parentdc4146a968b71ee098810c02c5ec4957d0d25817 (diff)
downloadplus-f0ccfc8b1c210202f0eab10d9bf2381a27f8750a.tar.gz
plus-f0ccfc8b1c210202f0eab10d9bf2381a27f8750a.tar.bz2
plus-f0ccfc8b1c210202f0eab10d9bf2381a27f8750a.tar.xz
plus-f0ccfc8b1c210202f0eab10d9bf2381a27f8750a.zip
Fix compilation warnings.
Diffstat (limited to 'src/gui/widgets')
-rw-r--r--src/gui/widgets/browserbox.cpp15
-rw-r--r--src/gui/widgets/window.cpp5
2 files changed, 13 insertions, 7 deletions
diff --git a/src/gui/widgets/browserbox.cpp b/src/gui/widgets/browserbox.cpp
index bc5583148..20d1803f8 100644
--- a/src/gui/widgets/browserbox.cpp
+++ b/src/gui/widgets/browserbox.cpp
@@ -83,6 +83,9 @@ void BrowserBox::addRow(const std::string &row, bool atTop)
std::string::size_type idx1, idx2, idx3;
gcn::Font *font = getFont();
+ if (getWidth() < 0)
+ return;
+
// Use links and user defined colors
if (mUseLinksAndUserColors)
{
@@ -201,7 +204,7 @@ void BrowserBox::addRow(const std::string &row, bool atTop)
row.substr(nextChar,
(nextSpacePos - nextChar)));
- if ((x + nextWordWidth + 10) > getWidth())
+ if ((x + nextWordWidth + 10) > (unsigned)getWidth())
{
x = 15; // Ident in new line
y += 1;
@@ -209,7 +212,7 @@ void BrowserBox::addRow(const std::string &row, bool atTop)
}
}
// Wrapping looong lines (brutal force)
- else if ((x + 2 * hyphenWidth) > getWidth())
+ else if ((x + 2 * hyphenWidth) > (unsigned)getWidth())
{
x = 15; // Ident in new line
y += 1;
@@ -338,6 +341,9 @@ int BrowserBox::calcHeight()
unsigned x = 0, y = 0;
int wrappedLines = 0;
int link = 0;
+ if (getWidth() < 0)
+ return 1;
+
gcn::Font *font = getFont();
int fontHeight = font->getHeight();
@@ -464,7 +470,7 @@ int BrowserBox::calcHeight()
// Auto wrap mode
if (mMode == AUTO_WRAP && getWidth() > 0
&& font->getWidth(part) > 0
- && (x + font->getWidth(part) + 10) > getWidth())
+ && (x + font->getWidth(part) + 10) > (unsigned)getWidth())
{
bool forced = false;
@@ -494,7 +500,8 @@ int BrowserBox::calcHeight()
part = row.substr(start, end - start + 1);
}
while (end > start && font->getWidth(part) > 0
- && (x + font->getWidth(part) + 10) > getWidth());
+ && (x + font->getWidth(part) + 10)
+ > (unsigned)getWidth());
if (forced)
{
diff --git a/src/gui/widgets/window.cpp b/src/gui/widgets/window.cpp
index 1b008a087..22daa5dc9 100644
--- a/src/gui/widgets/window.cpp
+++ b/src/gui/widgets/window.cpp
@@ -774,9 +774,8 @@ int Window::getResizeHandles(gcn::MouseEvent &event)
int resizeHandles = 0;
const unsigned y = event.getY();
- if (mGrip && (y > static_cast<int>(mTitleBarHeight)
- || (y < static_cast<int>(getPadding()) && mTitleBarHeight
- > getPadding())))
+ if (mGrip && (y > mTitleBarHeight || (y < getPadding()
+ && mTitleBarHeight > getPadding())))
{
const unsigned x = event.getX();