You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by GitBox <gi...@apache.org> on 2020/02/18 14:24:46 UTC
[GitHub] [commons-codec] sguill opened a new pull request #36: CODEC-281:
encode kc as K
sguill opened a new pull request #36: CODEC-281: encode kc as K
URL: https://github.com/apache/commons-codec/pull/36
**Double metaphone should handle 'K' for 'kc' entries instead of 'KK'**
One example is Kir***kc***aldy: which is presently encoded as "KR***KK***".
When omitting 'k' or 'c' letters (for example, Kircaldy or Kirkaldy) it is encoded as "KRKL".
The correction consists to verify when letter 'c' follows 'k', if 'c' is encoded as 'K', it is ignored as for a succession of 'kk' letters. With this correction, Kirkcaldy would be encoded as "KRKL" which is more discriminant and compliant with the phonetic.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [commons-codec] coveralls commented on issue #36: CODEC-281: encode
kc as K
Posted by GitBox <gi...@apache.org>.
coveralls commented on issue #36: CODEC-281: encode kc as K
URL: https://github.com/apache/commons-codec/pull/36#issuecomment-587498658
[](https://coveralls.io/builds/28800046)
Coverage decreased (-0.1%) to 93.74% when pulling **444f8d206e253c98b5c0efb8531e4c36f13ed8c7 on sguill:CODEC-281-DoubleMetaphoneKC** into **126f90424cc8f62d928686ec20c6c8bd944b527b on apache:master**.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [commons-codec] coveralls edited a comment on issue #36: CODEC-281:
encode kc as K
Posted by GitBox <gi...@apache.org>.
coveralls edited a comment on issue #36: CODEC-281: encode kc as K
URL: https://github.com/apache/commons-codec/pull/36#issuecomment-587498658
[](https://coveralls.io/builds/28819764)
Coverage increased (+0.01%) to 93.853% when pulling **eb9ca3d53badf69e94b1c90cc6c7daea4638015a on sguill:CODEC-281-DoubleMetaphoneKC** into **126f90424cc8f62d928686ec20c6c8bd944b527b on apache:master**.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services