[MPlayer-dev-eng] text_font_scale_factor

Anatoli Marinov anatoli.marinov at gmail.com
Mon Jul 9 11:25:39 CEST 2007


Thanks Alban,
See the attacked file. It has all fixes.

Regards
Toli

2007/7/9, Anatoli Marinov <anatoli.marinov at gmail.com>:
> Hello,
> Thanks for your hints.
>
> > > +            M_PROPERTY_CLAMP(prop, text_font_scale_factor);
> > > +            force_load_font = 1;
> > > +            return M_PROPERTY_OK;
> > > +        default:
> > > +            return m_property_float_ro(prop, action, arg,
> > > text_font_scale_factor);
> > > +    }
> > > +
> > > +    return M_PROPERTY_NOT_IMPLEMENTED;
> >
> > This is useless, it can't be reached. Remove it or replace it with the
> > default case code (i tend to prefer the latter but this is just taste).
> >
> > > +}
>
> What do you mean there is useless?
>
> Toli
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: sub_scale.patch
Type: application/octet-stream
Size: 5860 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20070709/22f11f3d/attachment.obj>


More information about the MPlayer-dev-eng mailing list