all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Move 'with-buffer-modified-unmodified' to simple.el
@ 2020-09-16 23:12 Yuan Fu
  2020-09-17  3:11 ` Stefan Monnier
  0 siblings, 1 reply; 6+ messages in thread
From: Yuan Fu @ 2020-09-16 23:12 UTC (permalink / raw)
  To: emacs-devel

Currently it’s in bookmark.el. It seems to make more sense to be in simple.el or sub.el. WDYT?

Yuan





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

* Re: Move 'with-buffer-modified-unmodified' to simple.el
  2020-09-16 23:12 Move 'with-buffer-modified-unmodified' to simple.el Yuan Fu
@ 2020-09-17  3:11 ` Stefan Monnier
  2020-09-18 18:25   ` Yuan Fu
  0 siblings, 1 reply; 6+ messages in thread
From: Stefan Monnier @ 2020-09-17  3:11 UTC (permalink / raw)
  To: Yuan Fu; +Cc: emacs-devel

> Currently it’s in bookmark.el.  It seems to make more sense to be in simple.el or sub.el.  WDYT?

Most use cases should be covered by `with-silent-modifications`, but
there might indeed be cases as in bookmark.el where the extra bindings
in `with-silent-modifications` aren't right.

Could you look through our code (e.g. with something like

    grep '(set-buffer-modified-p' **/*.el | grep -v 'd-p \(nil\|t\))'

) to see where we could use `with-silent-modifications` and where only
'with-buffer-modified-unmodified' can be used?


        Stefan




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

* Re: Move 'with-buffer-modified-unmodified' to simple.el
  2020-09-17  3:11 ` Stefan Monnier
@ 2020-09-18 18:25   ` Yuan Fu
  2020-09-18 19:56     ` Stefan Monnier
  0 siblings, 1 reply; 6+ messages in thread
From: Yuan Fu @ 2020-09-18 18:25 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel



