Mercurial > octave
annotate scripts/strings/deblank.m @ 12975:f89449967f0d
deblank.m, strtrim.m: Correctly validate cellstr inputs (Bug #34038)
deblank.m, strtrim.m: Use iscellstr() instead of iscell() to validate input.
author | Rik <octave@nomad.inbox5.com> |
---|---|
date | Thu, 18 Aug 2011 22:23:16 -0700 |
parents | 1c71c9bf0570 |
children | 4ec4096f65d1 |
rev | line source |
---|---|
11523 | 1 ## Copyright (C) 1996-2011 Kurt Hornik |
2325 | 2 ## |
2313 | 3 ## This file is part of Octave. |
4 ## | |
5 ## Octave is free software; you can redistribute it and/or modify it | |
6 ## under the terms of the GNU General Public License as published by | |
7016 | 7 ## the Free Software Foundation; either version 3 of the License, or (at |
8 ## your option) any later version. | |
2313 | 9 ## |
10 ## Octave is distributed in the hope that it will be useful, but | |
11 ## WITHOUT ANY WARRANTY; without even the implied warranty of | |
12 ## MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | |
13 ## General Public License for more details. | |
14 ## | |
15 ## You should have received a copy of the GNU General Public License | |
7016 | 16 ## along with Octave; see the file COPYING. If not, see |
17 ## <http://www.gnu.org/licenses/>. | |
2268 | 18 |
3361 | 19 ## -*- texinfo -*- |
20 ## @deftypefn {Function File} {} deblank (@var{s}) | |
12925
1c71c9bf0570
deblank.m: Speed up 15x on cellstr inputs. Restrict input to strings or cellstrings.
Rik <octave@nomad.inbox5.com>
parents:
11589
diff
changeset
|
21 ## Remove trailing whitespace and nulls from @var{s}. If @var{s} |
5462 | 22 ## is a matrix, @var{deblank} trims each row to the length of longest |
23 ## string. If @var{s} is a cell array, operate recursively on each | |
24 ## element of the cell array. | |
3361 | 25 ## @end deftypefn |
2311 | 26 |
5428 | 27 ## Author: Kurt Hornik <Kurt.Hornik@wu-wien.ac.at> |
2355 | 28 ## Adapted-By: jwe |
2314 | 29 |
5462 | 30 function s = deblank (s) |
2325 | 31 |
2268 | 32 if (nargin != 1) |
6046 | 33 print_usage (); |
2268 | 34 endif |
2325 | 35 |
12925
1c71c9bf0570
deblank.m: Speed up 15x on cellstr inputs. Restrict input to strings or cellstrings.
Rik <octave@nomad.inbox5.com>
parents:
11589
diff
changeset
|
36 if (ischar (s)) |
2268 | 37 |
12925
1c71c9bf0570
deblank.m: Speed up 15x on cellstr inputs. Restrict input to strings or cellstrings.
Rik <octave@nomad.inbox5.com>
parents:
11589
diff
changeset
|
38 k = find (! isspace (s) & s != "\0"); |
1c71c9bf0570
deblank.m: Speed up 15x on cellstr inputs. Restrict input to strings or cellstrings.
Rik <octave@nomad.inbox5.com>
parents:
11589
diff
changeset
|
39 if (isempty (s) || isempty (k)) |
1c71c9bf0570
deblank.m: Speed up 15x on cellstr inputs. Restrict input to strings or cellstrings.
Rik <octave@nomad.inbox5.com>
parents:
11589
diff
changeset
|
40 s = ""; |
1c71c9bf0570
deblank.m: Speed up 15x on cellstr inputs. Restrict input to strings or cellstrings.
Rik <octave@nomad.inbox5.com>
parents:
11589
diff
changeset
|
41 else |
1c71c9bf0570
deblank.m: Speed up 15x on cellstr inputs. Restrict input to strings or cellstrings.
Rik <octave@nomad.inbox5.com>
parents:
11589
diff
changeset
|
42 s = s(:,1:ceil (max (k) / rows (s))); |
2268 | 43 endif |
44 | |
12975
f89449967f0d
deblank.m, strtrim.m: Correctly validate cellstr inputs (Bug #34038)
Rik <octave@nomad.inbox5.com>
parents:
12925
diff
changeset
|
45 elseif (iscellstr (s)) |
5462 | 46 |
12925
1c71c9bf0570
deblank.m: Speed up 15x on cellstr inputs. Restrict input to strings or cellstrings.
Rik <octave@nomad.inbox5.com>
parents:
11589
diff
changeset
|
47 s = regexprep (s, "[\\s\v\\0]+$", ''); |
5462 | 48 |
2268 | 49 else |
12925
1c71c9bf0570
deblank.m: Speed up 15x on cellstr inputs. Restrict input to strings or cellstrings.
Rik <octave@nomad.inbox5.com>
parents:
11589
diff
changeset
|
50 error ("deblank: S argument must be a string or cellstring"); |
2268 | 51 endif |
52 | |
53 endfunction | |
7411 | 54 |
7429 | 55 |
12925
1c71c9bf0570
deblank.m: Speed up 15x on cellstr inputs. Restrict input to strings or cellstrings.
Rik <octave@nomad.inbox5.com>
parents:
11589
diff
changeset
|
56 %!assert (strcmp (deblank (" f o o \0"), " f o o")); |
7429 | 57 %!assert (deblank (' '), '') |
58 %!assert (deblank (" "), "") | |
12925
1c71c9bf0570
deblank.m: Speed up 15x on cellstr inputs. Restrict input to strings or cellstrings.
Rik <octave@nomad.inbox5.com>
parents:
11589
diff
changeset
|
59 %!assert (deblank (""), "") |
1c71c9bf0570
deblank.m: Speed up 15x on cellstr inputs. Restrict input to strings or cellstrings.
Rik <octave@nomad.inbox5.com>
parents:
11589
diff
changeset
|
60 %!assert (deblank ({}), {}) |
7429 | 61 |
12925
1c71c9bf0570
deblank.m: Speed up 15x on cellstr inputs. Restrict input to strings or cellstrings.
Rik <octave@nomad.inbox5.com>
parents:
11589
diff
changeset
|
62 %!error <Invalid call to deblank> deblank (); |
1c71c9bf0570
deblank.m: Speed up 15x on cellstr inputs. Restrict input to strings or cellstrings.
Rik <octave@nomad.inbox5.com>
parents:
11589
diff
changeset
|
63 %!error <Invalid call to deblank> deblank ("foo", "bar"); |
1c71c9bf0570
deblank.m: Speed up 15x on cellstr inputs. Restrict input to strings or cellstrings.
Rik <octave@nomad.inbox5.com>
parents:
11589
diff
changeset
|
64 %!error <argument must be a string> deblank (1); |
7429 | 65 |