Notes |
|
(0005404)
|
Fay
|
2015-02-01 21:39
|
|
Hmm, could it depend on OS/compiler? On my win7/mingw it works correctly.
Does it look the same during the hero movement?
Does it happen in world view mode as well? |
|
|
(0005406)
|
SXX
|
2015-02-02 00:49
|
|
When I tested Fay branch there was no such issue on Linux. |
|
|
|
It also happens during movement and in world view.
I can see console message "Call loadBitmap with void name". |
|
|
(0005408)
|
Fay
|
2015-02-02 17:01
|
|
Well, that's strange. From what I've checked, hero images are .defs and they don't use CBitmapHandler::loadBitmap (the one that prints this message) at all. Maybe it's not related (also I don't have any of these messages).
I can cause this issue (cyan bg) by changing this call https://github.com/vcmi/vcmi/blob/develop/client/mapHandler.cpp#L799 [^] to pass false as the last param (but only in world view mode, normal map still works correctly for me). |
|
|
(0005426)
|
Warmonger
|
2015-02-09 07:49
(edited on: 2015-02-09 08:01) |
|
Okay, that message was related just to "world view" option:
Call to loadBitmap with void fname!
Call to loadBitmap with void fname!
Error: Failed to find file
bgWorldView not defined in resolution config; fallback to VWorld.bmp
|
|
|
(0005427)
|
Fay
|
2015-02-09 11:29
|
|
Yeah, this one is expected & correct as long as you don't have this value defined in resolutions.json.
Are you able to check what happens for you after changing this this call https://github.com/vcmi/vcmi/blob/develop/client/mapHandler.cpp#L799 [^] to pass false instead of true as the last param? I think this might have changed from the previous implementation (it looks like it was drawn without alpha for some reason, maybe it was the correct behavior). |
|
|
(0005430)
|
Warmonger
|
2015-02-09 15:03
(edited on: 2015-02-09 15:04) |
|
Hey it works now. Was there a reason to change this line before?
However, def in world view is still incorrect.
|
|
|
(0005431)
|
Fay
|
2015-02-09 15:18
|
|
Well, THAT was the reason. :)
For me, flags in normal view work both with and without alpha blitting. In world view only alpha blitting worked (because wv works slightly differently, since it needs to copy the surfaces when resizing).
I'll look into it. |
|
|
(0005432)
|
Fay
|
2015-02-09 16:31
|
|
I think I found the problem in world view when alpha blitting is off. Apparently SDL loses information about color keying when copying (or scaling?) the surface.
Seems that adding SDL_SetColorKey(...cyan...) on the scaled surface solves the problem. I'll test it some more and push the fix later today. |
|
|
(0005437)
|
Fay
|
2015-02-09 19:45
|
|
|
|
|
|