> On Sep 16, 2020, at 11:11 PM, Stefan Monnier <monnier@iro.umontreal.ca> wrote:
> 
>> Currently it’s in bookmark.el.  It seems to make more sense to be in simple.el or sub.el.  WDYT?
> 
> Most use cases should be covered by `with-silent-modifications`, but
> there might indeed be cases as in bookmark.el where the extra bindings
> in `with-silent-modifications` aren't right.
> 
> Could you look through our code (e.g. with something like
> 
>    grep '(set-buffer-modified-p' **/*.el | grep -v 'd-p \(nil\|t\))'
> 
> ) to see where we could use `with-silent-modifications` and where only
> 'with-buffer-modified-unmodified' can be used?
> 
> 
>        Stefan
> 

How should we determine if the extra bindings in `with-silent-modifications’ aren’t right? In my understanding `with-silent-modifications’ should only be used when changing text properties and if you actually insert something you should use `with-buffer-modified-unmodified’. Is that correct?

Looking though the result of the grep, I see some text property changes and some content changes. But the content changes seems to appear in program-generated buffers so I’m not sure if `with-silent-modifications’ would do any harm anyway.

So, uh, maybe subr-x.el?

Yuan


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

* Re: Move 'with-buffer-modified-unmodified' to simple.el
  2020-09-18 18:25   ` Yuan Fu
@ 2020-09-18 19:56     ` Stefan Monnier
  2020-10-01 16:24       ` Yuan Fu
  0 siblings, 1 reply; 6+ messages in thread
From: Stefan Monnier @ 2020-09-18 19:56 UTC (permalink / raw)
  To: Yuan Fu; +Cc: emacs-devel

> How should we determine if the extra bindings in `with-silent-modifications’
> aren’t right? In my understanding `with-silent-modifications’ should only be
> used when changing text properties and if you actually insert something you
> should use `with-buffer-modified-unmodified’. Is that correct?

I think so, yes.

> Looking though the result of the grep, I see some text property changes and
> some content changes.

How many cases (approximately) involve content changes?

> But the content changes seems to appear in program-generated buffers
> so I’m not sure if `with-silent-modifications’ would do any
> harm anyway.

`with-silent-modifications’ would prevent jit-lock from doing its job,
so it would interfere with the use (within that buffer) of any package
that depends on jit-lock, such as font-lock-mode, nlinum-mode,
nhexl-mode, goto-address-mode, bug-reference-mode, glasses-mode.


        Stefan




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

* Re: Move 'with-buffer-modified-unmodified' to simple.el
  2020-09-18 19:56     ` Stefan Monnier
@ 2020-10-01 16:24       ` Yuan Fu
  2020-10-01 16:51         ` Stefan Monnier
  0 siblings, 1 reply; 6+ messages in thread
From: Yuan Fu @ 2020-10-01 16:24 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel


> 
>> Looking though the result of the grep, I see some text property changes and
>> some content changes.
> 
> How many cases (approximately) involve content changes?


In total, about 33 content changes (c) and 9 property changes (p), 20 other cases (n), 7 that I don't understand (?).

lisp/allout-widgets.el          p (decorate)
lisp/allout-widgets.el          p
lisp/allout-widgets.el          p
lisp/allout.el                  c (encryption)
lisp/allout.el                  p (annotate)
lisp/arc-mode.el                c (archive summary)
lisp/autoinsert.el              c (insert default content)
lisp/bindings.el                ? (mode-line-toggle-modified?)
lisp/bookmark.el                n (with-buffer-modified-unmodified)
lisp/bookmark.el                c (insert)
lisp/buff-menu.el               n (set to non modify)
lisp/calc/calc-embed.el         n (update modeline)
lisp/calc/calc-embed.el         n (same)
lisp/calc/calc-embed.el         n (same)
lisp/calc/calc-embed.el         n (same)
lisp/calc/calc-embed.el         n (same)
lisp/calc/calc.el               n (same)
lisp/calc/calc.el               n (same)
lisp/dired-aux.el               c (rename)
lisp/dired-x.el                 c (omit)
lisp/ehelp.el                   c (erase-buffer)
lisp/emacs-lisp/cl-lib.el       n
lisp/emacs-lisp/testcover.el    p (overlay)
lisp/emulation/edt.el           ? ((set-buffer-modified-p t)?)
lisp/erc/erc.el                 c (prompt)
lisp/files.el                   c (insert)
lisp/format.el                  c (format file content)
lisp/gnus/gnus-group.el         n (update modeline)
lisp/help-mode.el               c (insert)
lisp/ibuffer.el                 ?
lisp/ibuffer.el                 n (toggle modify)
lisp/image-mode.el              p
lisp/international/kkc.el       c (convert)
lisp/international/quail.el     ?
lisp/international/quail.el     c (insert)
lisp/language/china-util.el     c (decode)
lisp/language/tibet-util.el     c
lisp/language/viet-util.el      c (decode)
lisp/leim/quail/uni-input.el    c
lisp/mail/feedmail.el           c (insert)
lisp/mh-e/mh-acros.el           n (macro)
lisp/mh-e/mh-folder.el          n (set)
lisp/mh-e/mh-mime.el            c
lisp/mh-e/mh-search.el          c
lisp/mh-e/mh-search.el          c
lisp/mh-e/mh-show.el            c
lisp/mh-e/mh-show.el            c
lisp/mh-e/mh-thread.el          c
lisp/mh-e/mh-thread.el          c
lisp/mh-e/mh-utils.el           c
lisp/mh-e/mh-xface.el           c
lisp/net/tramp-adb.el           c
lisp/net/tramp-cmds.el          ?
lisp/net/tramp-sh.el            ?
lisp/net/tramp-smb.el           ?
lisp/obsolete/fast-lock.el
lisp/obsolete/longlines.el
lisp/obsolete/longlines.el
lisp/obsolete/longlines.el
lisp/org/org-macs.el            p (org-unmodified)
lisp/org/org-src.el             c
lisp/org/org.el                 p
lisp/progmodes/idlwave.el       n (update modeline)
lisp/simple.el                  n undo
lisp/textmodes/ispell.el        p
lisp/textmodes/reftex-global.el n (update modeline)
lisp/type-break.el              n (auto save)
lisp/vc/ediff-ptch.el           n 
lisp/vc/ediff-util.el           c
nextstep/Emacs.app/Contents/Resources/lisp/mh-e/mh-acros.el     c

Yuan


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

* Re: Move 'with-buffer-modified-unmodified' to simple.el
  2020-10-01 16:24       ` Yuan Fu
@ 2020-10-01 16:51         ` Stefan Monnier
  0 siblings, 0 replies; 6+ messages in thread
From: Stefan Monnier @ 2020-10-01 16:51 UTC (permalink / raw)
  To: Yuan Fu; +Cc: emacs-devel

>>> Looking though the result of the grep, I see some text property changes and
>>> some content changes.
>> How many cases (approximately) involve content changes?
> In total, about 33 content changes (c) and 9 property changes (p), 20 other
> cases (n), 7 that I don't understand (?).

Thanks.  So it looks like we have enough cases to justify such a macro.


        Stefan




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

end of thread, other threads:[~2020-10-01 16:51 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-09-16 23:12 Move 'with-buffer-modified-unmodified' to simple.el Yuan Fu
2020-09-17  3:11 ` Stefan Monnier
2020-09-18 18:25   ` Yuan Fu
2020-09-18 19:56     ` Stefan Monnier
2020-10-01 16:24       ` Yuan Fu
2020-10-01 16:51         ` Stefan Monnier

Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.