Replacing Text::paletteShift

[Yankes]

Nice job, surprised to see it actually works.

I noticed your text colors seem off though. You probably need to update Surface::paletteShift to copy SDL_Color's new alpha field. There might be other places where you need to check this (I assume you already corrected the palettes to be fully opaque).

how about dropping `Surface::paletteShift` all together? I have test branch that replace it with custom blit function.
I dont publish it because of feature freeze before 1.0 and low performance grain (new function is faster that old one but because paletteShift isnt used offten, its hard to see difference).

Mod Edit: Split discussion since it's not Android-related.

[SupSuper]

how about dropping `Surface::paletteShift` all together? I have test branch that replace it with custom blit function.
I dont publish it because of feature freeze before 1.0 and low performance grain (new function is faster that old one but because paletteShift isnt used offten, its hard to see difference).

As long as it doesn't break anything.

[Yankes]

As long as it doesn't break anything.

you can test it yourself that didn't do it
https://github.com/Yankes/OpenXcom/tree/NewTextPrint

[SupSuper]

you can test it yourself that didn't do it
https://github.com/Yankes/OpenXcom/tree/NewTextPrint

It breaks the Battlescape text.

[Yankes]

Yes, recently I spot this too Is already fixed.
Btw in orginal when you select aim type you can see some dark pixels separate form rest of characters. I think I could blend or remove them.

Some screenshots form hack that remove darkest pixels from fonts (this is far form proper fix but it can work as example)

[SupSuper]

I think the darkest pixels help keep the text from blending into the background. Anyways post-editing the fonts should probably be avoided as they are user-editable and people can just change the PNG themselves if they want an alternate look (like the Amiga fonts).

Also I don't see a fix in your branch.

[Yankes]

Ok, branch get updated.

[SupSuper]

Looks good, merged.

Edit: Reverted, it breaks "soldier hair colors" mod (they become invisible).

[Yankes]

Looks good, merged.

Edit: Reverted, it breaks "soldier hair colors" mod (they become invisible).

I see where I failed :/ Hint for future, dont do two things at once. I was in the middle of improving "soldier hair colors" that cause that bug to disappeared (I didnt even saw it).
After cherry picking commits I bring it back. Sorry for that SupSuper, I will create proper commit.

[Yankes]

https://github.com/Yankes/OpenXcom/tree/NewTextPrint <- new fixed commit (old ones was trashed)

[SupSuper]

Thanks, it's in.

Btw someone pointed me to an issue in your FixedFloat.h:

Engine/FixedFloat.h(59): warning C4244: '=' : conversion from 'const double' to 'long', possible loss of data

Shouldn't you be using a int64_t instead of a long?

[Yankes]

I will look into it.

[Yankes]

Intended sized was int32, line 59 lack only cast to long.
And now fun part, when I wrote it back then I didnt know that shift `-1 << OFF` cause undefined behavior. But after some lurking on https://www.open-std.org/pipermail/ub/ (mailing list of standard comity of C++ working on undefined behavior) show that I was wrong.

After some thoughts I think best way to fix it is remove this file all together
It was used in experimental globe shading but finally I didnt used it. I include it because possible future use, but after more than year nobody used it (except for `pch.h` that include all headers).
Same history is for `ShaderRotate`.

This is branch that remove this headers (form VS files and makefiles too):
https://github.com/Yankes/OpenXcom/tree/RemoveTrash

[SupSuper]

Merged.