Reviewing merge request #480: fixes and improvements for qunicodetables data generator

most important of them:

*) Unicode character property 'General_Category=Cn' was erroneously ignored causing two related bugs in QChar::category() and QChar::isPrint().

As described in tr44, 4.2.8 Default Values
General_Category character property should be setted to Cn
for all codepoints omitted in UCD.
Instead they was mapped to QChar::NoCategory (=0) which have no equivalent
in Unicode specs and as result was ignored in filtering by binary mask.

*) reduce count of the unique unicode properties by fixing specialCaseMap entries uniqueness

this is a required set of changes on the road to Unicode 5.2.0 support

Commits that would be merged:

Version 6
  • Version 1
  • Version 2
  • Version 3
  • Version 4
  • Version 4
  • Version 5
  • Version 5
  • Version 6
  • Version 6
  • e49afde
  • Ritt Konstantin
almost 2 years ago
  • ff0da4a
  • Ritt Konstantin
almost 2 years ago
  • e0818b3
  • Ritt Konstantin
almost 2 years ago
  • 58e5170
  • Ritt Konstantin
almost 2 years ago
  • 449286d
  • Ritt Konstantin
almost 2 years ago
  • 83f1771
  • Ritt Konstantin
almost 2 years ago
  • a4cbbe8
  • Ritt Konstantin
almost 2 years ago
  • 93dd200
  • Ritt Konstantin
almost 2 years ago
  • 764759c
  • Ritt Konstantin
almost 2 years ago
  • fe336d1
  • Ritt Konstantin
almost 2 years ago
  • f01ba38
  • Ritt Konstantin
almost 2 years ago
  • 11ffc5f
  • Ritt Konstantin
almost 2 years ago
  • 1a09c14
  • Ritt Konstantin
almost 2 years ago
  • 82939eb
  • Ritt Konstantin
almost 2 years ago
  • 61800c9
  • Ritt Konstantin
almost 2 years ago
  • 039bbfd
  • Ritt Konstantin
almost 2 years ago
  • 7947538
  • Ritt Konstantin
almost 2 years ago
  • ee61b6a
  • Ritt Konstantin
almost 2 years ago
  • dd03be5
  • Ritt Konstantin
almost 2 years ago
  • 143c9fb
  • Ritt Konstantin
almost 2 years ago
Showing 4d61576-e49afde

Comments

→ State changed from New to Reviewing

Thanks.

→ State changed from Reviewing to Merged

Merged to 4.7 as 5b05cdccda99d5889c6a3db41d5c494d25798840

Add a new comment:

Login or create an account to post a comment

How to apply this merge request to your repository