Difference between revisions of "AGOS/TODO"

From ScummVM :: Wiki
Jump to navigation Jump to search
(eriktorbjorn fixed peg puzzle problems in FF)
(Update FF todo)
Line 16: Line 16:
 
* Fix VGA resource management, sometimes can't find free block.
 
* Fix VGA resource management, sometimes can't find free block.
 
* Fix the image glitch when moving near the trunk in the Junkyard.
 
* Fix the image glitch when moving near the trunk in the Junkyard.
* Fix the image glitch when switching between Feeble and Sam, after capturing Filbbo.
+
* Fix glitches when scrolling in Oracle interface, all text in oracle should be drawn to background buffer.
* Fix glitches when scrolling in Oracle interface, all text in oracle should be drawn to background buffer (Kirben).
 
 
* Fix glitch with automatic scrolling in Oracle interface. See [[media:Oracle-glitch.png|this series of screenshots]] from the DIRECTIVE CHARTER article.
 
* Fix glitch with automatic scrolling in Oracle interface. See [[media:Oracle-glitch.png|this series of screenshots]] from the DIRECTIVE CHARTER article.
  

Revision as of 02:11, 29 April 2006

Simon the Sorcerer 1

  • Add support for decoding graphics in the Amiga verisons
  • Add support for Protracker MOD format, used for music in Amiga versions.
  • Improve support for GMF music format used in DOS versions.
  • Fix chest disappearing in Swampling's house in DOS versions.

Simon the Sorcerer 2

  • Add support for language files used by Amiga and Macintosh verisons.
  • Add support for music fade out in vc62_fastFadeOut() when changing locations.
  • Fix missing trails during lion ride to Goblin Camp in non-English versions.

Feeble Files

  • Add support for cursors made up of multiple images.
  • Add support for resuming position of last OmniTV video, after changing rooms.
  • Fix VGA resource management, sometimes can't find free block.
  • Fix the image glitch when moving near the trunk in the Junkyard.
  • Fix glitches when scrolling in Oracle interface, all text in oracle should be drawn to background buffer.
  • Fix glitch with automatic scrolling in Oracle interface. See this series of screenshots from the DIRECTIVE CHARTER article.

Some observations about the missing line breaks

(This text is kept for historical purposes. The line break bug should be fixed now.)

In the article titled THE COMPANY, only the first line is broken. The subsequent lines are not. The reason for this can be found in showmessage_print_char(): Every time an end of word (character codes 0, 10 or 32) appears, the function will check if the currently buffered word fits on the line or not:

    } else if (chr == 0 || chr == ' ' || chr == 10) {
        uint count = (getGameType() == GType_FF) ? _printCharPixelCount + 1 : _printCharPixelCount;
        if (_printCharMaxPos - _printCharCurPos >= count) {
            _printCharCurPos += _printCharPixelCount;

The test checks if the remaining space on the line is larger than the length of the buffered word, i.e. if the test succeeds the word will fit. In this case, _printCharMaxPos is always 360.

At the first line break, _printCharCurPos is 325 and count is 40. The word does not fit, and therefore the line is broken.

Where the second line break should be, _printCharCurPos is 366 and count is 25. At first glance it looks like the test will fail, because the remaining space is negative. However, we're using unsigned variables, so what at first apperas to be a small negative value is actually a large positive one.

So how did this happen? Shouldn't the line have been broken at the previous word? No, in fact it should not. At that point, _printCharCurPos is 327 and count is 32. The word fits, just barely, and _printCharCurPos is increased to 358. Problem is, the line break check is triggered, here as almost always, by a space. And in that case the width of that space will also be added to _printCharCurPos, bringing it up to 366.

There are several possible fixes. A few that come to mind:

  • Change the variables from unsigned to signed.
  • Check for overflow before adding the width of the space. (This may be what it's trying to do already by checking if _printCharCurPos == _printCharMaxPos.)
  • Rewrite the test so that signedness doesn't matter, e.g. if (_printCharCurPos + count < _printCharMaxPos)

Oddly enough, the original does use unsigned variables, and take none of these precautions. Is it possible that the compiler Adventure Soft used behaved differently (buggily?), or are we missing some subtle detail...?