eden-emu/FFmpeg
20
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Refactor libavutil/parseutils.c

Browse source 
All tests were in the main method which produces a long main. Now, each test
is in his own method.

I think this produces a more clear code and follows more with the main
priority of FFmpeg "simplicity and small code size"

Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
This commit is contained in:
Petru Rares Sincraian 2016-03-23 15:32:56 +00:00 • committed by Michael Niedermayer
parent fe3de6bc62
commit 68e5976543
1 changed files with 177 additions and 165 deletions
Download patch file Download diff file Expand all files Collapse all files

38
libavutil/parseutils.c
View file

@ -749,10 +749,8 @@ static uint32_t av_get_random_seed_deterministic(void)
return randomv = randomv * 1664525 + 1013904223; return randomv = randomv * 1664525 + 1013904223;
} }
int main(void) static void test_av_parse_video_rate(void)
{ {
printf("Testing av_parse_video_rate()\n");
{
int i; int i;
static const char *const rates[] = { static const char *const rates[] = {
"-inf", "-inf",
@ -789,10 +787,10 @@ int main(void)
printf("'%s' -> %d/%d %s\n", printf("'%s' -> %d/%d %s\n",
rates[i], q.num, q.den, ret ? "ERROR" : "OK"); rates[i], q.num, q.den, ret ? "ERROR" : "OK");
} }
} }
printf("\nTesting av_parse_color()\n"); static void test_av_parse_color(void)
{ {
int i; int i;
uint8_t rgba[4]; uint8_t rgba[4];
static const char *const color_names[] = { static const char *const color_names[] = {
@ -843,10 +841,10 @@ int main(void)
else else
printf("%s -> error\n", color_names[i]); printf("%s -> error\n", color_names[i]);
} }
} }
printf("\nTesting av_small_strptime()\n"); static void test_av_small_strptime(void)
{ {
int i; int i;
struct tm tm = { 0 }; struct tm tm = { 0 };
struct fmt_timespec_entry { struct fmt_timespec_entry {
@ -872,10 +870,10 @@ int main(void)
printf("error\n"); printf("error\n");
} }
} }
} }
printf("\nTesting av_parse_time()\n"); static void test_av_parse_time(void)
{ {
int i; int i;
int64_t tv; int64_t tv;
time_t tvi; time_t tvi;
@ -922,7 +920,21 @@ int main(void)
printf("%+21"PRIi64"\n", tv); printf("%+21"PRIi64"\n", tv);
} }
} }
} }
int main(void)
{
printf("Testing av_parse_video_rate()\n");
test_av_parse_video_rate();
printf("\nTesting av_parse_color()\n");
test_av_parse_color();
printf("\nTesting av_small_strptime()\n");
test_av_small_strptime();
printf("\nTesting av_parse_time()\n");
test_av_parse_time();
return 0; return 0;
} }