Re: GLib Patch



Owen Taylor wrote:


If you are into this type of project, there are two more you
could tackle:

 A) gunidecomp.h is a far worse offender for relocations than
    the stuff you fixed ... it has about 3500 relocations!


...


   A simpler approach would be simply have indirection
   with:

     unsigned gushort expansion_index;

Being an offset into a simple char * holding all the expansions concatenated together; I think this is
   probably the right approach.


I've created a bug for this issue, along with a patch based on the approach you described here: http://bugzilla.gnome.org/show_bug.cgi?id=64982. It gets rid of another 3500 or so relocations, as well as moving more data into the .rodata section.

Andrew





[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]