walkero
/
harfbuzz
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

minor, add unlikely around lookup_limit_exceeded checks 

addresses https://github.com/harfbuzz/harfbuzz/pull/2294#issuecomment-618022488
This commit is contained in:
Ebrahim Byagowi 2020-04-23 11:23:54 +04:30 committed by GitHub
parent 3e686d2459
commit 0c65a232fa
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/hb-ot-layout-gsubgpos.hh
View File

@ -1400,7 +1400,7 @@ struct Rule
void closure (hb_closure_context_t *c, ContextClosureLookupContext &lookup_context) const void closure (hb_closure_context_t *c, ContextClosureLookupContext &lookup_context) const
{ {
if (c->lookup_limit_exceeded ()) return; if (unlikely (c->lookup_limit_exceeded ())) return;
const UnsizedArrayOf<LookupRecord> &lookupRecord = StructAfter<UnsizedArrayOf<LookupRecord>> const UnsizedArrayOf<LookupRecord> &lookupRecord = StructAfter<UnsizedArrayOf<LookupRecord>>
(inputZ.as_array ((inputCount ? inputCount - 1 : 0))); (inputZ.as_array ((inputCount ? inputCount - 1 : 0)));
@ -1412,7 +1412,7 @@ struct Rule
void closure_lookups (hb_closure_lookups_context_t *c) const void closure_lookups (hb_closure_lookups_context_t *c) const
{ {
if (c->lookup_limit_exceeded ()) return; if (unlikely (c->lookup_limit_exceeded ())) return;
const UnsizedArrayOf<LookupRecord> &lookupRecord = StructAfter<UnsizedArrayOf<LookupRecord>> const UnsizedArrayOf<LookupRecord> &lookupRecord = StructAfter<UnsizedArrayOf<LookupRecord>>
(inputZ.as_array (inputCount ? inputCount - 1 : 0)); (inputZ.as_array (inputCount ? inputCount - 1 : 0));
@ -1492,7 +1492,7 @@ struct RuleSet
void closure (hb_closure_context_t *c, void closure (hb_closure_context_t *c,
ContextClosureLookupContext &lookup_context) const ContextClosureLookupContext &lookup_context) const
{ {
if (c->lookup_limit_exceeded ()) return; if (unlikely (c->lookup_limit_exceeded ())) return;
return return
+ hb_iter (rule) + hb_iter (rule)
@ -1503,7 +1503,7 @@ struct RuleSet
void closure_lookups (hb_closure_lookups_context_t *c) const void closure_lookups (hb_closure_lookups_context_t *c) const
{ {
if (c->lookup_limit_exceeded ()) return; if (unlikely (c->lookup_limit_exceeded ())) return;
return return
+ hb_iter (rule) + hb_iter (rule)
@ -2092,7 +2092,7 @@ struct ChainRule
void closure (hb_closure_context_t *c, void closure (hb_closure_context_t *c,
ChainContextClosureLookupContext &lookup_context) const ChainContextClosureLookupContext &lookup_context) const
{ {
if (c->lookup_limit_exceeded ()) return; if (unlikely (c->lookup_limit_exceeded ())) return;
const HeadlessArrayOf<HBUINT16> &input = StructAfter<HeadlessArrayOf<HBUINT16>> (backtrack); const HeadlessArrayOf<HBUINT16> &input = StructAfter<HeadlessArrayOf<HBUINT16>> (backtrack);
const ArrayOf<HBUINT16> &lookahead = StructAfter<ArrayOf<HBUINT16>> (input); const ArrayOf<HBUINT16> &lookahead = StructAfter<ArrayOf<HBUINT16>> (input);
@ -2107,7 +2107,7 @@ struct ChainRule
void closure_lookups (hb_closure_lookups_context_t *c) const void closure_lookups (hb_closure_lookups_context_t *c) const
{ {
if (c->lookup_limit_exceeded ()) return; if (unlikely (c->lookup_limit_exceeded ())) return;
const HeadlessArrayOf<HBUINT16> &input = StructAfter<HeadlessArrayOf<HBUINT16>> (backtrack); const HeadlessArrayOf<HBUINT16> &input = StructAfter<HeadlessArrayOf<HBUINT16>> (backtrack);
const ArrayOf<HBUINT16> &lookahead = StructAfter<ArrayOf<HBUINT16>> (input); const ArrayOf<HBUINT16> &lookahead = StructAfter<ArrayOf<HBUINT16>> (input);
@ -2275,7 +2275,7 @@ struct ChainRuleSet
} }
void closure (hb_closure_context_t *c, ChainContextClosureLookupContext &lookup_context) const void closure (hb_closure_context_t *c, ChainContextClosureLookupContext &lookup_context) const
{ {
if (c->lookup_limit_exceeded ()) return; if (unlikely (c->lookup_limit_exceeded ())) return;
return return
+ hb_iter (rule) + hb_iter (rule)
@ -2286,7 +2286,7 @@ struct ChainRuleSet
void closure_lookups (hb_closure_lookups_context_t *c) const void closure_lookups (hb_closure_lookups_context_t *c) const
{ {
if (c->lookup_limit_exceeded ()) return; if (unlikely (c->lookup_limit_exceeded ())) return;
return return
+ hb_iter (rule) + hb_iter (rule)