From: Cyril Hrubis <chrubis@suse.cz>
To: Zeng Linggang <zenglg.jy@cn.fujitsu.com>
Cc: ltp-list <ltp-list@lists.sourceforge.net>
Subject: Re: [LTP] [RFC PATCH v2 1/2] testcases/lib/test.sh: added SAFE_CALL*()
Date: Thu, 23 Apr 2015 12:27:25 +0200 [thread overview]
Message-ID: <20150423102725.GA18237@rei.suse.de> (raw)
In-Reply-To: <1429755598.2416.14.camel@G08JYZSD130126.localdomain>
Hi!
> > Looks good but the names are a bit confusing. The SAFE_CALL does
> > redirect the output while SAFWE_CALL_REDIRECT does not.
> >
>
> If *users* want to redirect the output they need use SAFE_REDIRECT. :-/
I know how the name was choosen, but that does not change the fact that
it was unintuitive :).
--
Cyril Hrubis
chrubis@suse.cz
------------------------------------------------------------------------------
BPM Camp - Free Virtual Workshop May 6th at 10am PDT/1PM EDT
Develop your own process in accordance with the BPMN 2 standard
Learn Process modeling best practices with Bonita BPM through live exercises
http://www.bonitasoft.com/be-part-of-it/events/bpm-camp-virtual- event?utm_
source=Sourceforge_BPM_Camp_5_6_15&utm_medium=email&utm_campaign=VA_SF
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
next prev parent reply other threads:[~2015-04-23 10:27 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-02 10:35 [LTP] [RFC PATCH] commands/du: Added new testcase to test du(1) command Zeng Linggang
2015-04-02 11:59 ` Cyril Hrubis
[not found] ` <1429010688.7190.24.camel@G08JYZSD130126.localdomain>
2015-04-22 12:47 ` [LTP] [RFC PATCH v2 1/2] testcases/lib/test.sh: added SAFE_CALL*() Cyril Hrubis
[not found] ` <1429755598.2416.14.camel@G08JYZSD130126.localdomain>
2015-04-23 10:27 ` Cyril Hrubis [this message]
[not found] ` <1429010747.7190.25.camel@G08JYZSD130126.localdomain>
2015-04-22 12:58 ` [LTP] [RFC PATCH v2 2/2] commands/du: Added new testcase to test du(1) command Cyril Hrubis
[not found] ` <1429784431-14017-1-git-send-email-zenglg.jy@cn.fujitsu.com>
2015-04-28 14:10 ` [LTP] [PATCH v3 1/2] testcases/lib/test.sh: Added ROD_SILENT() and ROD() Cyril Hrubis
[not found] ` <1430270054.7745.11.camel@G08JYZSD130126.localdomain>
[not found] ` <55417DFD.1040207@huawei.com>
2015-04-30 7:51 ` Cyril Hrubis
2015-04-30 7:52 ` Cyril Hrubis
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=20150423102725.GA18237@rei.suse.de \
--to=chrubis@suse.cz \
--cc=ltp-list@lists.sourceforge.net \
--cc=zenglg.jy@cn.fujitsu.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.
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.