unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: "Drew Adams" <drew.adams@oracle.com>
To: 7390@debbugs.gnu.org
Subject: bug#7390: 24.0.50; pos-col-row gives incorrect row
Date: Sat, 13 Nov 2010 09:55:15 -0800	[thread overview]
Message-ID: <9C53EFF0802044A489EF842E6A1B8150@us.oracle.com> (raw)

emacs -Q
 
Load David Ponce's tabbar.el version 1.3, from here:
http://sourceforge.net/projects/emhacks/files/tabbar/1.3/tabbar-1.3.tar.gz/downl
oad
 
M-x tabbar-mode
 
That adds a tab bar to each window.  I don't see anything in the
tabbar.el code that should perturb the behavior of pos-col-row, but
something does.  I'm no expert on the tabbar code or pos-col-row.  I
hope that someone can debug this and fix it.  My guess is that
pos-col-row is incorrectly thinking that the tab bar is part of the
buffer.
 
The easiest way to see the problem is to do this:
 
M-x picture-mode
 
Click mouse-1 somewhere - e.g. the fourth char in some buffer.  The
result is that the cursor (point) is moved down one line from where you
clicked, so the active region shows from the click position to point
(one line down).
 
Do the same thing with tabbar-mode turned off, and there is no problem:
point does not move down a line: it is exactly where you clicked.
 
The problem arises in picture-mouse-set-point, where it calls
pos-col-row: the wrong row is returned.  This is presumably because it
is counting the tabbar row, which is not part of the buffer.
 
Dunno whether this is a bug in pos-col-row (that would be my guess) or
in tabbar.el.  I shouldn't think that tabbar.el should need to do
something to compensate and make pos-col-row DTRT.  But again, I'm no
expert on this.

In GNU Emacs 24.0.50.1 (i386-mingw-nt5.1.2600)
 of 2010-11-09 on 3249CTO
Windowing system distributor `Microsoft Corp.', version 5.1.2600
configured using `configure --with-gcc (4.4) --no-opt --cflags
-Ic:/imagesupport/include'
 






             reply	other threads:[~2010-11-13 17:55 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-13 17:55 Drew Adams [this message]
2010-11-13 19:02 ` bug#7390: 24.0.50; pos-col-row gives incorrect row Eli Zaretskii
2010-11-13 19:17   ` Drew Adams
2010-11-13 21:10   ` Eli Zaretskii
2010-11-13 21:30     ` Eli Zaretskii
2010-11-13 21:37       ` Chong Yidong
2010-11-13 22:00         ` Eli Zaretskii
2010-11-16 19:59           ` Chong Yidong
2010-12-12  7:15     ` YAMAMOTO Mitsuharu
2010-12-12  7:29       ` Eli Zaretskii
2010-12-12  7:37         ` YAMAMOTO Mitsuharu
2010-12-12 20:47           ` Eli Zaretskii
2010-11-13 21:13   ` Chong Yidong
2010-11-13 21:21     ` Eli Zaretskii
2010-11-13 21:38     ` Drew Adams
2010-11-13 22:31       ` Eli Zaretskii
2010-11-13 21:44     ` Chong Yidong
2010-11-13 21:55       ` Eli Zaretskii
2010-11-14 17:35         ` Chong Yidong

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=9C53EFF0802044A489EF842E6A1B8150@us.oracle.com \
    --to=drew.adams@oracle.com \
    --cc=7390@debbugs.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).