From: Stefan Monnier <monnier@iro.umontreal.ca>
To: joakim@verona.se
Cc: emacs-devel@gnu.org
Subject: Re: Adding parameters to windows
Date: Fri, 23 May 2008 15:47:17 -0400 [thread overview]
Message-ID: <jwvlk203jav.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <m3d4ndun5g.fsf@verona.se> (joakim@verona.se's message of "Fri, 23 May 2008 16:10:19 +0200")
Your patch needs a few tweaks, e.g. to adjust it to the coding
conventions:
- The docstring of window-parameters is poor.
E.g. the first line doesn't say anything. The second line is indented
in the source, which looks bad when you do C-h f window-parameters.
nil is an aliast as well, so there's no need to mention it explicitly.
- I'd prefer to have `window-parameter' rather than `window-parameters'.
- The docstring of set-window-parameter needs to start with a capital
letter and end with a "." and should mention its arguments.
Feel free to plagiarize set-frame-parameter.
- The comment for window_parameters refers to `pin' and `group'.
- See http://www.gnu.org/prep/standards/standards.html#Change-Logs for
the format to use in ChangeLogs:
- put together changes to a file.
- your changelog seems to say you've changed `DEFUN'.
- there's a missing space after colon in "(window:Added".
- And the "Added" should be "Add" of course.
Thank you very much for your help. BTW, could you point me to your
latest version of the pin&group patch?
Stefan
> === modified file 'src/window.c'
> --- src/window.c 2008-04-03 02:15:43 +0000
> +++ src/window.c 2008-04-29 09:40:34 +0000
> @@ -275,6 +278,7 @@
p-> frame = Qnil;
p-> display_table = Qnil;
p-> dedicated = Qnil;
> + p->window_parameters = Qnil;
p-> pseudo_window_p = 0;
> bzero (&p->cursor, sizeof (p->cursor));
> bzero (&p->last_cursor, sizeof (p->last_cursor));
> @@ -1325,6 +1329,38 @@
> return w->dedicated;
> }
> +DEFUN ("window-parameters", Fwindow_parameters, Swindow_parameters,
> + 1, 1, 0,
> + doc: /* Return WINDOW's window-parameters.
> + returns nil or an alist where values affect window operations. */)
> + (window)
> + Lisp_Object window;
> +{
> + return decode_window (window)->window_parameters;
> +}
> +
> +
> +DEFUN ("set-window-parameter", Fset_window_parameter,
> + Sset_window_parameter, 3, 3, 0,
> + doc: /* set window parameters */)
> + (window, prop, val)
> + Lisp_Object window, prop, val;
> +{
> + register struct window *w = decode_window (window);
> + register Lisp_Object old_alist_elt;
> +
> + old_alist_elt = Fassq (prop, w->window_parameters);
> + if (EQ (old_alist_elt, Qnil))
> + w->window_parameters = Fcons (Fcons (prop, val), w->window_parameters);
> + else
> + Fsetcdr (old_alist_elt, val);
> +
> + //Fplist_put(w->window_parameters,key,value);
> +
> + return w->window_parameters;
> +}
> +
> +
> DEFUN ("window-display-table", Fwindow_display_table, Swindow_display_table,
> 0, 1, 0,
> doc: /* Return the display-table that WINDOW is using.
> @@ -7714,6 +7771,9 @@
> defsubr (&Sset_window_vscroll);
> defsubr (&Scompare_window_configurations);
> defsubr (&Swindow_list);
> + defsubr (&Swindow_parameters);
> + defsubr (&Sset_window_parameter);
> +
> }
> void
> === modified file 'src/window.h'
> --- src/window.h 2008-01-29 02:05:10 +0000
> +++ src/window.h 2008-04-29 10:04:07 +0000
> @@ -229,6 +229,11 @@
> enlarged. */
> Lisp_Object orig_total_lines, orig_top_line;
> + /* an alist with flags that modifies behaviour of certain window operations.
> + currently "pin" and "group" are special
> + */
> + Lisp_Object window_parameters;
> +
> /* No Lisp data may follow below this point without changing
> mark_object in alloc.c. The member current_matrix must be the
> first non-Lisp member. */
> --
> Joakim Verona
next prev parent reply other threads:[~2008-05-23 19:47 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-15 17:51 Adding parameters to windows Stefan Monnier
2008-05-23 14:10 ` joakim
2008-05-23 19:47 ` Stefan Monnier [this message]
2008-05-23 20:28 ` joakim
2008-06-04 11:01 ` joakim
2008-06-04 11:18 ` David Kastrup
2008-06-04 11:24 ` joakim
2008-06-04 11:46 ` David Kastrup
2008-06-04 15:10 ` Stefan Monnier
2008-06-04 15:58 ` joakim
2008-06-04 17:24 ` Stefan Monnier
2008-06-04 21:13 ` Juanma Barranquero
2008-06-04 21:25 ` joakim
2008-06-04 21:56 ` Juanma Barranquero
2008-06-04 22:05 ` joakim
2008-06-04 21:27 ` Juanma Barranquero
2008-06-05 2:07 ` Stefan Monnier
2008-06-05 3:25 ` Juanma Barranquero
-- strict thread matches above, loose matches on Subject: below --
2008-05-16 23:37 Seiji Zenitani
2008-05-20 1:17 ` Stefan Monnier
2008-06-03 9:04 ` Seiji Zenitani
2008-06-03 17:38 ` Glenn Morris
2008-06-04 6:46 ` Jan Djärv
2008-06-04 7:08 ` Glenn Morris
2008-06-04 9:10 ` Miles Bader
2008-06-05 1:45 ` Seiji Zenitani
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=jwvlk203jav.fsf-monnier+emacs@gnu.org \
--to=monnier@iro.umontreal.ca \
--cc=emacs-devel@gnu.org \
--cc=joakim@verona.se \
/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 external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.