fix infinite loop in getRangeGlyphClass

[LabhanshAgrawal]

Fixes #23

Just adding search++; similar to

font-ligatures/src/processors/substitution.ts

Line 29 in c9ad109

search++;

However, I think both the places should also have something like

start = search;
currentClass = clazz;

Can add that too if you think it sounds correct.

[boldandbrad]

Can this please be merged?

[Tyriar]

Thanks @LabhanshAgrawal looks good, wasn't watching this repo

[Tyriar]

Can add that too if you think it sounds correct.

Would like to minimize the changes as much as possible as I'm not familiar with the code, if you could come up with a test case that covers this that would be a nice addition.

[LabhanshAgrawal]

Can add that too if you think it sounds correct.

Would like to minimize the changes as much as possible as I'm not familiar with the code, if you could come up with a test case that covers this that would be a nice addition.

I'll have to go through the full code to be 100% sure, but based on how this type of code is written generally, that's what I got at. Will check it out, didn't know you could merge pr's here, that's great.

[princjef]

🎉 This PR is included in version 1.4.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