[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#22763: 25.1.50; Feature Request -- A faster method to obtain line nu

From: Keith David Bershatsky
Subject: bug#22763: 25.1.50; Feature Request -- A faster method to obtain line number at position.
Date: Sun, 21 Feb 2016 18:42:52 -0800

A while back, I posed a question on emacs.stackexchange.com for a faster method 
to obtain `line-number-at-pos`.  A user by the name of Constantine came up with 
`(format-mode-line "%l")`, which is indeed much faster.  I think the draft code 
below may be just a tad faster, with the added feature of the POS argument.


A user named wasamasa posted a comment: "Be aware that this method will give 
you "??" for lines exceeding line-number-display-limit-width which is set to a 
value of 200 per default as I found out here."

And Stefan posted a comment:  "IIRC the result may also be unreliable if there 
have been modifications in the buffer since the last redisplay."

The thread has received 555 hits in the past year, and several have star-ed it 
and up-voted the question and answer.

The following is a draft in C based on the existing function `decode_mode_spec` 
that lets the user input the POS as an argument without the necessity to use 
`goto-char`.  I'm not sure if it resolves either of the comments above.  The 
draft is not meant to be a patch per se, because I'm not a programmer and am 
just beginning my tinkering quest into the language of C.  I have been using it 
in my own setup for about a week and I haven't seen any ill effects.  It can of 
course use some TLC by someone more knowledgeable than myself.

static const char *
internal_line_number_at_position (struct window *w, register int c, int 
field_width, Lisp_Object *string)
  Lisp_Object obj;
  struct frame *f = XFRAME (WINDOW_FRAME (w));
  char *decode_mode_spec_buf = f->decode_mode_spec_buffer;
  /* We are going to use f->decode_mode_spec_buffer as the buffer to
     produce strings from numerical values, so limit preposterously
     large values of FIELD_WIDTH to avoid overrunning the buffer's
     end.  The size of the buffer is enough for FRAME_MESSAGE_BUF_SIZE
     bytes plus the terminating null.  */
  int width = min (field_width, FRAME_MESSAGE_BUF_SIZE (f));
  struct buffer *b = current_buffer;
  obj = Qnil;
  *string = Qnil;
  switch (c)
    case 'l':
  ptrdiff_t startpos, startpos_byte, line, linepos, linepos_byte;
  ptrdiff_t topline, nlines, height;
  ptrdiff_t junk;
  /* %c and %l are ignored in `frame-title-format'.  */
  if (mode_line_target == MODE_LINE_TITLE)
    return "";
  startpos = marker_position (w->start);
  startpos_byte = marker_byte_position (w->start);
  height = WINDOW_TOTAL_LINES (w);
  /* If we decided that this buffer isn't suitable for line numbers,
     don't forget that too fast.  */
  if (w->base_line_pos == -1)
    goto no_value;
  /* If the buffer is very big, don't waste time.  */
  if (INTEGERP (Vline_number_display_limit)
      && BUF_ZV (b) - BUF_BEGV (b) > XINT (Vline_number_display_limit))
      w->base_line_pos = 0;
      w->base_line_number = 0;
      goto no_value;
  if (w->base_line_number > 0
      && w->base_line_pos > 0
      && w->base_line_pos <= startpos)
      line = w->base_line_number;
      linepos = w->base_line_pos;
      linepos_byte = buf_charpos_to_bytepos (b, linepos);
      line = 1;
      linepos = BUF_BEGV (b);
      linepos_byte = BUF_BEGV_BYTE (b);
  /* Count lines from base line to window start position.  */
  nlines = display_count_lines (linepos_byte,
              startpos, &junk);
  topline = nlines + line;
  /* Determine a new base line, if the old one is too close
     or too far away, or if we did not have one.
     "Too close" means it's plausible a scroll-down would
     go back past it.  */
  if (startpos == BUF_BEGV (b))
      w->base_line_number = topline;
      w->base_line_pos = BUF_BEGV (b);
  else if (nlines < height + 25 || nlines > height * 3 + 50
     || linepos == BUF_BEGV (b))
      ptrdiff_t limit = BUF_BEGV (b);
      ptrdiff_t limit_byte = BUF_BEGV_BYTE (b);
      ptrdiff_t position;
      ptrdiff_t distance =
        (height * 2 + 30) * line_number_display_limit_width;
      if (startpos - distance > limit)
    limit = startpos - distance;
    limit_byte = CHAR_TO_BYTE (limit);
      nlines = display_count_lines (startpos_byte,
            - (height * 2 + 30),
      /* If we couldn't find the lines we wanted within
         line_number_display_limit_width chars per line,
         give up on line numbers for this window.  */
      if (position == limit_byte && limit == startpos - distance)
    w->base_line_pos = -1;
    w->base_line_number = 0;
    goto no_value;
      w->base_line_number = topline - nlines;
      w->base_line_pos = BYTE_TO_CHAR (position);
  /* Now count lines from the start pos to point.  */
  nlines = display_count_lines (startpos_byte,
              PT_BYTE, PT, &junk);
  /* Record that we did display the line number.  */
  line_number_displayed = true;
  /* Make the string to show.  */
  pint2str (decode_mode_spec_buf, width, topline + nlines);
  return decode_mode_spec_buf;
    char *p = decode_mode_spec_buf;
    int pad = width - 2;
    while (pad-- > 0)
      *p++ = ' ';
    *p++ = '?';
    *p++ = '?';
    *p = '\0';
    return decode_mode_spec_buf;
  if (STRINGP (obj))
      *string = obj;
      return SSDATA (obj);
    return "";

DEFUN ("line-number-at-position", Fline_number_at_position, 
Sline_number_at_position, 1, 2, 0,
       doc: /* Return line number at position.  */)
  (Lisp_Object window, Lisp_Object pos)
  struct window *w = decode_live_window (window);
  Lisp_Object buf;
  struct buffer *b;
  buf = w->contents;
  b = XBUFFER (buf);
  struct buffer *old_buffer = NULL;
  Lisp_Object foo_string;
  const char *spec;
  EMACS_INT opoint;
  EMACS_INT posint;
  Lisp_Object value;
  if (b != current_buffer)
      old_buffer = current_buffer;
      set_buffer_internal (b);
  if (!NILP (pos))
      opoint = PT;
      posint = XINT (pos);
      SET_PT (posint);
  spec = internal_line_number_at_position (w, 'l', 0, &foo_string);
  if (!NILP (pos))
    SET_PT (opoint);
  if (old_buffer)
    set_buffer_internal (old_buffer);
  value = build_string (spec);
  value = Fstring_to_number (value, make_number (10));
  return value;

DEFSYM (Qline_number_at_position, "line-number-at-position");

reply via email to

[Prev in Thread] Current Thread [Next in Thread]