LGTM, patched and checked in.
>>>>> "Tyler" == Tyler Spivey <tspivey@xxxxxxxxxxx> writes:
Tyler> I have had issues with Gnus reading too little or too
Tyler> much. I think that this patch solves them: 1. Patches
Tyler> emacspeak-gnus-speak-article-body to use the
Tyler> dimensions of the window containing the article
Tyler> buffer.
Tyler>
Tyler> 2. patches gnus-summary-prev-page and
Tyler> gnus-summary-next-page to use move-to-window-line
Tyler> instead of forward-line and window-height in order to
Tyler> avoid reading too much due to the mode line being
Tyler> included in window-height.
Tyler>
Tyler> 3. Does the same to
Tyler> gnus-summary-kill-same-subject-and-select.
Tyler>
Tyler> Index: emacspeak-gnus.el
Tyler> ===================================================================
Tyler> --- emacspeak-gnus.el (revision 5939) +++
Tyler> emacspeak-gnus.el (working copy) @@ -155,13 +155,16 @@
Tyler> (emacspeak-dtk-sync) (cond ((< (count-lines
Tyler> (point-min) (point-max)) -
Tyler> emacspeak-gnus-large-article) +
Tyler> emacspeak-gnus-large-article) (emacspeak-speak-buffer
Tyler> )) (t (emacspeak-auditory-icon 'large-movement ) -
Tyler> (let ((start (point))) - (move-to-window-line -1) -
Tyler> (end-of-line) - (emacspeak-speak-region start
Tyler> (point))))))) + (let ((start (point)) + (window
Tyler> (get-buffer-window (current-buffer)))) +
Tyler> (with-selected-window window + (save-excursion +
Tyler> (move-to-window-line -1) + (end-of-line) +
Tyler> (emacspeak-speak-region start (point)))))))))
Tyler>
Tyler> ;;}}} ;;{{{ Advise top-level gnus command @@ -648,8
Tyler> +651,11 @@ (emacspeak-dtk-sync) (let ((start (point ))
Tyler> (window (get-buffer-window (current-buffer )))) -
Tyler> (forward-line (window-height window)) -
Tyler> (emacspeak-speak-region start (point )))))) +
Tyler> (with-selected-window window + (save-excursion +
Tyler> (move-to-window-line -1) + (end-of-line) +
Tyler> (emacspeak-speak-region start (point ))))))))
Tyler>
Tyler> (defadvice gnus-summary-kill-same-subject (after
Tyler> emacspeak pre act) "Speak the line. @@ -726,8 +732,11
Tyler> @@ (set-buffer "*Article*") (let ((start (point ))
Tyler> (window (get-buffer-window (current-buffer )))) -
Tyler> (forward-line (window-height window)) -
Tyler> (emacspeak-speak-region start (point ))))) +
Tyler> (with-selected-window window + (save-excursion +
Tyler> (move-to-window-line -1) + (end-of-line) +
Tyler> (emacspeak-speak-region start (point )))))))
Tyler>
Tyler> (defadvice gnus-summary-prev-page (after emacspeak
Tyler> pre act) "Speak the previous pageful " @@ -737,8
Tyler> +746,11 @@ (set-buffer "*Article*") (let ((start
Tyler> (point )) (window (get-buffer-window (current-buffer
Tyler> )))) - (forward-line (- (window-height window))) -
Tyler> (emacspeak-speak-region start (point ))))) +
Tyler> (with-selected-window window + (save-excursion +
Tyler> (move-to-window-line -1) + (end-of-line) +
Tyler> (emacspeak-speak-region start (point )))))))
Tyler>
Tyler> (defadvice gnus-summary-beginning-of-article (after
Tyler> emacspeak pre act) "Speak the first
Tyler> line. "(save-excursion
Tyler>
Tyler> -----------------------------------------------------------------------------
Tyler> To unsubscribe from the emacspeak list or change your
Tyler> address on the emacspeak list send mail to
Tyler> "emacspeak-request@xxxxxxxxxxx" with a subject of
Tyler> "unsubscribe" or "help"
--
Best Regards,
--raman
Email: raman@xxxxxxxxxxx
WWW: http://emacspeak.sf.net/raman/
AIM: emacspeak GTalk: tv.raman.tv@xxxxxxxxxxx
PGP: http://emacspeak.sf.net/raman/raman-almaden.asc
Google: tv+raman
IRC: irc://irc.freenode.net/#emacs
-----------------------------------------------------------------------------
To unsubscribe from the emacspeak list or change your address on the
emacspeak list send mail to "emacspeak-request@xxxxxxxxxxx" with a
subject of "unsubscribe" or "help"
If you have questions about this archive or had problems using it, please send mail to:
priestdo@xxxxxxxxxxx No Soliciting!Emacspeak List Archive | 2007 | 2006 | 2005 | 2004 | 2003 | 2002 | 2001 | 2000 | 1999 | 1998 | Pre 1998