diff options
Diffstat (limited to 'src/engine')
| -rw-r--r-- | src/engine/save.c | 6 | ||||
| -rw-r--r-- | src/engine/save_failed_screen.c | 4 | ||||
| -rw-r--r-- | src/engine/sprite.c | 11 | ||||
| -rw-r--r-- | src/engine/string_util.c | 34 | ||||
| -rw-r--r-- | src/engine/text.c | 4 | ||||
| -rw-r--r-- | src/engine/tileset_anim.c | 46 | ||||
| -rw-r--r-- | src/engine/trade.c | 34 | 
7 files changed, 70 insertions, 69 deletions
| diff --git a/src/engine/save.c b/src/engine/save.c index eae3f4470..5ea949cae 100644 --- a/src/engine/save.c +++ b/src/engine/save.c @@ -282,7 +282,7 @@ u8 sub_81255B8(u16 a1, const struct SaveSectionLocation *location)      for (i = 0; i < sizeof(struct UnkSaveSection); i++)      { -        if (ProgramFlashByte(sector, i, ((u8 *)gFastSaveSection)[i])) +        if (ProgramFlashByte(sector, i, gFastSaveSection->data[i]))          {              status = 0xFF;              break; @@ -402,7 +402,7 @@ u8 sub_81258BC(u16 a1, const struct SaveSectionLocation *location)          {              u16 j;              for (j = 0; j < location[id].size; j++) -                ((u8 *)location[id].data)[j] = gFastSaveSection->data[j]; +                location[id].data[j] = gFastSaveSection->data[j];          }      } @@ -774,7 +774,7 @@ u8 unref_sub_8125FF0(u8 *data, u16 size)      for (i = 0; i < size; i++)          section->data[i] = data[i]; -    gLastSaveSectorStatus = ProgramFlashSectorAndVerifyNBytes(gFlashSectors[0], (u8 *)section, sizeof(struct SaveSection)); +    gLastSaveSectorStatus = ProgramFlashSectorAndVerifyNBytes(gFlashSectors[0], section, sizeof(struct SaveSection));      if (gLastSaveSectorStatus)          return 0xFF; diff --git a/src/engine/save_failed_screen.c b/src/engine/save_failed_screen.c index b91e8b5bf..17bda237e 100644 --- a/src/engine/save_failed_screen.c +++ b/src/engine/save_failed_screen.c @@ -267,10 +267,10 @@ static void VBlankCB_UpdateClockGraphics(void)  static bool8 VerifySectorWipe(u16 sector)  { -    u32 *ptr = (u32 *)unk_2000000; +    u32 *ptr = (u32 *)&unk_2000000;      u16 i; -    ReadFlash(sector, 0, (u8 *)ptr, 4096); +    ReadFlash(sector, 0, ptr, 4096);      for (i = 0; i < 0x400; i++, ptr++)          if (*ptr) diff --git a/src/engine/sprite.c b/src/engine/sprite.c index 5d394502a..34ca3802d 100644 --- a/src/engine/sprite.c +++ b/src/engine/sprite.c @@ -833,13 +833,13 @@ static void RequestSpriteFrameImageCopy(u16 index, u16 tileNum, const struct Spr      if (gSpriteCopyRequestCount < MAX_SPRITE_COPY_REQUESTS)      {          gSpriteCopyRequests[gSpriteCopyRequestCount].src = images[index].data; -        gSpriteCopyRequests[gSpriteCopyRequestCount].dest = (u8 *)OBJ_VRAM0 + TILE_SIZE_4BPP * tileNum; +        gSpriteCopyRequests[gSpriteCopyRequestCount].dest = OBJ_VRAM0 + TILE_SIZE_4BPP * tileNum;          gSpriteCopyRequests[gSpriteCopyRequestCount].size = images[index].size;          gSpriteCopyRequestCount++;      }  } -void RequestSpriteCopy(const u8 *src, u8 *dest, u16 size) +void RequestSpriteCopy(const void *src, u8 *dest, u16 size)  {      if (gSpriteCopyRequestCount < MAX_SPRITE_COPY_REQUESTS)      { @@ -850,6 +850,7 @@ void RequestSpriteCopy(const u8 *src, u8 *dest, u16 size)      }  } +// these two functions are unused.  void CopyFromSprites(u8 *dest)  {      u32 i; @@ -1479,7 +1480,7 @@ u16 LoadSpriteSheet(const struct SpriteSheet *sheet)      else      {          AllocSpriteTileRange(sheet->tag, (u16)tileStart, sheet->size / TILE_SIZE_4BPP); -        CpuCopy16(sheet->data, (u8 *)OBJ_VRAM0 + TILE_SIZE_4BPP * tileStart, sheet->size); +        CpuCopy16(sheet->data, OBJ_VRAM0 + TILE_SIZE_4BPP * tileStart, sheet->size);          return (u16)tileStart;      }  } @@ -1517,7 +1518,7 @@ void LoadTilesForSpriteSheet(const struct SpriteSheet *sheet)  {      const u8 *data = sheet->data;      u16 tileStart = GetSpriteTileStartByTag(sheet->tag); -    CpuCopy16(data, (u8 *)OBJ_VRAM0 + TILE_SIZE_4BPP * tileStart, sheet->size); +    CpuCopy16(data, OBJ_VRAM0 + TILE_SIZE_4BPP * tileStart, sheet->size);  }  void LoadTilesForSpriteSheets(struct SpriteSheet *sheets) @@ -1603,7 +1604,7 @@ void RequestSpriteSheetCopy(const struct SpriteSheet *sheet)  {      const u8 *data = sheet->data;      u16 tileStart = GetSpriteTileStartByTag(sheet->tag); -    RequestSpriteCopy(data, (u8 *)OBJ_VRAM0 + tileStart * TILE_SIZE_4BPP, sheet->size); +    RequestSpriteCopy(data, OBJ_VRAM0 + tileStart * TILE_SIZE_4BPP, sheet->size);  }  u16 LoadSpriteSheetDeferred(const struct SpriteSheet *sheet) diff --git a/src/engine/string_util.c b/src/engine/string_util.c index 9686256a1..7316f533a 100644 --- a/src/engine/string_util.c +++ b/src/engine/string_util.c @@ -376,7 +376,7 @@ u8 *StringExpandPlaceholders(u8 *dest, const u8 *src)      {          u8 c = *src++;          u8 placeholderId; -        u8 *expandedString; +        const u8 *expandedString;          u8 length;          switch (c) @@ -432,45 +432,45 @@ u8 *StringBraille(u8 *dest, const u8 *src)      }  } -static u8 *ExpandPlaceholder_UnknownStringVar(void) +static const u8 *ExpandPlaceholder_UnknownStringVar(void)  {      return gUnknownStringVar;  } -static u8 *ExpandPlaceholder_PlayerName(void) +static const u8 *ExpandPlaceholder_PlayerName(void)  {      return gSaveBlock2.playerName;  } -static u8 *ExpandPlaceholder_StringVar1(void) +static const u8 *ExpandPlaceholder_StringVar1(void)  {      return gStringVar1;  } -static u8 *ExpandPlaceholder_StringVar2(void) +static const u8 *ExpandPlaceholder_StringVar2(void)  {      return gStringVar2;  } -static u8 *ExpandPlaceholder_StringVar3(void) +static const u8 *ExpandPlaceholder_StringVar3(void)  {      return gStringVar3;  } -static u8 *ExpandPlaceholder_KunChan(void) +static const u8 *ExpandPlaceholder_KunChan(void)  {      if (gSaveBlock2.playerGender == MALE) -        return (u8 *) gExpandedPlaceholder_Kun; +        return gExpandedPlaceholder_Kun;      else -        return (u8 *) gExpandedPlaceholder_Chan; +        return gExpandedPlaceholder_Chan;  } -static u8 *ExpandPlaceholder_RivalName(void) +static const u8 *ExpandPlaceholder_RivalName(void)  {      if (gSaveBlock2.playerGender == MALE) -        return (u8 *) gExpandedPlaceholder_May; +        return gExpandedPlaceholder_May;      else -        return (u8 *) gExpandedPlaceholder_Brendan; +        return gExpandedPlaceholder_Brendan;  }  #define VERSION_DEPENDENT_PLACEHOLDER_LIST \ @@ -484,19 +484,19 @@ static u8 *ExpandPlaceholder_RivalName(void)  #ifdef SAPPHIRE  #define X(ph, r, s) \ -static u8 *ExpandPlaceholder_##ph(void) { return (u8 *) gExpandedPlaceholder_##s; } +static const u8 *ExpandPlaceholder_##ph(void) { return gExpandedPlaceholder_##s; }  VERSION_DEPENDENT_PLACEHOLDER_LIST  #else  #define X(ph, r, s) \ -static u8 *ExpandPlaceholder_##ph(void) { return (u8 *) gExpandedPlaceholder_##r; } +static const u8 *ExpandPlaceholder_##ph(void) { return gExpandedPlaceholder_##r; }  VERSION_DEPENDENT_PLACEHOLDER_LIST  #endif  #undef X -u8 *GetExpandedPlaceholder(u32 id) +const u8 *GetExpandedPlaceholder(u32 id)  { -    typedef u8 *(*ExpandPlaceholderFunc)(void); +    typedef const u8 *(*ExpandPlaceholderFunc)(void);      static const ExpandPlaceholderFunc funcs[] =      { @@ -517,7 +517,7 @@ u8 *GetExpandedPlaceholder(u32 id)      };      if (id >= ARRAY_COUNT(funcs)) -        return (u8 *) gExpandedPlaceholder_Empty; +        return gExpandedPlaceholder_Empty;      else          return funcs[id]();  } diff --git a/src/engine/text.c b/src/engine/text.c index 9bb33f3c2..1d1d31383 100644 --- a/src/engine/text.c +++ b/src/engine/text.c @@ -536,7 +536,7 @@ const struct WindowConfig gWindowConfig_81E6C74 =      0, // tilemap top coordinate      30, // width      20, // height -    (u8 *)OBJ_VRAM0, // tile data +    OBJ_VRAM0, // tile data      NULL, // tilemap  }; @@ -1670,7 +1670,7 @@ const struct WindowConfig gWindowConfig_81E725C =      0, // tilemap top coordinate      30, // width      20, // height -    (u8 *)OBJ_VRAM0, // tile data +    OBJ_VRAM0, // tile data      NULL, // tilemap  }; diff --git a/src/engine/tileset_anim.c b/src/engine/tileset_anim.c index 34685381d..f892e5e9a 100644 --- a/src/engine/tileset_anim.c +++ b/src/engine/tileset_anim.c @@ -32,8 +32,8 @@ extern u8 *gTilesetAnimTable_BikeShop[];  struct Dma  { -    u8 *src; -    u8 *dest; +    void *src; +    void *dest;      u16 size;  }; @@ -95,7 +95,7 @@ static void ClearTilesetAnimDmas(void)      CpuFill32(0, &gTilesetAnimDmas, sizeof(gTilesetAnimDmas));  } -static void QueueTilesetAnimDma(u8 *src, u8 *dest, u16 size) +static void QueueTilesetAnimDma(void *src, void *dest, u16 size)  {      if (gNumTilesetAnimDmas < 20)      { @@ -217,28 +217,28 @@ static void sub_8073070(u16 a1)  {      int v1;      v1 = a1 % 4; -    QueueTilesetAnimDma(gTilesetAnimTable_General_0[v1], (u8 *)(BG_VRAM + 0x3f80), 0x80); +    QueueTilesetAnimDma(gTilesetAnimTable_General_0[v1], BG_TILE_ADDR(127), 0x80);  }  static void sub_8073098(u16 a1)  {      u8 v1;      v1 = a1 % 8; -    QueueTilesetAnimDma(gTilesetAnimTable_General_1[v1], (u8 *)(BG_VRAM + 0x3600), 0x3c0); +    QueueTilesetAnimDma(gTilesetAnimTable_General_1[v1], BG_TILE_ADDR(108), 0x3c0);  }  static void sub_80730C0(u16 a1)  {      int v1;      v1 = a1 % 8; -    QueueTilesetAnimDma(gTilesetAnimTable_General_2[v1], (u8 *)(BG_VRAM + 0x3a00), 0x140); +    QueueTilesetAnimDma(gTilesetAnimTable_General_2[v1], BG_TILE_ADDR(116), 0x140);  }  static void sub_80730E8(u16 a1)  {      int v1;      v1 = a1 % 4; -    QueueTilesetAnimDma(gTilesetAnimTable_General_3[v1], (u8 *)(BG_VRAM + 0x3e00), 0xc0); +    QueueTilesetAnimDma(gTilesetAnimTable_General_3[v1], BG_TILE_ADDR(124), 0xc0);  }  void TilesetCB_Petalburg(void) @@ -454,37 +454,37 @@ static void sub_807361C(u16 a1)  {      int v1;      v1 = a1 % 4; -    QueueTilesetAnimDma(gTilesetAnimTable_General_4[v1], (u8 *)(BG_VRAM + 0x3c00), 0x140); +    QueueTilesetAnimDma(gTilesetAnimTable_General_4[v1], BG_TILE_ADDR(120), 0x140);  }  static void sub_8073644(u8 a1)  {      u8 v1;      v1 = a1 % 4; -    QueueTilesetAnimDma(gTilesetAnimTable_Lavaridge[v1], (u8 *)(BG_VRAM + 0x6400), 0x80); +    QueueTilesetAnimDma(gTilesetAnimTable_Lavaridge[v1], BG_TILE_ADDR(200), 0x80);      v1 = (a1 + 2) % 4; -    QueueTilesetAnimDma(gTilesetAnimTable_Lavaridge[v1], (u8 *)(BG_VRAM + 0x6480), 0x80); +    QueueTilesetAnimDma(gTilesetAnimTable_Lavaridge[v1], BG_TILE_ADDR(201), 0x80);  }  static void sub_807368C(u8 a1)  {      int v1;      v1 = a1 % 4; -    QueueTilesetAnimDma(gTilesetAnimTable_Pacifidlog_0[v1], (u8 *)(BG_VRAM + 0x7a00), 0x3c0); +    QueueTilesetAnimDma(gTilesetAnimTable_Pacifidlog_0[v1], BG_TILE_ADDR(244), 0x3c0);  }  static void sub_80736B4(u8 a1)  {      int v1;      v1 = a1 % 4; -    QueueTilesetAnimDma(gTilesetAnimTable_Underwater[v1], (u8 *)(BG_VRAM + 0x7e00), 0x80); +    QueueTilesetAnimDma(gTilesetAnimTable_Underwater[v1], BG_TILE_ADDR(252), 0x80);  }  static void sub_80736DC(u8 a1)  {      int v1;      v1 = a1 % 8; -    QueueTilesetAnimDma(gTilesetAnimTable_Pacifidlog_1[v1], (u8 *)(BG_VRAM + 0x7e00), 0x100); +    QueueTilesetAnimDma(gTilesetAnimTable_Pacifidlog_1[v1], BG_TILE_ADDR(252), 0x100);  }  static void sub_8073704(u16 a1, u8 a2) @@ -521,14 +521,14 @@ static void sub_80737E0(u16 a1)  {      int v1;      v1 = a1 % 2; -    QueueTilesetAnimDma(gTilesetAnimTable_Rustboro_1[v1], (u8 *)(BG_VRAM + 0x7800), 0x80); +    QueueTilesetAnimDma(gTilesetAnimTable_Rustboro_1[v1], BG_TILE_ADDR(240), 0x80);  }  static void sub_8073808(u16 a1)  {      int v1;      v1 = a1 % 4; -    QueueTilesetAnimDma(gTilesetAnimTable_Cave[v1], (u8 *)(BG_VRAM + 0x5400), 0x80); +    QueueTilesetAnimDma(gTilesetAnimTable_Cave[v1], BG_TILE_ADDR(168), 0x80);  }  static void sub_8073830(u16 a1, u8 a2) @@ -543,7 +543,7 @@ static void sub_8073868(u16 a1)  {      int v1;      v1 = a1 % 4; -    QueueTilesetAnimDma(gTilesetAnimTable_Cave[v1], (u8 *)(BG_VRAM + 0x7400), 0x80); +    QueueTilesetAnimDma(gTilesetAnimTable_Cave[v1], BG_TILE_ADDR(232), 0x80);  }  static void sub_8073890(u16 a1) @@ -590,41 +590,41 @@ static void sub_8073904(u16 a1)  {      int v1;      v1 = a1 % 2; -    QueueTilesetAnimDma(gTilesetAnimTable_Building[v1], (u8 *)(BG_VRAM + 0x3e00), 0x80); +    QueueTilesetAnimDma(gTilesetAnimTable_Building[v1], BG_TILE_ADDR(124), 0x80);  }  static void sub_807392C(u16 a1)  {      int v1;      v1 = a1 % 3; -    QueueTilesetAnimDma(gTilesetAnimTable_SootopolisGym_0[v1], (u8 *)(BG_VRAM + 0x7e00), 0x180); -    QueueTilesetAnimDma(gTilesetAnimTable_SootopolisGym_1[v1], (u8 *)(BG_VRAM + 0x7a00), 0x280); +    QueueTilesetAnimDma(gTilesetAnimTable_SootopolisGym_0[v1], BG_TILE_ADDR(252), 0x180); +    QueueTilesetAnimDma(gTilesetAnimTable_SootopolisGym_1[v1], BG_TILE_ADDR(244), 0x280);  }  static void sub_8073974(u16 a1)  {      int v1;      v1 = a1 % 4; -    QueueTilesetAnimDma(gTilesetAnimTable_EliteFour_0[v1], (u8 *)(BG_VRAM + 0x7f00), 0x20); +    QueueTilesetAnimDma(gTilesetAnimTable_EliteFour_0[v1], BG_TILE_ADDR(254), 0x20);  }  static void sub_807399C(u16 a1)  {      int v1;      v1 = a1 % 2; -    QueueTilesetAnimDma(gTilesetAnimTable_EliteFour_1[v1], (u8 *)(BG_VRAM + 0x7c00), 0x80); +    QueueTilesetAnimDma(gTilesetAnimTable_EliteFour_1[v1], BG_TILE_ADDR(248), 0x80);  }  static void sub_80739C4(u16 a1)  {      int v1;      v1 = a1 % 2; -    QueueTilesetAnimDma(gTilesetAnimTable_MauvilleGym[v1], (u8 *)(BG_VRAM + 0x5200), 0x200); +    QueueTilesetAnimDma(gTilesetAnimTable_MauvilleGym[v1], BG_TILE_ADDR(164), 0x200);  }  static void sub_80739EC(u16 a1)  {      int v1;      v1 = a1 % 2; -    QueueTilesetAnimDma(gTilesetAnimTable_BikeShop[v1], (u8 *)(BG_VRAM + 0x7e00), 0x120); +    QueueTilesetAnimDma(gTilesetAnimTable_BikeShop[v1], BG_TILE_ADDR(252), 0x120);  } diff --git a/src/engine/trade.c b/src/engine/trade.c index 8bc293b95..8fad94be8 100644 --- a/src/engine/trade.c +++ b/src/engine/trade.c @@ -112,7 +112,6 @@ struct TradeEwramSubstruct {      /*0x0087*/ u8 unk_0087;      /*0x0088*/ u8 filler_0088[2];      /*0x008a*/ u8 unk_008a; -    /*0x008b*/ u8 unk_008b;      /*0x008c*/ u16 linkData[20];      /*0x00b4*/ u8 unk_00b4;      /*0x00b5*/ u8 unk_00b5[11]; @@ -1706,8 +1705,11 @@ static void sub_8048C70(void)  static void nullsub_5(u8 a0, u8 a1) {} -static void sub_8048D24(u8 *dest, const u8 *src, u32 size) +// why not just use memcpy? +static void Trade_Memcpy(void *dataDest, void *dataSrc, u32 size)  { +    u8 *dest = dataDest; +    u8 *src = dataSrc;      int i;      for (i = 0; i < size; i ++) dest[i] = src[i];  } @@ -1724,7 +1726,7 @@ static bool8 sub_8048D44(void)      switch (gUnknown_03004824->unk_0075)      {          case  0: -            sub_8048D24(gBlockSendBuffer, (const u8 *)&gPlayerParty[0], 2 * sizeof(struct Pokemon)); +            Trade_Memcpy(gBlockSendBuffer, &gPlayerParty[0], 2 * sizeof(struct Pokemon));              gUnknown_03004824->unk_0075 ++;              break;          case  1: @@ -1751,13 +1753,13 @@ static bool8 sub_8048D44(void)          case  3:              if (GetBlockReceivedStatus() == 3)              { -                sub_8048D24((u8 *)&gEnemyParty[0], (const u8 *)gBlockRecvBuffer[mpId ^ 1], 2 * sizeof(struct Pokemon)); +                Trade_Memcpy(&gEnemyParty[0], gBlockRecvBuffer[mpId ^ 1], 2 * sizeof(struct Pokemon));                  ResetBlockReceivedFlags();                  gUnknown_03004824->unk_0075 ++;              }              break;          case  4: -            sub_8048D24(gBlockSendBuffer, (const u8 *)&gPlayerParty[2], 2 * sizeof(struct Pokemon)); +            Trade_Memcpy(gBlockSendBuffer, &gPlayerParty[2], 2 * sizeof(struct Pokemon));              gUnknown_03004824->unk_0075 ++;              break;          case  5: @@ -1770,13 +1772,13 @@ static bool8 sub_8048D44(void)          case  6:              if (GetBlockReceivedStatus() == 3)              { -                sub_8048D24((u8 *)&gEnemyParty[2], (const u8 *)gBlockRecvBuffer[mpId ^ 1], 2 * sizeof(struct Pokemon)); +                Trade_Memcpy(&gEnemyParty[2], gBlockRecvBuffer[mpId ^ 1], 2 * sizeof(struct Pokemon));                  ResetBlockReceivedFlags();                  gUnknown_03004824->unk_0075 ++;              }              break;          case  7: -            sub_8048D24(gBlockSendBuffer, (const u8 *)&gPlayerParty[4], 2 * sizeof(struct Pokemon)); +            Trade_Memcpy(gBlockSendBuffer, &gPlayerParty[4], 2 * sizeof(struct Pokemon));              gUnknown_03004824->unk_0075 ++;              break;          case  8: @@ -1789,13 +1791,13 @@ static bool8 sub_8048D44(void)          case  9:              if (GetBlockReceivedStatus() == 3)              { -                sub_8048D24((u8 *)&gEnemyParty[4], (const u8 *)gBlockRecvBuffer[mpId ^ 1], 2 * sizeof(struct Pokemon)); +                Trade_Memcpy(&gEnemyParty[4], gBlockRecvBuffer[mpId ^ 1], 2 * sizeof(struct Pokemon));                  ResetBlockReceivedFlags();                  gUnknown_03004824->unk_0075 ++;              }              break;          case 10: -            sub_8048D24(gBlockSendBuffer, (const u8 *)&gSaveBlock1.mail[0], 6 * sizeof(struct MailStruct) + 4); +            Trade_Memcpy(gBlockSendBuffer, &gSaveBlock1.mail[0], 6 * sizeof(struct MailStruct) + 4);              gUnknown_03004824->unk_0075 ++;              break;          case 11: @@ -1808,13 +1810,13 @@ static bool8 sub_8048D44(void)          case 12:              if (GetBlockReceivedStatus() == 3)              { -                sub_8048D24((u8 *)&gUnknown_02029700[0], (const u8 *)gBlockRecvBuffer[mpId ^ 1], 6 * sizeof(struct MailStruct)); +                Trade_Memcpy(&gUnknown_02029700[0], gBlockRecvBuffer[mpId ^ 1], 6 * sizeof(struct MailStruct));                  ResetBlockReceivedFlags();                  gUnknown_03004824->unk_0075 ++;              }              break;          case 13: -            sub_8048D24(gBlockSendBuffer, (const u8 *)gSaveBlock1.giftRibbons, 11); +            Trade_Memcpy(gBlockSendBuffer, gSaveBlock1.giftRibbons, 11);              gUnknown_03004824->unk_0075 ++;              break;          case 14: @@ -1827,7 +1829,7 @@ static bool8 sub_8048D44(void)          case 15:              if (GetBlockReceivedStatus() == 3)              { -                sub_8048D24((u8 *)gUnknown_03004824->unk_00b5, (const u8 *)gBlockRecvBuffer[mpId ^ 1], 11); +                Trade_Memcpy(gUnknown_03004824->unk_00b5, gBlockRecvBuffer[mpId ^ 1], 11);                  ResetBlockReceivedFlags();                  gUnknown_03004824->unk_0075 ++;              } @@ -1915,7 +1917,7 @@ static void sub_80491E4(u8 mpId, u8 status)                  gUnknown_03004824->unk_007b = 8;                  break;              case 0xdddd: -                gUnknown_03004824->unk_008a = ((u8 *)gBlockRecvBuffer[0])[1 * sizeof(u16)] + 6; +                gUnknown_03004824->unk_008a = gBlockRecvBuffer[0][1] + 6;                  sub_8049E9C(gUnknown_03004824->tradeMenuCursorPosition);                  sub_8049E9C(gUnknown_03004824->unk_008a);                  gUnknown_03004824->unk_007b = 7; @@ -5416,7 +5418,6 @@ static void sub_804D948(u8 whichPlayerMon, u8 whichInGameTrade)      struct MailStruct mail;      u8 metLocation = 0xFE;      u8 isMail; -    u8 *item;      struct Pokemon *pokemon = &gEnemyParty[0];      CreateMon(pokemon, inGameTrade->species, level, 32, TRUE, inGameTrade->personality, TRUE, inGameTrade->otId); @@ -5451,8 +5452,7 @@ static void sub_804D948(u8 whichPlayerMon, u8 whichInGameTrade)          }          else          { -            item = (u8 *)&inGameTrade->heldItem; -            SetMonData(pokemon, MON_DATA_HELD_ITEM, item); +            SetMonData(pokemon, MON_DATA_HELD_ITEM, &inGameTrade->heldItem);          }      }      CalculateMonStats(&gEnemyParty[0]); @@ -5705,7 +5705,7 @@ void sub_804E22C(void)  {      const u16 *src;      u16 *dest; -    LZDecompressVram(gUnknown_08D00000, (u8 *)VRAM); +    LZDecompressVram(gUnknown_08D00000, (void *)VRAM);      CpuCopy16(gUnknown_08D00524, ewram, 0x1000);      src = (const u16 *)ewram;      dest = BG_SCREEN_ADDR(5); | 
