diff options
| author | Nicholas Nethercote <nnethercote@mozilla.com> | 2016-06-21 09:59:17 +1000 |
|---|---|---|
| committer | Nicholas Nethercote <nnethercote@mozilla.com> | 2016-06-21 09:59:17 +1000 |
| commit | 6cb6a8e8e4543fe35c6015ba6de6311a1f6a4829 (patch) | |
| tree | 1f6eab3a66669bd98fd86105d9ff465c46e221de /coreconf | |
| parent | f0cf1974eb79c8184341db0e4bc62e7edd13c5c0 (diff) | |
| download | nss-hg-6cb6a8e8e4543fe35c6015ba6de6311a1f6a4829.tar.gz | |
Bug 1280844 - Remove useless assert() in pathsub.c. r=mt.
The assert() assumes that |from| and |to| are non-null, which the subsequent
check does not assume. It's also redundant w.r.t. the subsequent checks, which
will cause informative termination on failure. It's also the only assert() in
this file.
MozReview-Commit-ID: GSAgvINeEpJ
Diffstat (limited to 'coreconf')
| -rw-r--r-- | coreconf/nsinstall/pathsub.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/coreconf/nsinstall/pathsub.c b/coreconf/nsinstall/pathsub.c index 315cda9ca..a42a9f30a 100644 --- a/coreconf/nsinstall/pathsub.c +++ b/coreconf/nsinstall/pathsub.c @@ -5,7 +5,6 @@ /* ** Pathname subroutines. */ -#include <assert.h> #if defined(FREEBSD) || defined(BSDI) || defined(DARWIN) #include <sys/types.h> #endif /* FREEBSD */ @@ -158,7 +157,6 @@ relatepaths(char *from, char *to, char *outpath) int len; char buf[NAME_MAX]; - assert(*from == '/' && *to == '/'); if (!from || *from != '/') fail("relatepaths: from path does not start with /"); if (!to || *to != '/') |