Soldiers' healthbar misses pixel

Bug #1296889 reported by wl-zocker
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Fix Released
Low
Unassigned

Bug Description

See attached screenshot. For the soldiers inside the building, the pixel on the top right of their lifebar is missing. For the soldiers outside, it is the pixel on bottom right.

Related branches

Revision history for this message
wl-zocker (wl-zocker) wrote :
Revision history for this message
SirVer (sirver) wrote :

That is some sharp eyes you have there. I think this is a regression.

Changed in widelands:
status: New → Confirmed
milestone: none → build19-rc1
wl-zocker (wl-zocker)
description: updated
Revision history for this message
Hans Joachim Desserud (hjd) wrote :

Nice catch. :)
Looking at the screenshot there's obviously an issue here, but I don't seem to be able to reproduce it in r6900. Could you check whether this is still reproducible, and possibly whether switching opengl rendering on/off makes a difference?

tags: added: military ui
Revision history for this message
wl-zocker (wl-zocker) wrote :

Since I do not compile myself, I have to rely on Tino's builds (and he has some trouble compiling on Windows). Therefore the latest available build is r6890.
When OpenGL is enabled, the pixels are missing. When I disable it, both health bars are drawn correctly.

tags: added: opengl
Revision history for this message
wl-zocker (wl-zocker) wrote :

Just one more update: The pixels are still missing in r6910. In build 18, it is like in the current development builds.
In build 17, the pixel has already been missing, but in both cases (inside and ouside of a military building), it was on the bottom right corner (therefore no regression). Obviously, my eyes are not that sharp.

Revision history for this message
SirVer (sirver) wrote :

Setting to incomplete for bug sweeping.

Changed in widelands:
status: Confirmed → Incomplete
Revision history for this message
Hans Joachim Desserud (hjd) wrote :

I can't see this in r7199 on Ubuntu 14.04, with or without opengl enabled. Could you check whether you still have this issue? Might be a Windows- and/or opengl-driver issue.

Revision history for this message
wl-zocker (wl-zocker) wrote :

Still the same situation in r7199: With OpenGL, missing pixels. When disabled, all pixels are shown.

Revision history for this message
Hans Joachim Desserud (hjd) wrote :

Ok, thanks for checking. Is this on Windows? Could you check whether you see it on Ubuntu too?

Revision history for this message
wl-zocker (wl-zocker) wrote :

Yes, #8 is on windows. On Ubuntu, everything is fine.

tags: added: windows
Revision history for this message
Hans Joachim Desserud (hjd) wrote :

Ok, could you check which graphics driver you are running? (And maybe if there's a newer version of that driver, if that fixes the issue?)

Revision history for this message
wl-zocker (wl-zocker) wrote :

Some information attached, maybe it helps. According to the update function, I am using the latest version.
If no one else can reproduce this, I can also live with those missing pixels (they do not hinder gameplay that much).

Revision history for this message
SirVer (sirver) wrote :

I finally found out what this is causing: http://stackoverflow.com/questions/4532342/opengl-gl-lines-enpoints-not-joining

A fix could be to make a line loop with 5 points instead of only 4 to draw a rectangle.

Changed in widelands:
status: Incomplete → Confirmed
importance: Undecided → Low
SirVer (sirver)
Changed in widelands:
assignee: nobody → SirVer (sirver)
Revision history for this message
SirVer (sirver) wrote :

Could/Should be fixed in 7343. Can somebody verify?

Changed in widelands:
status: Confirmed → Fix Committed
assignee: SirVer (sirver) → nobody
Revision history for this message
wl-zocker (wl-zocker) wrote :

No, it isn't. If you look at the attached screenshot from r7343, you see that the blue area does not even fill everything inside the white border (and the pixel at the corner is still missing).

Changed in widelands:
status: Fix Committed → Confirmed
Revision history for this message
SirVer (sirver) wrote :

Thanks for checking, I'll look into this again.

Changed in widelands:
assignee: nobody → SirVer (sirver)
SirVer (sirver)
Changed in widelands:
status: Confirmed → In Progress
Revision history for this message
kaputtnik (franku) wrote :
GunChleoc (gunchleoc)
Changed in widelands:
status: In Progress → Fix Committed
assignee: SirVer (sirver) → nobody
GunChleoc (gunchleoc)
Changed in widelands:
status: Fix Committed → Fix Released
Revision history for this message
GunChleoc (gunchleoc) wrote :

Fixed in build19-rc1.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.