Add link to Context LookupFlag discussion
https://github.com/harfbuzz/harfbuzz/discussions/3538
This commit is contained in:
parent
af9eb6850a
commit
a85461b9b6
|
@ -1457,11 +1457,10 @@ static inline void apply_lookup (hb_ot_apply_context_t *c,
|
||||||
* NOT the one after it.
|
* NOT the one after it.
|
||||||
*
|
*
|
||||||
* - If buffer length was decreased by n, it does not necessarily
|
* - If buffer length was decreased by n, it does not necessarily
|
||||||
* mean that n match positions where removed, as there might
|
* mean that n match positions where removed, as there recursed-to
|
||||||
* have been marks and default-ignorables in the sequence. We
|
* lookup might had a different LookupFlag. Here's a constructed
|
||||||
* should instead drop match positions between current-position
|
* case of that:
|
||||||
* and current-position + n instead. Though, am not sure which
|
* https://github.com/harfbuzz/harfbuzz/discussions/3538
|
||||||
* one is better. Both cases have valid uses. Sigh.
|
|
||||||
*
|
*
|
||||||
* It should be possible to construct tests for both of these cases.
|
* It should be possible to construct tests for both of these cases.
|
||||||
*/
|
*/
|
||||||
|
|
Loading…
Reference in New Issue