Skip to content
This repository was archived by the owner on Jan 30, 2020. It is now read-only.

Commit 45ce8d9

Browse files
committed
Updated comments to use @todo annotation
Also updated to provide correct grammar, as well as formatting.
1 parent ae7a465 commit 45ce8d9

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/Decode.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -122,19 +122,19 @@ public static function splitMessage($message, &$headers, &$body, $EOL = Mime::LI
122122
}
123123
}
124124

125-
// TODO: splitMime replaces removes \r which breaks
126-
// valid mime messages as returned by many mail servers
125+
// @todo splitMime removes "\r" sequences, which breaks valid mime
126+
// messages as returned by many mail servers
127127
$headersEOL = $EOL;
128128

129129
// find an empty line between headers and body
130130
// default is set new line
131-
// TODO: Maybe this is too much "magic", we should be more strict here
131+
// @todo Maybe this is too much "magic"; we should be more strict here
132132
if (strpos($message, $EOL . $EOL)) {
133133
list($headers, $body) = explode($EOL . $EOL, $message, 2);
134134
// next is the standard new line
135135
} elseif ($EOL != "\r\n" && strpos($message, "\r\n\r\n")) {
136136
list($headers, $body) = explode("\r\n\r\n", $message, 2);
137-
$headersEOL = "\r\n"; // Headers::fromString will fail with incorrect eol
137+
$headersEOL = "\r\n"; // Headers::fromString will fail with incorrect EOL
138138
// next is the other "standard" new line
139139
} elseif ($EOL != "\n" && strpos($message, "\n\n")) {
140140
list($headers, $body) = explode("\n\n", $message, 2);

0 commit comments

Comments
 (0)