[MPlayer-dev-eng] [PATCH] media-based aspect ratio
Nico Sabbi
nicola_sabbi at fastwebnet.it
Thu Aug 30 23:19:07 CEST 2007
Reimar Döffinger wrote:
> The name makes no sense to me. Unless you have a better idea, how about
> just calling it stream_aspect?
done
> Is at least similar to movie_aspect. It's probably not worth dealing with now
> but this solution isn't very flexible, e.g. different aspect for different -vid wouldn't work...
in this case the stream should report the correct aspect for the
correct id, but atm there are no cases of that kind, so let's keep on
ignoring them for a while
>
>
>>+ case STREAM_CTRL_GET_ASPECT_RATIO:
>>+ {
>>+ *((double *)arg) = (double) (!d->vts_file->vtsi_mat->vts_video_attr.display_aspect_ratio ? 4.0/3.0 : 16.0/9.0);
>
>
> That cast and the () both before and after the = are not really
> possible. I don't really mind the (), but I do mind useless casts.
>
updated
More information about the MPlayer-dev-eng
mailing list