Re: [FFmpeg-devel] [PATCH] fftools/ffprobe: Use proper enum type

2023-08-08 Thread Andreas Rheinhardt
Stefano Sabatini: > On date Sunday 2023-08-06 15:28:00 +0200, Andreas Rheinhardt wrote: >> This is a bit cleaner as int need not be the underlying type >> of an enum if a smaller type can hold all its values. >> Also declare the children_ids array as const as it never changes. >> >> Signed-off-by:

Re: [FFmpeg-devel] [PATCH] fftools/ffprobe: Use proper enum type

2023-08-08 Thread Stefano Sabatini
On date Sunday 2023-08-06 15:28:00 +0200, Andreas Rheinhardt wrote: > This is a bit cleaner as int need not be the underlying type > of an enum if a smaller type can hold all its values. > Also declare the children_ids array as const as it never changes. > > Signed-off-by: Andreas Rheinhardt >

[FFmpeg-devel] [PATCH] fftools/ffprobe: Use proper enum type

2023-08-06 Thread Andreas Rheinhardt
This is a bit cleaner as int need not be the underlying type of an enum if a smaller type can hold all its values. Also declare the children_ids array as const as it never changes. Signed-off-by: Andreas Rheinhardt --- fftools/ffprobe.c | 38 ++ 1 file