[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
Re: bug#34160: About commit bff64115a0ad081282e0f99305f41c8dd1917d67, bu
From:
Lars Ingebrigtsen
Subject:
Re: bug#34160: About commit bff64115a0ad081282e0f99305f41c8dd1917d67, bug#34160, json.el
Date:
2019年7月31日 22:21:10 +0200
User-agent:
Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux)
Tassilo Horn <address@hidden> writes:
> It's not just moving point. replace-region-contents also keeps marks,
> text properties and fontification intact. So we should definitely be
> using it here.
Ah, I see. I only gave that function a cursory look-over when fixing
this bug and I didn't quite understand what it was doing here, since I
couldn't recall any other pretty-printer doing anything similar.
Sorry for the confusion here; I've now restored the
replace-region-contents logic. Or at least I think so; it works with
the test cases in this bug report, at least.
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no