[Libreoffice-commits] core.git: Changes to 'feature/glyphid-cleanup'

[Khaled Hosny]

New branch 'feature/glyphid-cleanup' available with the following commits:
commit 948bfe4c5000ec36666a479dd1550dd038aa0c84
Author: Khaled Hosny <khaledhosny at eglug.org>
Date:   Thu Dec 8 07:17:00 2016 +0200

    Drop ComplexDisabled flags that does nothing
    
    There is no such thing as simple text layout in this brave new world.
    
    Change-Id: I641b7efa5ec8fdd9839c291e7f2fdb736361ef1e

commit e7db4445736dd5b0066b1abb2dabfb43f11969ea
Author: Khaled Hosny <khaledhosny at eglug.org>
Date:   Thu Dec 8 07:04:27 2016 +0200

    Drop a hack for Type 1, the future is here
    
    The original issue for reference:
    https://bz.apache.org/ooo/show_bug.cgi?id=36691
    
    Change-Id: I3b68c2c9f7cb94445875ef5a005e73dcfe65fa3a

commit 64a6642ecb160d0e237248b71ea320f8b6d3d404
Author: Khaled Hosny <khaledhosny at eglug.org>
Date:   Thu Dec 8 07:00:02 2016 +0200

    Drop SalLayout::IsSpacingGlyph()
    
    Replace it with a check from ICU that actually makes sense.
    
    Change-Id: Ie8789444a967420d58f5575f42bc45677bfa96a1

commit 43eb6766e367109f99c237886d6adf16fab606bd
Author: Khaled Hosny <khaledhosny at eglug.org>
Date:   Thu Dec 8 06:37:42 2016 +0200

    Remove noop SalLayout::UseCommonLayout()
    
    Change-Id: Ifc143f33bc4e087726ada1e3b1b5182bf830b0cb

commit 254ef0d03efc70e7e7bd96c508fce09570acbf57
Author: Khaled Hosny <khaledhosny at eglug.org>
Date:   Thu Dec 8 06:21:23 2016 +0200

    GF_VERT is only used once, move it
    
    Change-Id: I15289eecfbfb29ee549470a7f752e0c7fd9f811e

commit 65ec1f97b378d75b54de46784dce6411e346fea0
Author: Khaled Hosny <khaledhosny at eglug.org>
Date:   Thu Dec 8 05:57:40 2016 +0200

    sal_GlyphId is a simple glyph id now
    
    No more cleverness.
    
    Change-Id: I760c602802961b37728c6987c4ade14fb02ca034

commit af1cf288d1edfc86f2974ce9823c46ab7e2948e5
Author: Khaled Hosny <khaledhosny at eglug.org>
Date:   Thu Dec 8 05:38:08 2016 +0200

    Don’t encode font fallback level in the glyph id
    
    Change-Id: I747d969c3c1dc42b1a3b5d12d06fed3af9a64675

commit 2c5bd6ff71fa223b57c49210d3299cb6f721c526
Author: Khaled Hosny <khaledhosny at eglug.org>
Date:   Thu Dec 8 04:46:32 2016 +0200

    Minor
    
    Change-Id: I5e3480b647ec3851e9125188d9cd04227050dc6d

commit 64724fb16991a7d93e1d18209075ab2267d75b02
Author: Khaled Hosny <khaledhosny at eglug.org>
Date:   Thu Dec 8 04:39:16 2016 +0200

    Don’t encode the vertical flag in the glyph id
    
    Change-Id: I00485dd4d42004e4eaa163a9e6ad0a43cf98a30a

commit ffe9f6e7c227c0aa0f7b3f38decb4aa029b46521
Author: Khaled Hosny <khaledhosny at eglug.org>
Date:   Thu Dec 8 04:13:26 2016 +0200

    Use GlyphItem in more places
    
    Also simplify code to what is actually being used.
    
    Change-Id: Ic640ea7f3e426e3f5f4f9fef30d774fe5ac111ff

commit 25f48c49e6c7619ccf668398c1ed5bb536506cf5
Author: Khaled Hosny <khaledhosny at eglug.org>
Date:   Thu Dec 8 03:01:14 2016 +0200

    Use GlyphItem in more places
    
    Change-Id: Ic0dd8c0311161271817eb9a3f454d00a319e43a8

commit cd6cd674c9fbbefa546c0db2817ae5b7ccbbce67
Author: Khaled Hosny <khaledhosny at eglug.org>
Date:   Thu Dec 8 00:43:09 2016 +0200

    Pass GlyphItem around
    
    We have this nice structure that contains (almost) all the information
    we need, so pass it around instead of passing separate fragments of said
    information.
    
    The ultimate is to kill the horrible sal_GlyphId hack if encoding
    various bits of information in the higher bits of a 32-bit integer.
    
    Change-Id: Ie496bb4c2932157527a388e2a94e46bf0a325a70