public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* DT_ENCODING
@ 2007-04-05  8:56 Jan Beulich
  2007-04-05 13:28 ` DT_ENCODING Alan Modra
  0 siblings, 1 reply; 4+ messages in thread
From: Jan Beulich @ 2007-04-05  8:56 UTC (permalink / raw)
  To: Alan Modra; +Cc: binutils

Back in 2003, you changed DT_ENCODING from 32 (as documented) to 31.
Can you explain this (the ChangeLog entry unfortunately says nothing)?

Thanks, Jan

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: DT_ENCODING
  2007-04-05  8:56 DT_ENCODING Jan Beulich
@ 2007-04-05 13:28 ` Alan Modra
  2007-04-26 13:06   ` DT_ENCODING Jan Beulich
  0 siblings, 1 reply; 4+ messages in thread
From: Alan Modra @ 2007-04-05 13:28 UTC (permalink / raw)
  To: Jan Beulich; +Cc: binutils

On Thu, Apr 05, 2007 at 09:56:57AM +0100, Jan Beulich wrote:
> Back in 2003, you changed DT_ENCODING from 32 (as documented) to 31.
> Can you explain this (the ChangeLog entry unfortunately says nothing)?

I don't know now, sorry.  Please change it back as I clearly did the
wrong thing.

-- 
Alan Modra
IBM OzLabs - Linux Technology Centre

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: DT_ENCODING
  2007-04-05 13:28 ` DT_ENCODING Alan Modra
@ 2007-04-26 13:06   ` Jan Beulich
  2007-04-26 13:57     ` DT_ENCODING Hans-Peter Nilsson
  0 siblings, 1 reply; 4+ messages in thread
From: Jan Beulich @ 2007-04-26 13:06 UTC (permalink / raw)
  To: Alan Modra; +Cc: binutils

>>> Alan Modra <amodra@bigpond.net.au> 05.04.07 15:28 >>>
>On Thu, Apr 05, 2007 at 09:56:57AM +0100, Jan Beulich wrote:
>> Back in 2003, you changed DT_ENCODING from 32 (as documented) to 31.
>> Can you explain this (the ChangeLog entry unfortunately says nothing)?
>
>I don't know now, sorry.  Please change it back as I clearly did the
>wrong thing.

Committed:

Revert the value change of DT_ENCODING from 2003-01-17. Luckily (and
surprisingly) the value isn't really being used anywhere, henc no other
changes are needed.

include/elf/
2007-04-26  Jan Beulich  <jbeulich@novell.com>

	* common.h (DT_ENCODING): Correct value (back to spec mandated
	value).

--- 2007-04-26/include/elf/common.h
+++ 2007-04-26/include/elf/common.h
@@ -550,7 +550,7 @@
 #define DT_FINI_ARRAYSZ 28
 #define DT_RUNPATH	29
 #define DT_FLAGS	30
-#define DT_ENCODING	31
+#define DT_ENCODING	32
 #define DT_PREINIT_ARRAY   32
 #define DT_PREINIT_ARRAYSZ 33
 



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: DT_ENCODING
  2007-04-26 13:06   ` DT_ENCODING Jan Beulich
@ 2007-04-26 13:57     ` Hans-Peter Nilsson
  0 siblings, 0 replies; 4+ messages in thread
From: Hans-Peter Nilsson @ 2007-04-26 13:57 UTC (permalink / raw)
  To: binutils

On Thu, 26 Apr 2007, Jan Beulich wrote:
> >>> Alan Modra <amodra@bigpond.net.au> 05.04.07 15:28 >>>
> >On Thu, Apr 05, 2007 at 09:56:57AM +0100, Jan Beulich wrote:
> >> Back in 2003, you changed DT_ENCODING from 32 (as documented) to 31.
> >> Can you explain this (the ChangeLog entry unfortunately says nothing)?
> >
> >I don't know now, sorry.  Please change it back as I clearly did the
> >wrong thing.

For the record, a STFW turned up
<http://sourceware.org/ml/binutils/2000-01/msg00059.html>
and <http://www.sco.com/developers/gabi/latest/ch5.dynamic.html>
regarding the apparent but wrongly percepted collision.

> Revert the value change of DT_ENCODING from 2003-01-17. Luckily (and
> surprisingly)

It was used at some time (see above) but only wrongly so. :].

brgds, H-P

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2007-04-26 13:06 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-04-05  8:56 DT_ENCODING Jan Beulich
2007-04-05 13:28 ` DT_ENCODING Alan Modra
2007-04-26 13:06   ` DT_ENCODING Jan Beulich
2007-04-26 13:57     ` DT_ENCODING Hans-Peter Nilsson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).