Allow to display all builtin Wine video codecs inICCompressorChoose

Andreas Mohr andi at rhlx01.fht-esslingen.de
Fri Nov 25 04:18:57 CST 2005


Hi,

On Fri, Nov 25, 2005 at 06:12:03PM +0800, Dmitry Timoshkov wrote:
> "Andreas Mohr" <andi at rhlx01.fht-esslingen.de> wrote:
> 
> > > All the codecs except ir32_32 return dwVersionICM set to ICVERSION (0x0104),
> > > so that looks like a common practice. However dwVersion field doesn't look
> > > like a reasonably set at all. I decided to follow msrle32 and set to 0x0104
> > > as well.
> > Sorry, bad decision IMHO.
> > Everything indicates that ICVERSION is always used for dwVersionICM
> > only, so you shouldn't give the impression that it could be used for
> > dwVersion, too.
> > IMHO it's much better to choose an arbitrary value for dwVersion with
> > a FIXME comment behind it.
> 
> I don't see how a FIXME could improve Windows drivers behaviour to return
> arbitrary values in the dwVersion field. IMHO it's much better to use
> a definitive and reasonable one, and ICVERSION qualifies for that.
"Definitive" yes, "reasonable" simply NO.
(it is not customary to abuse ICVERSION for that, so one shouldn't
do it; just doing Google "dwVersion ICVERSION" will make it obvious)

Andreas



More information about the wine-devel mailing list