unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [sa2c@sa2c.net: pp-eval-last-sexp() does not eval last sexp]
@ 2006-11-02 21:46 Richard Stallman
  2006-11-03 14:21 ` Ramprasad
  0 siblings, 1 reply; 6+ messages in thread
From: Richard Stallman @ 2006-11-02 21:46 UTC (permalink / raw)


Would someone please install this, then ack?

------- Start of forwarded message -------
To: emacs-pretest-bug@gnu.org
From: NIIMI Satoshi <sa2c@sa2c.net>
Date: Wed, 01 Nov 2006 22:30:15 +0900
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Subject: pp-eval-last-sexp() does not eval last sexp
X-Spam-Status: No, score=0.0 required=5.0 tests=RCVD_BY_IP autolearn=failed 
	version=3.0.4


Please write in English if possible, because the Emacs maintainers
usually do not have translators to read other languages for them.

Your bug report will be posted to the emacs-pretest-bug@gnu.org mailing list.

Please describe exactly what actions triggered the bug
and the precise symptoms of the bug:

pp-eval-expression() was changed to use (interactive "X") and not to
pass its argument to eval(), but pp-eval-last-sexp() is not changed.

Because of that, target sexp is not evaluted.

Here is a patch to fix the problem.

*** pp.el	19 10 2006 19:33:06 +0900	1.29
- --- pp.el	01 11 2006 22:13:37 +0900	
***************
*** 161,167 ****
      (set-syntax-table stab)
      (if arg
  	(insert (pp-to-string (eval exp)))
!       (pp-eval-expression exp))))
  
  ;;; Test cases for quote
  ;; (pp-eval-expression ''(quote quote))
- --- 161,167 ----
      (set-syntax-table stab)
      (if arg
  	(insert (pp-to-string (eval exp)))
!       (pp-eval-expression (eval exp)))))
  
  ;;; Test cases for quote
  ;; (pp-eval-expression ''(quote quote))


If emacs crashed, and you have the emacs process in the gdb debugger,
please include the output from the following gdb commands:
    `bt full' and `xbacktrace'.
If you would like to further debug the crash, please read the file
/home/sa2c/opt/emacs22/share/emacs/22.0.90/etc/DEBUG for instructions.


In GNU Emacs 22.0.90.2 (i386-unknown-freebsd6.1, GTK+ Version 2.10.6)
 of 2006-11-01 on berkeley.l.sa2c.net
X server distributor `The XFree86 Project, Inc', version 11.0.40300000
configured using `configure '--prefix=/home/sa2c/opt/emacs22' '--with-gtk''

Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: nil
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: ja_JP.UTF-8
  locale-coding-system: utf-8
  default-enable-multibyte-characters: t

Major mode: Emacs-Lisp

Minor modes in effect:
  eldoc-mode: t
  show-paren-mode: t
  which-function-mode: t
  shell-dirtrack-mode: t
  tooltip-mode: t
  mouse-wheel-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  unify-8859-on-encoding-mode: t
  utf-translate-cjk-mode: t
  auto-compression-mode: t
  line-number-mode: t

Recent input:
o <tab> e m <tab> l i s <tab> <tab> <tab> / <tab> e 
m <tab> a <tab> i <tab> <tab> SPC <backspace> <tab> 
p p <tab> <return> C-s p p - e v a l - l a C-a C-M-n 
C-a M-x s e t - v a <tab> <return> d i f f - M-p <tab> 
s <tab> <return> " - c " <return> C-x v = C-n C-n C-n 
C-n C-n C-n C-x 0 M-x e m a <tab> <tab> C-a C-k s e 
n <tab> <tab> <M-backspace> r e p o <tab> r <tab> <tab> 
<return>

Recent messages:
20
Making completion list... [3 times]
Loading vc-cvs...done
Mark saved where search started
next-history-element: Beginning of history; no preceding item
Making completion list...
Loading diff...done
Loading vc...done
Making completion list... [3 times]
Loading emacsbug...done

- -- 
NIIMI Satoshi


_______________________________________________
emacs-pretest-bug mailing list
emacs-pretest-bug@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-pretest-bug
------- End of forwarded message -------

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

* Re: [sa2c@sa2c.net: pp-eval-last-sexp() does not eval last sexp]
  2006-11-02 21:46 [sa2c@sa2c.net: pp-eval-last-sexp() does not eval last sexp] Richard Stallman
@ 2006-11-03 14:21 ` Ramprasad
  2006-11-03 16:04   ` Juanma Barranquero
  0 siblings, 1 reply; 6+ messages in thread
From: Ramprasad @ 2006-11-03 14:21 UTC (permalink / raw)
  Cc: emacs-devel

On Fri, 2006-11-03 at 03:16, Richard Stallman wrote:
> Would someone please install this, then ack?

Done !

-
Ramprasad B

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

* Re: [sa2c@sa2c.net: pp-eval-last-sexp() does not eval last sexp]
  2006-11-03 14:21 ` Ramprasad
@ 2006-11-03 16:04   ` Juanma Barranquero
  2006-11-03 18:22     ` Ramprasad B
  0 siblings, 1 reply; 6+ messages in thread
From: Juanma Barranquero @ 2006-11-03 16:04 UTC (permalink / raw)
  Cc: emacs-devel

On 11/3/06, Ramprasad <ramprasad@gnu.org> wrote:

> Done !

Thanks for committing these patches.

Note, however, that you put the changelog entry for
lib-src/Makefile.in into lisp/ChangeLog, and that all three entries
had problems:

  - lib-src/Makefile.in: It didn't mention where was INSTALL_SCRIPT used.
  - lisp/sort.el: It didn't mention `sort-columns', where the change
