On Monday, June 28, 2010 15:53:11 Garrett Cooper wrote:
> On Mon, Jun 28, 2010 at 4:52 AM, Subrata Modak wrote:
> > +       if [ $ALT_DMESG_OUT -eq 1 ] ; then
> > +               PWD=`pwd`
> 
> Please use a different variable name. This obfuscates the meaning of
> the shell set value.

use a `find` as i suggested and changing dir isnt necessary.  it replaces the 
entire body of the if statement.
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

------------------------------------------------------------------------------
This SF.net email is sponsored by Sprint
What will you do first with EVO, the first 4G phone?
Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to