diff options
| author | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-18 22:13:24 +0000 |
|---|---|---|
| committer | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-18 22:13:24 +0000 |
| commit | 1ea8b13da3556cc2678bd8071ffffd23f9a05b6d (patch) | |
| tree | 3e769c91c966b0fe0c9c42b9feaf5f50f3911efa /printing/print_settings.h | |
| parent | deb958caa1a2d2390633e5b8194bc57c23fc4a7d (diff) | |
| download | chromium_src-1ea8b13da3556cc2678bd8071ffffd23f9a05b6d.zip chromium_src-1ea8b13da3556cc2678bd8071ffffd23f9a05b6d.tar.gz chromium_src-1ea8b13da3556cc2678bd8071ffffd23f9a05b6d.tar.bz2 | |
Small cleanups:
* SplitUrlAndTitle() can't fail, so it doesn't need to return a bool.
* assign() can do substrings in one shot
* Half the code used "*str =" and half "str->assign()", often right next to each other; pick one for consistency
* No need to calculate array sizes when we're the ones declaring them
* Use early return in a few places
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/405023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32414 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'printing/print_settings.h')
0 files changed, 0 insertions, 0 deletions