9fans - fans of the OS Plan 9 from Bell Labs
 help / color / mirror / Atom feed
* [9fans] the return of cat -v (NetBSD sources)
@ 2004-02-12  1:06 George Michaelson
  2004-02-12  1:59 ` Geoff Collyer
  0 siblings, 1 reply; 6+ messages in thread
From: George Michaelson @ 2004-02-12  1:06 UTC (permalink / raw)
  To: 9fans

from a posting to a NetBSD list...

-George

----
Hi people,

I'm not sure this is the right list, but I have noticed that the code for
/bin/cat includes several gotos which were inserted when the -f flag was
implemented (starting from line 252 on version 1.43).
To me these gotos look like a quick patch (especially with names like "skip" :))
and seem like a good start for a spaghetti code... but I'm no expert.

My question is this: wouldn't it be more clean to implement these using a flag?
This way the danger of code additions which will make "skip" move too far and
become obscure will disappear.

According to http://www.netbsd.org/Misc/features.html#clean-design, clean code
is quite important fot NetBSD.

If there's some optimization or other issues here, let me know.
If it's a matter of religious wars, well, I'll be happy to know too :)
And if it's a matter of finding  someone to make the change, again, let me know.

Thanks
    Ofer Waldman


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

end of thread, other threads:[~2004-02-12  3:00 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-02-12  1:06 [9fans] the return of cat -v (NetBSD sources) George Michaelson
2004-02-12  1:59 ` Geoff Collyer
2004-02-12  2:04   ` George Michaelson
2004-02-12  2:22     ` Bruce Ellis
2004-02-12  3:00     ` boyd, rounin
2004-02-12  2:58   ` boyd, rounin

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 NNTP newsgroup(s).