Fix requiredFeature stage handling logic
authorBehdad Esfahbod <behdad@behdad.org>
Wed, 2 Mar 2016 21:32:42 +0000 (13:32 -0800)
committerBehdad Esfahbod <behdad@behdad.org>
Wed, 2 Mar 2016 21:32:42 +0000 (13:32 -0800)
commit731a430cd36caabcef04e099b21ebc6daedd536f
tree41eceb454f92732a42cf8cba1c8fc87459bc6e2c
parent68b6296d3333fd38bd5442d2ba087ced481603bc
Fix requiredFeature stage handling logic

Originally the way Jonathan had written this was correct in
"continue"ing:

https://github.com/jfkthame/harfbuzz/commit/35e28c7a733eaffcd9f062b18d7db9fbb3d990fc#diff-ead86a33a5cc9ad7f6e6381031a0baddR199

When I rewrote his patch, I messed it up:

https://github.com/behdad/harfbuzz/commit/da132937989acb4d8ca9bd41c79f98750e7dda30#diff-ead86a33a5cc9ad7f6e6381031a0baddR209

the intended behavior was NOT to set found=TRUE and NOT to continue.
This was resulting in feature_index[table_index] being left unset.
Oops!
src/hb-ot-map.cc