Hi Keita, 2017-04-19 20:42 GMT+02:00 Ikumi Keita <ik...@ikumi.que.jp>: > Hi all, > > I'm going to check in the following change. Since loading tex-jp.el > replaces `TeX-error-description-list' with Japanese help messages, > comparison with a fixed string "No help available" no longer works after > that. Rather we should test whether `TeX-error-pointer' comes to the > last element of `TeX-error-description-list'. > > diff --git a/tex-buf.el b/tex-buf.el > --- a/tex-buf.el > +++ b/tex-buf.el > @@ -3306,7 +3306,8 @@ > (let ((help (cdr (nth TeX-error-pointer > TeX-error-description-list)))) > (save-excursion > - (if (and (string= help "No help available") > + (if (and (= (1+ TeX-error-pointer) > + (length TeX-error-description-list)) > (let* ((log-buffer (find-buffer-visiting log-file))) > (if log-buffer > (progn > > Though I have done a brief test with this change, feel free to revert > the commit if you find any problems with this.
The fix looks sensible. A request: please add tests when possible. This ensures we won't break anything in the feature, this is particularly important in areas that usually receive less love from other developers (like support for Japanese), and so are more likely to be unintentionally broken. Thanks, Mosè _______________________________________________ auctex-devel mailing list auctex-devel@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-devel
