karl williamson <public@khwilliamson.com> wrote: :A revised mktables is available, both at :git://github.com/khwilliamson/perl.git. (The branch is called mktables) : It fixes the minor bug #69018, concerning accepting the erroneous :\p{Script=InGreek}, and perhaps other bugs; I need to look. But it :fixes a number of things which I have not bothered to write bug reports :on, many of them have been aired on the p5p list over the last several :months. Awesome work, I'm so glad somebody has hands tough enough to grasp these thorns. I don't know this stuff in any depth, but a few general things struck me while reading. :All duplicate files have been eliminated. That means that if two :properties match the same exact set of code points, one file serves :both. This was not so much to save disk space, as to save memory, as :the same swash can now serve multiple properties. Is this dealt with automatically, so that if two such properties later diverge it will correctly split them out to separate files again? : -p tells mktables to give progress information as it works. If this is most useful for humans, it might be slightly nicer to make it on by default, and instead add a -q[uiet] option to suppress it. :The rest of this is text intended to be suitable for perldelta. NOTE :that this includes some anticipated documentation changes that haven't :been submitted yet. The perldelta text mentions lots of effects on properties, but doesn't say anything about things like \w and \d. If this patch affects what they match then it is vital to mention that; but if it doesn't, that's also important information that readers will be searching for. HugoThread Previous | Thread Next