Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(301)

Issue 1141010: Auxiliary function for args checks. (Closed)

Created:
10 years, 9 months ago by Luigi Semenzato
Modified:
9 years, 7 months ago
Reviewers:
adlr
CC:
chromium-os-reviews_chromium.org
Visibility:
Public.

Description

Auxiliary function for args checks.

Patch Set 1 #

Total comments: 1

Patch Set 2 : fix typo and reformat comment #

Unified diffs Side-by-side diffs Delta from patch set Stats (+23 lines, -0 lines) Patch
M src/scripts/common.sh View 1 1 chunk +23 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Luigi Semenzato
OK. I like your idea better.
10 years, 9 months ago (2010-03-23 17:28:56 UTC) #1
adlr
LGTM w/ comment http://codereview.chromium.org/1141010/diff/1/2 File src/scripts/common.sh (right): http://codereview.chromium.org/1141010/diff/1/2#newcode262 src/scripts/common.sh:262: # Usage: check_flags_only_and_allow_null_arg "$@" && set ...
10 years, 9 months ago (2010-03-23 17:40:34 UTC) #2
Luigi Semenzato
10 years, 9 months ago (2010-03-23 18:01:40 UTC) #3
Thanks.

For the record: adlr and I investigated this issue in depth, and found that the
meaning of -e applies only to isolated commands.  Expressions with multiple
operands and function calls do not force exit when -e is set.  For instance, the
expression "false" forces an exit, but "false && true" does not.  Another great
moment in the history of computing.

Powered by Google App Engine
This is Rietveld 408576698