unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#17952: Compatability for eieio between 24.4 and 24.3
@ 2014-07-06  4:34 Matthew Fidler
  2014-07-06  8:49 ` David Engster
  0 siblings, 1 reply; 5+ messages in thread
From: Matthew Fidler @ 2014-07-06  4:34 UTC (permalink / raw)
  To: 17952

[-- Attachment #1: Type: text/plain, Size: 373 bytes --]

The object-set-name-string function of eieio has changed to
eieio-object-set-name-string, at one point my
code (ergoemacs-mode) depended on that function, so I had to use a macro to
pick the right version for different versions of emacs.

You may keep it or change it as you wish.  Today ergoemacs-mode no longer
uses this function.

Just an FYI, prompted by Stefan.

Matt

[-- Attachment #2: Type: text/html, Size: 480 bytes --]

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

* bug#17952: Compatability for eieio between 24.4 and 24.3
  2014-07-06  4:34 bug#17952: Compatability for eieio between 24.4 and 24.3 Matthew Fidler
@ 2014-07-06  8:49 ` David Engster
  2014-07-06 12:24   ` Matthew Fidler
  0 siblings, 1 reply; 5+ messages in thread
From: David Engster @ 2014-07-06  8:49 UTC (permalink / raw)
  To: Matthew Fidler; +Cc: 17952

Matthew Fidler writes:
> The object-set-name-string function of eieio has changed to
> eieio-object-set-name-string, at one point my code (ergoemacs-mode) depended on
> that function, so I had to use a macro to pick the right version for different
> versions of emacs. 

The usual way for renaming a function in Emacs is to mark it as
deprecated and make it an alias for the new name, and after a few more
Emacs releases remove it altogether. So if you want to stay compatible
with older Emacsen, simply continue to use `object-set-name-string' for
the time being. You will get a warning on byte-compilation, but just
ignore it (or turn off the deprecation warning if it bothers you too
much).

-David





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

* bug#17952: Compatability for eieio between 24.4 and 24.3
  2014-07-06  8:49 ` David Engster
@ 2014-07-06 12:24   ` Matthew Fidler
  2014-07-06 18:01     ` Glenn Morris
  0 siblings, 1 reply; 5+ messages in thread
From: Matthew Fidler @ 2014-07-06 12:24 UTC (permalink / raw)
  To: David Engster; +Cc: 17952

[-- Attachment #1: Type: text/plain, Size: 1255 bytes --]

Thanks David

I don't believe object-set-name-string was marked as a depreciated
function. I may have not been running against the trunk, but I believe my
compile failed instead of issued a warning.

I think the authors intent is that people access the object name with
eieio-named so using this function directly was not supported.

I have changed my code to solely use eieio-named, so compile warnings on
either 24.3 or 24.4 are not issued.

Matt
On Jul 6, 2014 3:49 AM, "David Engster" <deng@randomsample.de> wrote:

> Matthew Fidler writes:
> > The object-set-name-string function of eieio has changed to
> > eieio-object-set-name-string, at one point my
> code (ergoemacs-mode) depended on
> > that function, so I had to use a macro to pick the right version for
> different
> > versions of emacs.
>
> The usual way for renaming a function in Emacs is to mark it as
> deprecated and make it an alias for the new name, and after a few more
> Emacs releases remove it altogether. So if you want to stay compatible
> with older Emacsen, simply continue to use `object-set-name-string' for
> the time being. You will get a warning on byte-compilation, but just
> ignore it (or turn off the deprecation warning if it bothers you too
> much).
>
> -David
>

[-- Attachment #2: Type: text/html, Size: 1639 bytes --]

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

* bug#17952: Compatability for eieio between 24.4 and 24.3
  2014-07-06 12:24   ` Matthew Fidler
@ 2014-07-06 18:01     ` Glenn Morris
  2014-07-06 18:03       ` Matthew Fidler
  0 siblings, 1 reply; 5+ messages in thread
From: Glenn Morris @ 2014-07-06 18:01 UTC (permalink / raw)
  To: Matthew Fidler; +Cc: 17952, David Engster


emacs-24.3.92 -Q -l eieio

C-h f object-set-name-string

    object-set-name-string is an alias for `eieio-object-set-name-string'
    in `eieio.el'.

    (object-set-name-string OBJ NAME)

    This function is obsolete since 24.4;
    use `eieio-object-set-name-string' instead.





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

* bug#17952: Compatability for eieio between 24.4 and 24.3
  2014-07-06 18:01     ` Glenn Morris
@ 2014-07-06 18:03       ` Matthew Fidler
  0 siblings, 0 replies; 5+ messages in thread
From: Matthew Fidler @ 2014-07-06 18:03 UTC (permalink / raw)
  To: Glenn Morris; +Cc: 17952, David Engster

[-- Attachment #1: Type: text/plain, Size: 384 bytes --]

My misunderstanding.
On Jul 6, 2014 1:01 PM, "Glenn Morris" <rgm@gnu.org> wrote:

>
> emacs-24.3.92 -Q -l eieio
>
> C-h f object-set-name-string
>
>     object-set-name-string is an alias for `eieio-object-set-name-string'
>     in `eieio.el'.
>
>     (object-set-name-string OBJ NAME)
>
>     This function is obsolete since 24.4;
>     use `eieio-object-set-name-string' instead.
>

[-- Attachment #2: Type: text/html, Size: 684 bytes --]

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

end of thread, other threads:[~2014-07-06 18:03 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-07-06  4:34 bug#17952: Compatability for eieio between 24.4 and 24.3 Matthew Fidler
2014-07-06  8:49 ` David Engster
2014-07-06 12:24   ` Matthew Fidler
2014-07-06 18:01     ` Glenn Morris
2014-07-06 18:03       ` Matthew Fidler

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

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).