Re: [david.hansen@physik.fu-berlin.de: WoMan Bug]

[Michael Welsh Duggan <md5i@cs.cmu.edu>]

Michael Welsh Duggan <md5i@cs.cmu.edu> writes:

> Stefan Monnier <monnier@iro.umontreal.ca> writes:
>
>>>> Would someone please fix this, then ack?
>>
>>> The following patch would appear to fix this problem:
>>
>> I believe the right patch is to change the marker's insertion type (so that
>> insertions are automatically made "before").  The patch below does that at
>> the spot caught by the OP.  There might be other places where we need some
>> fixup, so please try it and tell me if you hit the "Internal error" check
>> or not (and if you do, please provide a backtrace).
>
> Actually, this patch doesn't appear to change the marker insertion
> type at all.  Was something left out?

I take that back.  I simply misunderstood the code.  Ignore also my
followup patch, in favor of the following:

--- woman.el	04 Sep 2006 13:03:49 -0400	1.46
+++ woman.el	15 Sep 2006 01:57:31 -0400	
@@ -3442,9 +3442,7 @@
 (defsubst woman2-process-escapes-to-eol (&optional numeric)
   "Process remaining escape sequences up to eol.
 Handle numeric arguments specially if optional argument NUMERIC is non-nil."
-  (woman2-process-escapes
-   (save-excursion (end-of-line) (point-marker))
-   numeric))
+  (woman2-process-escapes (end-of-line) numeric))
 
 (defun woman2-nr (to)
   ".nr R +/-N M -- Assign +/-N (wrt to previous value, if any) to register R.
@@ -3645,6 +3643,7 @@
 	(woman-registers woman-registers)
 	fn request translations
 	tab-stop-list)
+    (set-marker-insertion-type to t)
     ;; ?roff does not squeeze multiple spaces, but does fill, so...
     (fset 'canonically-space-region 'ignore)
     ;; Try to avoid spaces inheriting underlines from preceding text!
@@ -3897,10 +3896,15 @@
 ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
 
 (defun woman2-process-escapes (to &optional numeric)
-  "Process remaining escape sequences up to marker TO, preserving point.
+  "Process remaining escape sequences up to TO, preserving point.
 Optional argument NUMERIC, if non-nil, means the argument is numeric."
   ;; The first two cases below could be merged (maybe)!
   (let ((from (point)))
+    ;; Make sure TO is a move-after-insertion kind of marker.
+    (if (markerp to)
+        (unless (marker-insertion-type to)
+          (error "Internal error: wrong marker type"))
+      (setq to (copy-marker to t)))
     ;; Discard zero width filler character used to hide leading dots
     ;; and zero width characters \|, \^:
     (while (re-search-forward "\\\\[&|^]" to t)

-- 
Michael Welsh Duggan
(md5i@cs.cmu.edu)