sm-discuss AT lists.ibiblio.org
Subject: Public SourceMage Discussion List
List archive
- From: Donald Johnson <ltdonny AT gmail.com>
- To: Ismael Luceno <ismael.luceno AT gmail.com>
- Cc: sm-discuss AT lists.ibiblio.org
- Subject: Re: [SM-Discuss] banish
- Date: Sun, 2 May 2010 22:13:11 -0500
On 05/01/2010 06:55 PM, Ismael Luceno wrote:
This came out of left field. I like it.
A few things:
-Backticks aren't an issue for me, but some people might like to see $(...) for executing statements.
-Comments pweeze? :3
-Should get_new_orphans really sort? That might be something to put into the executing statements outside of the functions. Nitpicky.
-I spy a few tabs
-There's a use of the `dialog` program explicitly instead of using the variable.
It's simple, and it works. Approve.
Check it out, tell me what could be improved... I would like to make it part of sorcery soon.
This came out of left field. I like it.
A few things:
-Backticks aren't an issue for me, but some people might like to see $(...) for executing statements.
-Comments pweeze? :3
-Should get_new_orphans really sort? That might be something to put into the executing statements outside of the functions. Nitpicky.
-I spy a few tabs
-There's a use of the `dialog` program explicitly instead of using the variable.
It's simple, and it works. Approve.
-
[SM-Discuss] banish,
Ismael Luceno, 05/01/2010
- Re: [SM-Discuss] banish, Donald Johnson, 05/02/2010
-
Re: [SM-Discuss] banish,
Ismael Luceno, 05/03/2010
-
Re: [SM-Discuss] banish,
Eric Sandall, 05/04/2010
- Re: [SM-Discuss] banish, Ismael Luceno, 05/04/2010
-
Re: [SM-Discuss] banish,
Eric Sandall, 05/04/2010
Archive powered by MHonArc 2.6.24.