MPEGGroup / OpenFontFormat

Official MPEG repository to discuss issues on Open Font Format (ISO/IEC 14496-22)
32 stars 6 forks source link

AMD2 WD, new additions in subclause 7.2.3 - VarFixed record #33

Closed vlevantovsky closed 3 years ago

vlevantovsky commented 4 years ago

VarFixed record is intended to be used for variation of floating point values, such as variation space coordinates, defined in Fixed (16.16) format. While the actual value is defined as Fixed (an equivalent representation of 32-bit unsigned integer), both inner and outer variation indices are defined as 'uint16' values. NEEDS REVIEW.

drott commented 4 years ago

@vlevantovsky do you mean this issue to clarify how to vary a 16.16 fixed point number? As discussed here: https://github.com/googlefonts/colr-gradients-spec/issues/29 ?

vlevantovsky commented 4 years ago

@drott Thank you Dominik, and yes - the discussion you linked to (as part of the original draft) is exactly what I expected to happen at some point. The resolution of the technical issues with VarFixed (and 32-bit value variations in general) would need to be brought back to the AHG as part of the submission for WD proposed updates.

PeterConstable commented 4 years ago

The outer/inner indices are indices into ItemVariationStore arrays. These are intentionally 16-bit. The size of the item being varied is orthogonal to the size of the indices. The current draft text is correct as is.

vlevantovsky commented 3 years ago

I believe this issue was resolved, closing.