public inbox for
 help / color / mirror / Atom feed
* "int pid" replaced with ProcessIdentifier
@ 2008-02-07 15:52 Andrew Cagney
  2008-02-14 11:00 ` Mark Wielaard
  0 siblings, 1 reply; 3+ messages in thread
From: Andrew Cagney @ 2008-02-07 15:52 UTC (permalink / raw)
  To: frysk


I've pushed a low-level change so that frysk.sys's process code uses a 
ProcessIdentifier object instead of "int pid"s; there is one 
ProcessIdentifier per "pid".  The change is slowly working its way 
through; it won't be taken further.

The motivation is simple, while "int pid" looks small and efficient; it 
turns out that the code using it really needed an object (leaking 
something like <<new Integer(pid)>>) to either log or hash the pid.  
With a proper object both of these cases can be greatly simplified.


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-02-14 14:25 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-02-07 15:52 "int pid" replaced with ProcessIdentifier Andrew Cagney
2008-02-14 11:00 ` Mark Wielaard
2008-02-14 14:25   ` Andrew Cagney

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).