diff options
author | Jeff Brown <jeffbrown@google.com> | 2011-07-13 22:22:02 -0700 |
---|---|---|
committer | Jeff Brown <jeffbrown@google.com> | 2011-07-14 04:11:22 -0700 |
commit | 11bf79d6e3542e40ebd212b0f5d7cb3370f3068d (patch) | |
tree | bca3c01bec346cc3c9dc6ab1ad45a8cb73062ef6 /api | |
parent | 9302c8796fc4dcda08d4bd1e11733848fd4fafaf (diff) | |
download | frameworks_base-11bf79d6e3542e40ebd212b0f5d7cb3370f3068d.zip frameworks_base-11bf79d6e3542e40ebd212b0f5d7cb3370f3068d.tar.gz frameworks_base-11bf79d6e3542e40ebd212b0f5d7cb3370f3068d.tar.bz2 |
Replace Vector _grow/_shrink checks with assert.
On review of the code, _grow and _shrink are checking for conditions
that cannot happen and that don't even really make sense. For
example, if _shrink is called with where + amount > mCount then
this is really bad, however the check only considered the case
when where >= mCount and then it would arbitrarily choose a new
value for where. Huh?
As it happens, the callers are correctly validating the
arguments before passing them down to these methods so we can
get rid of this code.
Change-Id: I921852dba8997065bb0e9cac733e82028d14afcd
Diffstat (limited to 'api')
0 files changed, 0 insertions, 0 deletions