Re: [PATCH 1/3] wineps.drv: Make the Unicode name-sorted glyph table fully const.

[Erich E. Hoover]

On Wed, Sep 16, 2015 at 10:42 AM, Henri Verbeet  wrote:
> On 16 September 2015 at 18:33, Erich E. Hoover
>  wrote:
> > Sorry, I've been incredibly busy and wasn't paying close enough
> > attention to the fact that those tables did not contain pointers.
> > Patch 3 still applies.
>
> They do, and that's what stops the compiler from putting them in
> .rodata. It's good to make the table in patch 3 const, but it won't
> really affect where it's placed. Either way, if you send this kind of
> patch you should verify it works.

Relevant section headers:
ehoover@lappy:/usr/local/lib/wine$ readelf --section-details wineps.drv.so
...
  [19] .data.rel.ro
       PROGBITS        000562c0 0552c0 024c20 00   0   0 32
       [00000003]: WRITE, ALLOC
...
  [23] .data
       PROGBITS        0007b0e0 07a0e0 007f08 00   0   0 32
       [00000003]: WRITE, ALLOC

Before:
ehoover@lappy:/usr/local/lib/wine$ readelf -s wineps.drv.so | grep -e
glyph_table -e Num
   Num:    Value  Size Type    Bind   Vis      Ndx Name
   214: 00082f00  1032 OBJECT  LOCAL  DEFAULT   23 glyph_table.17436

After:
ehoover@lappy:/usr/local/lib/wine$ readelf -s wineps.drv.so | grep -e
glyph_table -e Num
   Num:    Value  Size Type    Bind   Vis      Ndx Name
   214: 0007a8c0  1032 OBJECT  LOCAL  DEFAULT   19 glyph_table.17436

So yes, this works.

Best,
Erich