[ scummvm-Patches-1221592 ] SCUMM: NUT fonts codec 44

[SourceForge.net]

Patches item #1221592, was opened at 2005-06-16 00:53
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=1221592&group_id=37116

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Gregory Montoir (cyx)
Assigned to: Nobody/Anonymous (nobody)
Summary: SCUMM: NUT fonts codec 44

Initial Comment:
The TODO file was mentionning this so I had a look
(apparently the implementation in scummvm wasn't done
from RE'ing, so there are some differences). The
attached patch should implement the decoding of the
smush codec 44 as the original interpreters did. Just
for the reference, I don't know if it was already known
but codec 44 is just codec 21 with a slight difference
in the bytes copying (see the patch).

The patch also tries cleanup a bit the NutRenderer class :
- by re-using the original smush_decode_codec1 function
(is there a need to duplicate it ?)
- removing the getCharOffsX and getCharOffsY methods
(they were unused...)

I haven't committed this as-is, because :
- I haven't tested it that much
- This could lead to regressions
- I am not a 'SMUSH' expert, so it would be nice if
aquadran you could have a look at it, for objections or
anything else...
- I also got rid of the 'russian' FT version hack,
because I'd like to find a proper way of fixing this ;
_sev, can you test this patch with your russian version
and report if the problem still there ?


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=1221592&group_id=37116