Skip to content

Commit f85ed94

Browse files
committed
Refactor
1 parent 604a5b4 commit f85ed94

File tree

6 files changed

+102
-146
lines changed

6 files changed

+102
-146
lines changed

app/app.c

Lines changed: 57 additions & 79 deletions
Original file line numberDiff line numberDiff line change
@@ -750,53 +750,46 @@ static void HandleVox(void)
750750
return;
751751
#endif
752752

753-
if (gVoxResumeCountdown == 0)
754-
{
753+
if (gVoxResumeCountdown == 0) {
755754
if (gVoxPauseCountdown)
756755
return;
757756
}
758-
else
759-
{
757+
else {
760758
g_VOX_Lost = false;
761759
gVoxPauseCountdown = 0;
762760
}
763761

764-
#ifdef ENABLE_FMRADIO
765-
if (gFmRadioMode)
766-
return;
767-
#endif
762+
#ifdef ENABLE_FMRADIO
763+
if (gFmRadioMode)
764+
return;
765+
#endif
768766

769767
if (gCurrentFunction == FUNCTION_RECEIVE || gCurrentFunction == FUNCTION_MONITOR)
770768
return;
771769

772770
if (gScanStateDir != SCAN_OFF)
773771
return;
774772

775-
if (gVOX_NoiseDetected)
776-
{
773+
if (gVOX_NoiseDetected) {
777774
if (g_VOX_Lost)
778775
gVoxStopCountdown_10ms = vox_stop_count_down_10ms;
779776
else if (gVoxStopCountdown_10ms == 0)
780777
gVOX_NoiseDetected = false;
781778

782-
if (gCurrentFunction == FUNCTION_TRANSMIT && !gPttIsPressed && !gVOX_NoiseDetected)
783-
{
784-
if (gFlagEndTransmission)
785-
{
779+
if (gCurrentFunction == FUNCTION_TRANSMIT && !gPttIsPressed && !gVOX_NoiseDetected) {
780+
if (gFlagEndTransmission) {
786781
//if (gCurrentFunction != FUNCTION_FOREGROUND)
787782
FUNCTION_Select(FUNCTION_FOREGROUND);
788783
}
789-
else
790-
{
784+
else {
791785
APP_EndTransmission();
792786

793-
if (gEeprom.REPEATER_TAIL_TONE_ELIMINATION == 0)
794-
{
787+
if (gEeprom.REPEATER_TAIL_TONE_ELIMINATION == 0) {
795788
//if (gCurrentFunction != FUNCTION_FOREGROUND)
796789
FUNCTION_Select(FUNCTION_FOREGROUND);
797790
}
798791
else
799-
gRTTECountdown = gEeprom.REPEATER_TAIL_TONE_ELIMINATION * 10;
792+
gRTTECountdown_10ms = gEeprom.REPEATER_TAIL_TONE_ELIMINATION * 10;
800793
}
801794

802795
gUpdateStatus = true;
@@ -806,15 +799,13 @@ static void HandleVox(void)
806799
return;
807800
}
808801

809-
if (g_VOX_Lost)
810-
{
802+
if (g_VOX_Lost) {
811803
gVOX_NoiseDetected = true;
812804

813805
if (gCurrentFunction == FUNCTION_POWER_SAVE)
814806
FUNCTION_Select(FUNCTION_FOREGROUND);
815807

816-
if (gCurrentFunction != FUNCTION_TRANSMIT && !SerialConfigInProgress())
817-
{
808+
if (gCurrentFunction != FUNCTION_TRANSMIT && !SerialConfigInProgress()) {
818809
#ifdef ENABLE_DTMF_CALLING
819810
gDTMF_ReplyState = DTMF_REPLY_NONE;
820811
#endif
@@ -1156,8 +1147,7 @@ void APP_TimeSlice10ms(void)
11561147
#endif
11571148
}
11581149

1159-
if (gUpdateDisplay)
1160-
{
1150+
if (gUpdateDisplay) {
11611151
gUpdateDisplay = false;
11621152
GUI_DisplayScreen();
11631153
}
@@ -1184,65 +1174,56 @@ void APP_TimeSlice10ms(void)
11841174
gVoxPauseCountdown--;
11851175
#endif
11861176

1187-
if (gCurrentFunction == FUNCTION_TRANSMIT)
1188-
{
1189-
#ifdef ENABLE_ALARM
1190-
if (gAlarmState == ALARM_STATE_TXALARM || gAlarmState == ALARM_STATE_SITE_ALARM)
1191-
{
1192-
uint16_t Tone;
1177+
if (gCurrentFunction == FUNCTION_TRANSMIT) {
1178+
#ifdef ENABLE_ALARM
1179+
if (gAlarmState == ALARM_STATE_TXALARM || gAlarmState == ALARM_STATE_SITE_ALARM) {
1180+
uint16_t Tone;
11931181

1194-
gAlarmRunningCounter++;
1195-
gAlarmToneCounter++;
1182+
gAlarmRunningCounter++;
1183+
gAlarmToneCounter++;
11961184

1197-
Tone = 500 + (gAlarmToneCounter * 25);
1198-
if (Tone > 1500)
1199-
{
1200-
Tone = 500;
1201-
gAlarmToneCounter = 0;
1202-
}
1185+
Tone = 500 + (gAlarmToneCounter * 25);
1186+
if (Tone > 1500) {
1187+
Tone = 500;
1188+
gAlarmToneCounter = 0;
1189+
}
12031190

1204-
BK4819_SetScrambleFrequencyControlWord(Tone);
1191+
BK4819_SetScrambleFrequencyControlWord(Tone);
12051192

1206-
if (gEeprom.ALARM_MODE == ALARM_MODE_TONE && gAlarmRunningCounter == 512)
1207-
{
1208-
gAlarmRunningCounter = 0;
1193+
if (gEeprom.ALARM_MODE == ALARM_MODE_TONE && gAlarmRunningCounter == 512) {
1194+
gAlarmRunningCounter = 0;
12091195

1210-
if (gAlarmState == ALARM_STATE_TXALARM)
1211-
{
1212-
gAlarmState = ALARM_STATE_SITE_ALARM;
1196+
if (gAlarmState == ALARM_STATE_TXALARM) {
1197+
gAlarmState = ALARM_STATE_SITE_ALARM;
12131198

1214-
RADIO_EnableCxCSS();
1215-
BK4819_SetupPowerAmplifier(0, 0);
1216-
BK4819_ToggleGpioOut(BK4819_GPIO1_PIN29_PA_ENABLE, false);
1217-
BK4819_Enable_AfDac_DiscMode_TxDsp();
1218-
BK4819_ToggleGpioOut(BK4819_GPIO5_PIN1_RED, false);
1199+
RADIO_EnableCxCSS();
1200+
BK4819_SetupPowerAmplifier(0, 0);
1201+
BK4819_ToggleGpioOut(BK4819_GPIO1_PIN29_PA_ENABLE, false);
1202+
BK4819_Enable_AfDac_DiscMode_TxDsp();
1203+
BK4819_ToggleGpioOut(BK4819_GPIO5_PIN1_RED, false);
12191204

1220-
GUI_DisplayScreen();
1221-
}
1222-
else
1223-
{
1224-
gAlarmState = ALARM_STATE_TXALARM;
1205+
GUI_DisplayScreen();
1206+
}
1207+
else {
1208+
gAlarmState = ALARM_STATE_TXALARM;
12251209

1226-
GUI_DisplayScreen();
1210+
GUI_DisplayScreen();
12271211

1228-
BK4819_ToggleGpioOut(BK4819_GPIO5_PIN1_RED, true);
1229-
RADIO_SetTxParameters();
1230-
BK4819_TransmitTone(true, 500);
1231-
SYSTEM_DelayMs(2);
1232-
AUDIO_AudioPathOn();
1212+
BK4819_ToggleGpioOut(BK4819_GPIO5_PIN1_RED, true);
1213+
RADIO_SetTxParameters();
1214+
BK4819_TransmitTone(true, 500);
1215+
SYSTEM_DelayMs(2);
1216+
AUDIO_AudioPathOn();
12331217

1234-
gEnableSpeaker = true;
1235-
gAlarmToneCounter = 0;
1236-
}
1218+
gEnableSpeaker = true;
1219+
gAlarmToneCounter = 0;
12371220
}
12381221
}
1239-
#endif
1240-
1222+
}
1223+
#endif
12411224
// repeater tail tone elimination
1242-
if (gRTTECountdown > 0)
1243-
{
1244-
if (--gRTTECountdown == 0)
1245-
{
1225+
if (gRTTECountdown_10ms > 0) {
1226+
if (--gRTTECountdown_10ms == 0) {
12461227
//if (gCurrentFunction != FUNCTION_FOREGROUND)
12471228
FUNCTION_Select(FUNCTION_FOREGROUND);
12481229

@@ -1253,11 +1234,9 @@ void APP_TimeSlice10ms(void)
12531234
}
12541235

12551236
#ifdef ENABLE_FMRADIO
1256-
if (gFmRadioMode && gFM_RestoreCountdown_10ms > 0)
1257-
{
1258-
if (--gFM_RestoreCountdown_10ms == 0)
1259-
{ // switch back to FM radio mode
1260-
FM_Start();
1237+
if (gFmRadioMode && gFM_RestoreCountdown_10ms > 0) {
1238+
if (--gFM_RestoreCountdown_10ms == 0) {
1239+
FM_Start(); // switch back to FM radio mode
12611240
GUI_SelectNextDisplay(DISPLAY_FM);
12621241
}
12631242
}
@@ -1267,8 +1246,7 @@ void APP_TimeSlice10ms(void)
12671246
SCANNER_TimeSlice10ms();
12681247

12691248
#ifdef ENABLE_AIRCOPY
1270-
if (gScreenToDisplay == DISPLAY_AIRCOPY && gAircopyState == AIRCOPY_TRANSFER && gAirCopyIsSendMode == 1)
1271-
{
1249+
if (gScreenToDisplay == DISPLAY_AIRCOPY && gAircopyState == AIRCOPY_TRANSFER && gAirCopyIsSendMode == 1) {
12721250
if (!AIRCOPY_SendMessage()) {
12731251
GUI_DisplayScreen();
12741252
}
@@ -1780,7 +1758,7 @@ static void ProcessKey(KEY_Code_t Key, bool bKeyPressed, bool bKeyHeld)
17801758
if (gEeprom.REPEATER_TAIL_TONE_ELIMINATION == 0)
17811759
FUNCTION_Select(FUNCTION_FOREGROUND);
17821760
else
1783-
gRTTECountdown = gEeprom.REPEATER_TAIL_TONE_ELIMINATION * 10;
1761+
gRTTECountdown_10ms = gEeprom.REPEATER_TAIL_TONE_ELIMINATION * 10;
17841762

17851763
if (Key == KEY_PTT)
17861764
gPttWasPressed = true;

app/dtmf.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,8 +89,7 @@ void DTMF_SendEndOfTransmission(void)
8989
&& gDTMF_CallState == DTMF_CALL_STATE_NONE
9090
#endif
9191
) { // end-of-tx
92-
if (gEeprom.DTMF_SIDE_TONE)
93-
{
92+
if (gEeprom.DTMF_SIDE_TONE) {
9493
AUDIO_AudioPathOn();
9594
gEnableSpeaker = true;
9695
SYSTEM_DelayMs(60);

0 commit comments

Comments
 (0)