Skip to content

Use invariant case conversion - #82

Merged
ssg merged 2 commits into
ssg:mainfrom
skacurt:main
Jul 17, 2026
Merged

Use invariant case conversion#82
ssg merged 2 commits into
ssg:mainfrom
skacurt:main

Conversation

@skacurt

@skacurt skacurt commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

fixes #81

@ssg ssg left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

buna test de ekler misin, NUnit'in [SetCulture("tr-TR")] attribute'unu kullanarak?

@skacurt

skacurt commented Jul 17, 2026

Copy link
Copy Markdown
Contributor Author

Hataya sebep olan CodingAlphabet C'tor için testler ekledim.

Hatırlatmak adına:

CodingAlphabet C'tor'da belirtilen alfabedeki karakterler caseInsensitive: true olduğunda harfler büyükse küçük, küçükse büyük hallerine çevirilip alfabede harfin diğer formunun var olup olmadığı kontrol ediyor, varsa hata fırlatıyor. Bu yüzden ayrı ayrı test edeceğiz. Zaten bug hemen bu kontrol sonrası map işleminde 127 sınırını aştığı için oluşuyordu.

Özetle bu yüzden i ve I karakterlerinden ibaret caseInsensitive: true alfabeleri ayrı ayrı tr-TR ortamda test ediyoruz.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Culture-sensitive ToLower/ToUpper breaks encoding on Turkish locales

2 participants