had happened.
  - emacs-lisp/pp.el: It referenced `pp-eval-expression', but the
change is in `pp-eval-last-sexp'. Moreover, the filename was
lisp/emacs-lisp/pp.el (they should be relative from the directory
where the ChangeLog file resides).

All this makes me think you don't know about
`add-change-log-entry-other-window' (bound to C-x 4 a), which I
recommend.

Thanks,
                    /L/e/k/t/u

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

* Re: [sa2c@sa2c.net: pp-eval-last-sexp() does not eval last sexp]
  2006-11-03 16:04   ` Juanma Barranquero
@ 2006-11-03 18:22     ` Ramprasad B
  2006-11-04 11:11       ` Eli Zaretskii
  0 siblings, 1 reply; 6+ messages in thread
From: Ramprasad B @ 2006-11-03 18:22 UTC (permalink / raw)
  Cc: emacs-devel

On Fri, 2006-11-03 at 17:04 +0100, Juanma Barranquero wrote:

> Note, however, that you put the changelog entry for
> lib-src/Makefile.in into lisp/ChangeLog, and that all three entries
> had problems:
> 
>   - lib-src/Makefile.in: It didn't mention where was INSTALL_SCRIPT used.
>   - lisp/sort.el: It didn't mention `sort-columns', where the change
> had happened.
>   - emacs-lisp/pp.el: It referenced `pp-eval-expression', but the
> change is in `pp-eval-last-sexp'. Moreover, the filename was
> lisp/emacs-lisp/pp.el (they should be relative from the directory
> where the ChangeLog file resides).
> 
> All this makes me think you don't know about
> `add-change-log-entry-other-window' (bound to C-x 4 a), which I
> recommend.

I am sorry for those mistakes.

Thank you very much for working on this and guiding me.

-
Ramprasad B

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

* Re: [sa2c@sa2c.net: pp-eval-last-sexp() does not eval last sexp]
  2006-11-03 18:22     ` Ramprasad B
@ 2006-11-04 11:11       ` Eli Zaretskii
  2006-11-04 16:04         ` Ramprasad B
  0 siblings, 1 reply; 6+ messages in thread
From: Eli Zaretskii @ 2006-11-04 11:11 UTC (permalink / raw)
  Cc: emacs-devel

The ChangeLog entry for Yoni's change in lisp/faces.el was also
incorrect: it referenced lisp/faces.el instead of just faces.el.

Furthermore, the log text you used with the "cvs ci" command was just
this:

   Installed describe-face sample text

Please in the future put there the text of the ChangeLog entry, after
stripping the file name part and the leading whitespace.  Thus, for
the log entry you used:

	* lisp/faces.el (faces-sample-overlay): New defvar.
	(faces-sample-overlay): New function to show face sample text.
	(describe-face): Use it.

the appropriate "cvs ci" log text would be this:

(faces-sample-overlay): New defvar.
(faces-sample-overlay): New function to show face sample text.
(describe-face): Use it.


Note that similar problems exist in the other changes you installed.

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

* Re: [sa2c@sa2c.net: pp-eval-last-sexp() does not eval last sexp]
  2006-11-04 11:11       ` Eli Zaretskii
@ 2006-11-04 16:04         ` Ramprasad B
  0 siblings, 0 replies; 6+ messages in thread
From: Ramprasad B @ 2006-11-04 16:04 UTC (permalink / raw)
  Cc: emacs-devel

On Sat, 2006-11-04 at 13:11 +0200, Eli Zaretskii wrote:
> The ChangeLog entry for Yoni's change in lisp/faces.el was also
> incorrect: it referenced lisp/faces.el instead of just faces.el.
> 
> Furthermore, the log text you used with the "cvs ci" command was just
> this:
> 
>    Installed describe-face sample text
> 
> Please in the future put there the text of the ChangeLog entry, after
> stripping the file name part and the leading whitespace.  Thus, for
> the log entry you used:
> 
> 	* lisp/faces.el (faces-sample-overlay): New defvar.
> 	(faces-sample-overlay): New function to show face sample text.
> 	(describe-face): Use it.
> 
> the appropriate "cvs ci" log text would be this:
> 
> (faces-sample-overlay): New defvar.
> (faces-sample-overlay): New function to show face sample text.
> (describe-face): Use it.
> 
> 
> Note that similar problems exist in the other changes you installed.
> 

Thank you for fixing those mistakes.

-
Ramprasad B

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

end of thread, other threads:[~2006-11-04 16:04 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-11-02 21:46 [sa2c@sa2c.net: pp-eval-last-sexp() does not eval last sexp] Richard Stallman
2006-11-03 14:21 ` Ramprasad
2006-11-03 16:04   ` Juanma Barranquero
2006-11-03 18:22     ` Ramprasad B
2006-11-04 11:11       ` Eli Zaretskii
2006-11-04 16:04         ` Ramprasad B

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