Ticket #7918: more_readability.patch

File more_readability.patch, 8.6 KB (added by SF/japj, 18 years ago)

more readability patch

  • costume.cpp

    RCS file: /cvsroot/scummvm/scummvm/costume.cpp,v
    retrieving revision 1.9
    diff -u -r1.9 costume.cpp
     
    670670}
    671671
    672672void CostumeRenderer::loadCostume(int id) {
    673         _ptr = _vm->getResourceAddress(3, id);
     673        _ptr = _vm->getResourceAddress(rtCostume, id);
    674674       
    675675        if (_vm->_majorScummVersion == 6) {
    676676                _ptr += 8;
  • gfx.cpp

    RCS file: /cvsroot/scummvm/scummvm/gfx.cpp,v
    retrieving revision 1.19
    diff -u -r1.19 gfx.cpp
     
    255255        int t;
    256256        CameraData *cd = &camera;
    257257
    258         if (cd->_mode!=2 || abs(dest - cd->_curPos) > 160) {
     258        if (cd->_mode!=CM_FOLLOW_ACTOR || abs(dest - cd->_curPos) > 160) {
    259259                cd->_curPos = dest;
    260260        }
    261261        cd->_destPos = dest;
     
    279279        int t,i;
    280280        CameraData *cd = &camera;
    281281
    282         cd->_mode = 2;
     282        cd->_mode = CM_FOLLOW_ACTOR;
    283283        cd->_follows = a->number;
    284284
    285285        if (a->room != _currentRoom) {
    286286                startScene(a->room, 0, 0);
    287                 cd->_mode = 2;
     287                cd->_mode = CM_FOLLOW_ACTOR;
    288288                cd->_curPos = a->x;
    289289                setCameraAt(cd->_curPos);
    290290        }
     
    14531453                return;
    14541454        }
    14551455
    1456         if (cd->_mode==2) {
     1456        if (cd->_mode==CM_FOLLOW_ACTOR) {
    14571457                a = derefActorSafe(cd->_follows, "moveCamera");
    14581458               
    14591459                actorx = a->x;
     
    15201520void Scumm::panCameraTo(int x) {
    15211521        CameraData *cd = &camera;
    15221522        cd->_destPos = x;
    1523         cd->_mode = 3;
     1523        cd->_mode = CM_PANNING;
    15241524        cd->_movingToActor = 0;
    15251525}
    15261526
     
    15301530
    15311531        /* mi1 compatibilty */
    15321532        if (act==0) {
    1533                 cd->_mode = 1;
     1533                cd->_mode = CM_NORMAL;
    15341534                cd->_follows = 0;
    15351535                cd->_movingToActor = 0;
    15361536                return;
     
    15461546
    15471547void Scumm::setCameraAtEx(int at) {
    15481548        CameraData *cd = &camera;
    1549         cd->_mode = 1;
     1549        cd->_mode = CM_NORMAL;
    15501550        cd->_curPos = at;
    15511551        setCameraAt(at);
    15521552        cd->_movingToActor = 0;
  • gui.cpp

    RCS file: /cvsroot/scummvm/scummvm/gui.cpp,v
    retrieving revision 1.2
    diff -u -r1.2 gui.cpp
     
    450450        }
    451451       
    452452        _s->getKeyInput(0);
    453         if (_s->_mouseButStat&0x8000) {
     453        if (_s->_mouseButStat&MBS_LEFT_CLICK) {
    454454                leftMouseClick(_s->mouse.x, _s->mouse.y);
    455455        } else if (_s->_lastKeyHit) {
    456456                addLetter(_s->_lastKeyHit);
  • object.cpp

    RCS file: /cvsroot/scummvm/scummvm/object.cpp,v
    retrieving revision 1.14
    diff -u -r1.14 object.cpp
     
    758758                cdhd = (CodeHeader*)findResource(MKID('CDHD'), obim, 0);
    759759                imhd = (ImageHeader*)findResource(MKID('IMHD'), ptr, 0);
    760760        } else {
    761                 ensureResourceLoaded(1, room);
    762                 roomptr = getResourceAddress(1, room);
     761                ensureResourceLoaded(rtRoom, room);
     762                roomptr = getResourceAddress(rtRoom, room);
    763763                rmhd = (RoomHeader*)findResource(MKID('RMHD'), roomptr, 0);
    764764
    765765                numobj = READ_LE_UINT16(&rmhd->numObjects);
     
    881881        _lastXstart = vs->xstart;
    882882
    883883        if (eo->l==0) {
    884                 roomptr = getResourceAddress(1, _roomResource);
     884                roomptr = getResourceAddress(rtRoom, _roomResource);
    885885                index = getObjectIndex(eo->a);
    886886                ptr = roomptr + _objs[index].offs_obim_to_room;
    887887        } else if (eo->a!=0) {
  • resource.cpp

    RCS file: /cvsroot/scummvm/scummvm/resource.cpp,v
    retrieving revision 1.19
    diff -u -r1.19 resource.cpp
     
    395395
    396396        checkRange(_maxCharsets-1, 1, no, "Loading illegal charset %d");
    397397//      ensureResourceLoaded(6, no);
    398         ptr = getResourceAddress(6, no);
     398        ptr = getResourceAddress(rtCharset, no);
    399399
    400400        for (i=0; i<15; i++) {
    401401                _charsetData[no][i+1] = ptr[i+14];
     
    571571}
    572572
    573573void Scumm::setResourceCounter(int type, int index, byte flag) {
    574         res.flags[type][index] &= 0x80;
     574        res.flags[type][index] &= RF_LOCK;
    575575        res.flags[type][index] |= flag;
    576576}
    577577
     
    663663
    664664void Scumm::lock(int type, int i) {
    665665        validateResource("Locking", type, i);
    666         res.flags[type][i] |= 0x80;
     666        res.flags[type][i] |= RF_LOCK;
    667667
    668668//      debug(1, "locking %d,%d", type, i);
    669669}
    670670
    671671void Scumm::unlock(int type, int i) {
    672672        validateResource("Unlocking", type, i);
    673         res.flags[type][i] &= ~0x80;
     673        res.flags[type][i] &= ~RF_LOCK;
    674674
    675675//      debug(1, "unlocking %d,%d", type, i);
    676676}
     
    697697
    698698        for (i=rtFirst; i<=rtLast; i++) {
    699699                for(j=res.num[i]; --j>=0;) {
    700                         counter = res.flags[i][j] & 0x7F;
    701                         if (counter && counter < 0x7F) {
     700                        counter = res.flags[i][j] & ~RF_LOCK;
     701                        if (counter && counter < ~RF_LOCK) {
    702702                                setResourceCounter(i, j, counter+1);
    703703                        }
    704704                }
  • saveload.cpp

    RCS file: /cvsroot/scummvm/scummvm/saveload.cpp,v
    retrieving revision 1.14
    diff -u -r1.14 saveload.cpp
     
    466466        if (s->isSaving()) {
    467467                for (i=rtFirst; i<=rtLast; i++)
    468468                        for(j=1; j<res.num[i]; j++) {
    469                                 if (res.flags[i][j]&0x80) {
     469                                if (res.flags[i][j]&RF_LOCK) {
    470470                                        s->saveByte(i);
    471471                                        s->saveWord(j);
    472472                                }
     
    475475        } else {
    476476                int r;
    477477                while ((r = s->loadByte()) != 0xFF) {
    478                         res.flags[r][s->loadWord()] |= 0x80;
     478                        res.flags[r][s->loadWord()] |= RF_LOCK;
    479479                }
    480480        }
    481481}
  • script_v1.cpp

    RCS file: /cvsroot/scummvm/scummvm/script_v1.cpp,v
    retrieving revision 1.12
    diff -u -r1.12 script_v1.cpp
     
    16401640        case 3: /* set string char */
    16411641                a = getVarOrDirectByte(0x80);
    16421642                b = getVarOrDirectByte(0x40);
    1643                 ptr = getResourceAddress(7, a);
     1643                ptr = getResourceAddress(rtString, a);
    16441644                if (ptr==NULL) error("String %d does not exist", a);
    16451645                c = getVarOrDirectByte(0x20);
    16461646                ptr[b] = c;
  • scumm.h

    RCS file: /cvsroot/scummvm/scummvm/scumm.h,v
    retrieving revision 1.26
    diff -u -r1.26 scumm.h
     
    330330        OF_STATE_SHL = 4
    331331};
    332332
     333enum {
     334        MBS_LEFT_CLICK = 0x8000,
     335        MBS_RIGHT_CLICK = 0x4000,
     336        MBS_MOUSE_MASK = 0xC000,
     337        MBS_MAX_KEY     = 0x0200
     338};
     339
     340enum {
     341        RF_LOCK = 0x80
     342};
     343
     344enum {
     345        CM_NORMAL = 1,
     346        CM_FOLLOW_ACTOR = 2,
     347        CM_PANNING = 3,
     348};
     349
    333350#define _maxRooms res.num[rtRoom]
    334351#define _maxScripts res.num[rtScript]
    335352#define _maxCostumes res.num[rtCostume]
  • scummvm.cpp

    RCS file: /cvsroot/scummvm/scummvm/scummvm.cpp,v
    retrieving revision 1.25
    diff -u -r1.25 scummvm.cpp
     
    815815                _mouseButStat = 0;
    816816                _lastKeyHit = _vars[VAR_CUTSCENEEXIT_KEY];
    817817        } else if (_leftBtnPressed&1) {
    818                 _mouseButStat = 0x8000;
     818                _mouseButStat = MBS_LEFT_CLICK;
    819819        } else if (_rightBtnPressed&1) {
    820                 _mouseButStat = 0x4000;
     820                _mouseButStat = MBS_RIGHT_CLICK;
    821821        }
    822822       
    823823        _leftBtnPressed &= ~1;
     
    829829void Scumm::convertKeysToClicks() {
    830830        if (_lastKeyHit && _cursorState>0) {
    831831                if (_lastKeyHit==9) {
    832                         _mouseButStat = 0x4000;
     832                        _mouseButStat = MBS_RIGHT_CLICK;
    833833                }       else if (_lastKeyHit==13) {
    834                         _mouseButStat = 0x8000;
     834                        _mouseButStat = MBS_LEFT_CLICK;
    835835                } else
    836836                        return;
    837837                _lastKeyHit = 0;
  • verbs.cpp

    RCS file: /cvsroot/scummvm/scummvm/verbs.cpp,v
    retrieving revision 1.9
    diff -u -r1.9 verbs.cpp
     
    3636        if (_userPut<=0 || _mouseButStat==0)
    3737                return;
    3838
    39         if (_mouseButStat < 0x200) {
     39        if (_mouseButStat < MBS_MAX_KEY) {
    4040                /* Check keypresses */
    4141                vs = &_verbs[1];
    4242                for (i=1; i<_maxVerbs; i++,vs++) {
     
    4848                        }
    4949                }
    5050                runInputScript(4, _mouseButStat, 1);
    51         } else if (_mouseButStat&0xC000) {
    52                 byte code = _mouseButStat&0x8000 ? 1 : 2;
     51        } else if (_mouseButStat&MBS_MOUSE_MASK) {
     52                byte code = _mouseButStat&MBS_LEFT_CLICK ? 1 : 2;
    5353                if (mouse.y >= virtscr[0].topline && mouse.y < virtscr[0].topline + virtscr[0].height) {
    5454                        over = checkMouseOver(mouse.x, mouse.y);
    5555                        if (over != 0) {