[MPlayer-dev-eng] text_font_scale_factor

[Vladimir Voroshilov]

Hi, Anatoli

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?
>
All possible cases are already catched by "default: return ...."



-- 
Regards,
Vladimir Voroshilov     mailto:voroshil at gmail.com
JID: voroshil at gmail.com, voroshil at jabber.ru
ICQ: 95587719