[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