From: "Roland Winkler" <Roland.Winkler@physik.uni-erlangen.de>
To: Eli Zaretskii <eliz@gnu.org>
Cc: Juanma Barranquero <lekktu@gmail.com>, emacs-devel@gnu.org
Subject: Re: proced: ppid of process ID 0 can be 0
Date: Sat, 20 Dec 2008 20:41:20 +0100 [thread overview]
Message-ID: <18765.19040.568030.246241@tfkp04.physik.uni-erlangen.de> (raw)
In-Reply-To: <uhc4zdqrx.fsf@gnu.org>
On Sat Dec 20 2008 Eli Zaretskii wrote:
> > I see two ways of fixing this: either removing the assumption from
> > proced.el, or forcing the Windows implementation of
> > system_process_attributes to adapt. First is more correct, second is
> > much easier (see tiny patch below).
>
> I'd prefer that proced.el doesn't make such an unportable assumption.
I am not sure I get the point here. What is unportable where? I
mean, it seems to me that ultimately the solution is always the
same: when using the ppid attribute, one needs to make sure that it
is different from the corresponding pid. Or could one assign a
special meaning to the case that ppid=pid? There is no doubt that
proced assumes that ppid is always different from pid.
A comparison of ppid and pid can be implemented either on the level
of system_process_attributes. (I don't know whether for other
operating systems besides Windows we can have similar problems.) Or
it can be implemented inside proced. In the latter case, I would
implement the comparison of ppid and pid in
proced-process-attributes which is kind of the low-level function in
Proced calling system-process-attributes. (No other proced function
calls system-process-attributes.) So having the comparison of ppid
and pid in that function would work for proced for all operating
systems (requiring only a very small change in the code).
Yet anybody using system-process-attributes elsewhere might be
surprised that the return value of system-process-attributes can
give such meaningless results so that he/she probably would have to
implement the same check. So maybe for a clear definition of the
ppid attribute returned by system-process-attributes, we could
require that it must be different from pid.
Roland
next prev parent reply other threads:[~2008-12-20 19:41 UTC|newest]
Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-12-20 2:52 proced: ppid of process ID 0 can be 0 Juanma Barranquero
2008-12-20 3:27 ` Stefan Monnier
2008-12-20 4:31 ` Chong Yidong
2008-12-20 19:10 ` Stefan Monnier
2008-12-20 10:20 ` Juanma Barranquero
2008-12-20 11:34 ` Eli Zaretskii
2008-12-20 19:41 ` Roland Winkler [this message]
2008-12-20 21:02 ` Juanma Barranquero
2008-12-20 22:02 ` Eli Zaretskii
2008-12-20 22:41 ` Roland Winkler
2008-12-21 4:15 ` Eli Zaretskii
2008-12-21 4:48 ` Roland Winkler
2008-12-21 19:17 ` Eli Zaretskii
2008-12-21 21:26 ` Juanma Barranquero
2008-12-22 4:09 ` Eli Zaretskii
2008-12-22 10:41 ` Juanma Barranquero
2008-12-22 19:17 ` Eli Zaretskii
2008-12-22 19:26 ` Juanma Barranquero
2008-12-22 2:45 ` Stefan Monnier
2008-12-22 4:08 ` Eli Zaretskii
2008-12-22 4:17 ` Miles Bader
2008-12-22 19:10 ` Eli Zaretskii
2008-12-22 8:54 ` Harald Hanche-Olsen
2008-12-22 10:43 ` Juanma Barranquero
2008-12-22 10:50 ` Harald Hanche-Olsen
2008-12-22 18:03 ` Giorgos Keramidas
2008-12-22 10:59 ` Andreas Schwab
2008-12-22 19:22 ` Eli Zaretskii
2008-12-22 19:37 ` David De La Harpe Golden
2008-12-22 19:59 ` Andreas Schwab
2008-12-22 12:23 ` Stefan Monnier
2008-12-22 19:57 ` Eli Zaretskii
2008-12-22 22:24 ` Stefan Monnier
2008-12-23 4:02 ` Eli Zaretskii
2008-12-23 12:28 ` Roland Winkler
2008-12-23 14:44 ` Stefan Monnier
2008-12-23 19:12 ` Eli Zaretskii
2008-12-23 20:41 ` Stefan Monnier
2008-12-23 22:34 ` Giorgos Keramidas
2008-12-24 2:34 ` Stefan Monnier
2008-12-24 4:14 ` Eli Zaretskii
2008-12-24 16:47 ` Stefan Monnier
2008-12-24 4:15 ` Eli Zaretskii
2008-12-24 16:47 ` Stefan Monnier
2008-12-24 18:29 ` Eli Zaretskii
2008-12-24 18:55 ` Stefan Monnier
2008-12-24 21:12 ` Eli Zaretskii
2008-12-30 17:11 ` Stefan Monnier
2008-12-30 22:02 ` Eli Zaretskii
2008-12-31 1:40 ` Stefan Monnier
2008-12-22 8:28 ` Ulrich Mueller
2008-12-21 2:18 ` Stefan Monnier
-- strict thread matches above, loose matches on Subject: below --
2008-12-24 7:42 grischka
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=18765.19040.568030.246241@tfkp04.physik.uni-erlangen.de \
--to=roland.winkler@physik.uni-erlangen.de \
--cc=eliz@gnu.org \
--cc=emacs-devel@gnu.org \
--cc=lekktu@gmail.com \
/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.