diff options
| author | mdempsky@chromium.org <mdempsky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-08 23:29:30 +0000 |
|---|---|---|
| committer | mdempsky@chromium.org <mdempsky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-08 23:29:30 +0000 |
| commit | a1c9554114647996f50e05ea5ee95390125ebc30 (patch) | |
| tree | 3fcf56a8d59e8030004face6970bc4b5a3562911 | |
| parent | 5243bce526df42358983d40686475959a89ecaf5 (diff) | |
| download | chromium_src-a1c9554114647996f50e05ea5ee95390125ebc30.zip chromium_src-a1c9554114647996f50e05ea5ee95390125ebc30.tar.gz chromium_src-a1c9554114647996f50e05ea5ee95390125ebc30.tar.bz2 | |
Fix typo (approve -> approach) and cleanup wording while here
Review URL: https://codereview.chromium.org/229173002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@262551 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | base/posix/file_descriptor_shuffle.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/base/posix/file_descriptor_shuffle.h b/base/posix/file_descriptor_shuffle.h index 9cd918f..5fa590b 100644 --- a/base/posix/file_descriptor_shuffle.h +++ b/base/posix/file_descriptor_shuffle.h @@ -5,10 +5,10 @@ #ifndef BASE_POSIX_FILE_DESCRIPTOR_SHUFFLE_H_ #define BASE_POSIX_FILE_DESCRIPTOR_SHUFFLE_H_ -// This code exists to perform the shuffling of file descriptors which is -// commonly needed when forking subprocesses. The naive approve is very simple, -// just call dup2 to setup the desired descriptors, but wrong. It's tough to -// handle the edge cases (like mapping 0 -> 1, 1 -> 0) correctly. +// This code exists to shuffle file descriptors, which is commonly needed when +// forking subprocesses. The naive approach (just call dup2 to set up the +// desired descriptors) is very simple, but wrong: it won't handle edge cases +// (like mapping 0 -> 1, 1 -> 0) correctly. // // In order to unittest this code, it's broken into the abstract action (an // injective multimap) and the concrete code for dealing with file descriptors. |