summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/cable_club.c4
-rw-r--r--src/link.c4
-rw-r--r--src/link_rfu_2.c20
-rw-r--r--src/trade.c30
-rw-r--r--src/union_room.c2
5 files changed, 31 insertions, 29 deletions
diff --git a/src/cable_club.c b/src/cable_club.c
index 52856e3a3..801fca241 100644
--- a/src/cable_club.c
+++ b/src/cable_club.c
@@ -173,7 +173,7 @@ static void sub_80809C4(u8 taskId)
gTasks[taskId].data[0]++;
if (gTasks[taskId].data[0] == 10)
{
- sub_800A474(2);
+ Link_PrepareCmd0xCCCC_Rfu0xA100(2);
DestroyTask(taskId);
}
}
@@ -369,7 +369,7 @@ static void Task_LinkupMaster_6(u8 taskId)
sub_800A900(gFieldLinkPlayerCount);
TrainerCard_GenerateCardForLinkPlayer((void*)gBlockSendBuffer);
gTasks[taskId].func = Task_Linkup_6a;
- sub_800A474(2);
+ Link_PrepareCmd0xCCCC_Rfu0xA100(2);
}
}
}
diff --git a/src/link.c b/src/link.c
index 75e6b9927..8b33caf6f 100644
--- a/src/link.c
+++ b/src/link.c
@@ -999,11 +999,11 @@ bool8 SendBlock(u8 unused, const void *src, u16 size)
return InitBlockSend(src, size);
}
-bool8 sub_800A474(u8 blockRequestType)
+bool8 Link_PrepareCmd0xCCCC_Rfu0xA100(u8 blockRequestType)
{
if (gWirelessCommType == 1)
{
- return sub_80FA0F8(blockRequestType);
+ return LinkRfu_PrepareCmd0xA100(blockRequestType);
}
if (gLinkCallback == NULL)
{
diff --git a/src/link_rfu_2.c b/src/link_rfu_2.c
index bedc9a025..f937dfe9b 100644
--- a/src/link_rfu_2.c
+++ b/src/link_rfu_2.c
@@ -1145,7 +1145,7 @@ static void RfuPrepareSendBuffer(u16 command)
break;
case RFU_COMMAND_0xa100:
if (Cmd8000recvIsFinished())
- gSendCmd[1] = Rfu.unk_5a;
+ gSendCmd[1] = Rfu.cmdA100_blockRequestType;
break;
case RFU_COMMAND_0x7700:
case RFU_COMMAND_0x7800:
@@ -1272,9 +1272,9 @@ static void RfuFunc_SendLastBlock(void)
Rfu.RfuFunc = NULL;
}
-bool8 sub_80FA0F8(u8 a0)
+bool8 LinkRfu_PrepareCmd0xA100(u8 blockRequestType)
{
- Rfu.unk_5a = a0;
+ Rfu.cmdA100_blockRequestType = blockRequestType;
RfuPrepareSendBuffer(RFU_COMMAND_0xa100);
return TRUE;
}
@@ -1284,7 +1284,7 @@ static void RfuFunc_End5F00_PowerDownRfu(void)
rfu_clearAllSlot();
rfu_LMAN_powerDownRFU();
gReceivedRemoteLinkPlayers = 0;
- Rfu.unk_ef = 1;
+ Rfu.isShuttingDown = TRUE;
Rfu.RfuFunc = NULL;
}
@@ -1660,7 +1660,7 @@ static void sub_80FA834(u8 taskId)
{
if (Cmd8000recvIsFinished())
{
- Rfu.unk_5a = 0;
+ Rfu.cmdA100_blockRequestType = 0;
RfuPrepareSendBuffer(RFU_COMMAND_0xa100);
gTasks[taskId].data[0]++;
}
@@ -1879,7 +1879,7 @@ bool32 LinkRfuMain1(void)
bool32 retval = FALSE;
Rfu.parentId = 0;
rfu_LMAN_manager_entity(Random());
- if (Rfu.unk_ef == 0)
+ if (!Rfu.isShuttingDown)
{
switch (Rfu.parent_child)
{
@@ -1900,7 +1900,7 @@ bool32 LinkRfuMain1(void)
bool32 LinkRfuMain2(void)
{
bool32 retval = FALSE;
- if (Rfu.unk_ef == 0)
+ if (!Rfu.isShuttingDown)
{
if (Rfu.parent_child == MODE_PARENT)
retval = sub_80F9204();
@@ -2091,7 +2091,7 @@ static void LmanCallback_Parent2(u8 msg, u8 param_count)
case LMAN_MSG_LMAN_API_ERROR_RETURN:
RfuSetErrorStatus(1, msg);
GetLinkmanErrorParams(msg);
- Rfu.unk_ef = 1;
+ Rfu.isShuttingDown = TRUE;
break;
case LMAN_MSG_REQ_API_ERROR:
case LMAN_MSG_WATCH_DOG_TIMER_ERROR:
@@ -2164,7 +2164,7 @@ static void LmanCallback_Child(u8 msg, u8 param_count)
case LMAN_MSG_LMAN_API_ERROR_RETURN:
RfuSetErrorStatus(1, msg);
GetLinkmanErrorParams(msg);
- Rfu.unk_ef = 1;
+ Rfu.isShuttingDown = TRUE;
break;
case LMAN_MSG_REQ_API_ERROR:
case LMAN_MSG_WATCH_DOG_TIMER_ERROR:
@@ -2341,7 +2341,7 @@ static void LmanCallback_Parent(u8 msg, u8 param_count)
case LMAN_MSG_LMAN_API_ERROR_RETURN:
RfuSetErrorStatus(1, msg);
GetLinkmanErrorParams(msg);
- Rfu.unk_ef = 1;
+ Rfu.isShuttingDown = TRUE;
break;
case LMAN_MSG_REQ_API_ERROR:
case LMAN_MSG_WATCH_DOG_TIMER_ERROR:
diff --git a/src/trade.c b/src/trade.c
index f21c88e75..f6c2a7349 100644
--- a/src/trade.c
+++ b/src/trade.c
@@ -24,6 +24,8 @@
#include "constants/songs.h"
#include "constants/moves.h"
+#define NUM_TRADED_GIFT_RIBBONS 11
+
struct TradeMenuResources
{
/*0x0000*/ u8 bg2hofs;
@@ -54,7 +56,7 @@ struct TradeMenuResources
/*0x007F*/ u8 filler_7F;
/*0x0080*/ u16 linkData[20];
/*0x00A8*/ u8 loadUiSpritesState;
- /*0x00A9*/ u8 giftRibbons[11];
+ /*0x00A9*/ u8 giftRibbons[NUM_TRADED_GIFT_RIBBONS];
/*0x00B4*/ u8 filler_B4[0x8D0-0xB4];
/*0x08D0*/ struct {
bool8 active;
@@ -1357,7 +1359,7 @@ static bool8 shedinja_maker_maybe(void)
case 3:
if (id == 0)
{
- sub_800A474(1);
+ Link_PrepareCmd0xCCCC_Rfu0xA100(1);
}
sTradeMenuResourcesPtr->state++;
break;
@@ -1376,71 +1378,71 @@ static bool8 shedinja_maker_maybe(void)
case 7:
if (id == 0)
{
- sub_800A474(1);
+ Link_PrepareCmd0xCCCC_Rfu0xA100(1);
}
sTradeMenuResourcesPtr->state++;
break;
case 8:
if (GetBlockReceivedStatus() == 3)
{
- Trade_Memcpy(&gEnemyParty[2], gBlockRecvBuffer[id ^ 1], 200);
+ Trade_Memcpy(&gEnemyParty[2], gBlockRecvBuffer[id ^ 1], 2 * sizeof(struct Pokemon));
ResetBlockReceivedFlags();
sTradeMenuResourcesPtr->state++;
}
break;
case 9:
- Trade_Memcpy(gBlockSendBuffer, &gPlayerParty[4], 200);
+ Trade_Memcpy(gBlockSendBuffer, &gPlayerParty[4], 2 * sizeof(struct Pokemon));
sTradeMenuResourcesPtr->state++;
break;
case 11:
if (id == 0)
{
- sub_800A474(1);
+ Link_PrepareCmd0xCCCC_Rfu0xA100(1);
}
sTradeMenuResourcesPtr->state++;
break;
case 12:
if (GetBlockReceivedStatus() == 3)
{
- Trade_Memcpy(&gEnemyParty[4], gBlockRecvBuffer[id ^ 1], 200);
+ Trade_Memcpy(&gEnemyParty[4], gBlockRecvBuffer[id ^ 1], 2 * sizeof(struct Pokemon));
ResetBlockReceivedFlags();
sTradeMenuResourcesPtr->state++;
}
break;
case 13:
- Trade_Memcpy(gBlockSendBuffer, gSaveBlock1Ptr->mail, 220);
+ Trade_Memcpy(gBlockSendBuffer, gSaveBlock1Ptr->mail, PARTY_SIZE * sizeof(struct MailStruct) + 4); // why the extra 4 bytes?
sTradeMenuResourcesPtr->state++;
break;
case 15:
if (id == 0)
{
- sub_800A474(3);
+ Link_PrepareCmd0xCCCC_Rfu0xA100(3);
}
sTradeMenuResourcesPtr->state++;
break;
case 16:
if (GetBlockReceivedStatus() == 3)
{
- Trade_Memcpy(gLinkPartnerMail, gBlockRecvBuffer[id ^ 1], 216);
+ Trade_Memcpy(gLinkPartnerMail, gBlockRecvBuffer[id ^ 1], PARTY_SIZE * sizeof(struct MailStruct));
ResetBlockReceivedFlags();
sTradeMenuResourcesPtr->state++;
}
break;
case 17:
- Trade_Memcpy(gBlockSendBuffer, gSaveBlock1Ptr->giftRibbons, 11);
+ Trade_Memcpy(gBlockSendBuffer, gSaveBlock1Ptr->giftRibbons, NUM_TRADED_GIFT_RIBBONS);
sTradeMenuResourcesPtr->state++;
break;
case 19:
if (id == 0)
{
- sub_800A474(4);
+ Link_PrepareCmd0xCCCC_Rfu0xA100(4);
}
sTradeMenuResourcesPtr->state++;
break;
case 20:
if (GetBlockReceivedStatus() == 3)
{
- Trade_Memcpy(sTradeMenuResourcesPtr->giftRibbons, gBlockRecvBuffer[id ^ 1], 11);
+ Trade_Memcpy(sTradeMenuResourcesPtr->giftRibbons, gBlockRecvBuffer[id ^ 1], NUM_TRADED_GIFT_RIBBONS);
ResetBlockReceivedFlags();
sTradeMenuResourcesPtr->state++;
}
@@ -2185,7 +2187,7 @@ static void HandleRedrawTradeMenuOnSide(u8 side)
static u8 GetNicknameStringWidthByPartyAndMonIdx(u8 *dest, u8 whichParty, u8 partyIdx)
{
- u8 nickname[11];
+ u8 nickname[POKEMON_NAME_LENGTH];
if (whichParty == 0)
GetMonData(&gPlayerParty[partyIdx], MON_DATA_NICKNAME, nickname);
else
diff --git a/src/union_room.c b/src/union_room.c
index f24406267..ea9b213ab 100644
--- a/src/union_room.c
+++ b/src/union_room.c
@@ -1789,7 +1789,7 @@ static void Task_ExchangeCards(u8 taskId)
{
case 0:
if (GetMultiplayerId() == 0)
- sub_800A474(2);
+ Link_PrepareCmd0xCCCC_Rfu0xA100(2);
gTasks[taskId].data[0]++;
break;
case 1: