MAINT: Unnecessary character mapping process

[ssjkamei]

This is a fix for the problem that occurred when #2882 was changed.

The string length of characters was checked after conversion by cmap, but after cmap conversion, there is a pattern where the string length is more than one character, and it cannot be measured accurately.

This is necessary, for example, when considering whether to measure the distance from the ligature or the base character corresponding to the ligature in fixing #1351.

The change in handle_tj is because it cannot pass Ruff's check.
Error: PLR0915 Too many statements (nnn > 176)

The following code is only used to get the character code for a space.
However, I think it would be better to split the code into parts for obtaining the character code.
Style changes are considered in another PR.

_, space_code = parse_encoding(cmap[3], space_code)
_, space_code, _ = parse_to_unicode(cmap[3], space_code)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.36%. Comparing base (e825ac0) to head (b25b28f).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2888   +/-   ##
=======================================
  Coverage   96.35%   96.36%           
=======================================
  Files          52       52           
  Lines        8735     8738    +3     
  Branches     1723     1727    +4     
=======================================
+ Hits         8417     8420    +3     
  Misses        186      186           
  Partials      132      132           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ssjkamei]

I have addressed the parts that were returned due to code check errors.
Also, I'm sorry, it seems that I made a mistake and the past commit history is displayed..

[stefan6419846]

Thanks. I could not see any apparent issue with this and all the tests still pass without having to change anything. Thus I am going to merge this for now.

[stefan6419846]

Also, I'm sorry, it seems that I made a mistake and the past commit history is displayed.

Your previous PR was from your main branch, while this PR used a separate branch, but all branches usually originate from main. Thus your previous commits where included here verbosely.

I recommend you to reset the changes from your main branch and sync it with upstream. If this is too complex, consider deleting and re-creating your fork if there is no work which would be lost by such a process, and always use dedicated branches for further PRs.