fyb/calcurse-edge
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use do-while loops when reading start time and duration in apoint_add().

Browse Source 
Using do-while loops seems more appropriate here as "item_time" hasn't
even been read when either of the loops are entered. Spotted by
clang-analyzer.

Signed-off-by: Lukas Fleischer <calcurse@cryptocrack.de>
This commit is contained in:
Lukas Fleischer 2011-04-11 17:01:47 +02:00
parent 3a4431e568
commit 57b62a1e49
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/apoint.c
View File

@ -182,7 +182,7 @@ apoint_add (void)
int is_appointment = 1;
/* Get the starting time */
while (check_time (item_time) != 1)
do
{
status_mesg (mesg_1, "");
if (getstring (win[STA].p, item_time, LTIME, 0, 1) != GETSTRING_ESC)
@ -203,6 +203,8 @@ apoint_add (void)
else
return;
}
while (check_time (item_time) != 1);
/*
* Check if an event or appointment is entered,
* depending on the starting time, and record the
@ -211,7 +213,7 @@ apoint_add (void)
if (is_appointment)
{ /* Get the appointment duration */
item_time[0] = '\0';
while (check_time (item_time) == 0)
do
{
status_mesg (mesg_2, "");
if (getstring (win[STA].p, item_time, LTIME, 0, 1) != GETSTRING_VALID)
@ -241,6 +243,7 @@ apoint_add (void)
}
}
}
while (check_time (item_time) == 0);
}
else /* Insert the event Id */
Id = 1;