Derive index EUC-KR from ICU
Created attachment 422991 [details] Patch
Created attachment 423014 [details] Patch
Created attachment 423049 [details] Patch
Comment on attachment 423049 [details] Patch r=me
Comment on attachment 423049 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=423049&action=review > Source/WebCore/platform/text/EncodingTables.cpp:8654 > - ASSERT(isSortedByFirst(eucKRDecodingIndex)); > - ASSERT(sortedFirstsAreUnique(eucKRDecodingIndex)); > + ASSERT(isSortedByFirst(eucKR())); > + ASSERT(sortedFirstsAreUnique(eucKR())); Wouldn’t you want to keep both of these?
Comment on attachment 423049 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=423049&action=review >> Source/WebCore/platform/text/EncodingTables.cpp:8654 >> + ASSERT(sortedFirstsAreUnique(eucKR())); > > Wouldn’t you want to keep both of these? I don't t think I understand this comment. I am keeping both of these
commit-queue failed to commit attachment 423049 [details] to WebKit repository. To retry, please set cq+ flag again.
Comment on attachment 423049 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=423049&action=review >>> Source/WebCore/platform/text/EncodingTables.cpp:8654 >>> + ASSERT(sortedFirstsAreUnique(eucKR())); >> >> Wouldn’t you want to keep both of these? > > I don't t think I understand this comment. I am keeping both of these I was suggesting we could separately assert both that eucKRDecodingIndexReference is sorted and unique and that the actual generated table is?
We assert that they are equal when generating the table.
r274569
(In reply to Alex Christensen from comment #10) > We assert that they are equal when generating the table. Oh, right, that should have been obvious to me!