summaryrefslogtreecommitdiffstats
path: root/mailnews/compose/src/nsMsgCompose.cpp
diff options
context:
space:
mode:
Diffstat (limited to 'mailnews/compose/src/nsMsgCompose.cpp')
-rw-r--r--mailnews/compose/src/nsMsgCompose.cpp71
1 files changed, 39 insertions, 32 deletions
diff --git a/mailnews/compose/src/nsMsgCompose.cpp b/mailnews/compose/src/nsMsgCompose.cpp
index 58340bffa..4ce8e8def 100644
--- a/mailnews/compose/src/nsMsgCompose.cpp
+++ b/mailnews/compose/src/nsMsgCompose.cpp
@@ -4350,45 +4350,52 @@ nsMsgCompose::LoadDataFromFile(nsIFile *file, nsString &sigData,
* images loaded into the editor are available on send.
*/
nsresult
-nsMsgCompose::ReplaceFileURLs(nsAutoString &aData)
+nsMsgCompose::ReplaceFileURLs(nsString &aData)
{
int32_t fPos;
- int32_t offset = -1;
+ int32_t offset = -1; // We're using RFind(), so offset -1 is from the very right.
+
+ // XXX This code is rather incomplete since it looks for "file://" even
+ // outside tags.
while ((fPos = aData.RFind("file://", true, offset)) != kNotFound) {
- if (fPos != kNotFound && fPos > 0) {
- char16_t q = aData.CharAt(fPos - 1);
- bool quoted = (q == '"' || q == '\'');
- int32_t end = kNotFound;
- if (quoted) {
- end = aData.FindChar(q, fPos);
+ bool quoted = false;
+ char16_t q = 'x'; // initialise to anything to keep compilers happy.
+ if (fPos > 0) {
+ q = aData.CharAt(fPos - 1);
+ quoted = (q == '"' || q == '\'');
+ }
+ int32_t end = kNotFound;
+ if (quoted) {
+ end = aData.FindChar(q, fPos);
+ }
+ else {
+ int32_t spacePos = aData.FindChar(' ', fPos);
+ int32_t gtPos = aData.FindChar('>', fPos);
+ if (gtPos != kNotFound && spacePos != kNotFound) {
+ end = (spacePos < gtPos) ? spacePos : gtPos;
}
- else {
- int32_t spacePos = aData.FindChar(' ', fPos);
- int32_t gtPos = aData.FindChar('>', fPos);
- if (gtPos != kNotFound && spacePos != kNotFound) {
- end = (spacePos < gtPos) ? spacePos : gtPos;
- }
- else if (gtPos == kNotFound && spacePos != kNotFound) {
- end = spacePos;
- }
- else if (gtPos != kNotFound && spacePos == kNotFound) {
- end = gtPos;
- }
+ else if (gtPos == kNotFound && spacePos != kNotFound) {
+ end = spacePos;
}
- if (end == kNotFound) {
- break;
+ else if (gtPos != kNotFound && spacePos == kNotFound) {
+ end = gtPos;
}
- nsString fileURL;
- fileURL = Substring(aData, fPos, end - fPos);
- nsString dataURL;
- nsresult rv = DataURLForFileURL(fileURL, dataURL);
- // If this one failed, maybe because the file wasn't found,
- // continue to process the next one.
- if (NS_SUCCEEDED(rv)) {
- aData.Replace(fPos, end - fPos, dataURL);
- }
- offset = fPos - 1;
}
+ if (end == kNotFound) {
+ break;
+ }
+ nsString fileURL;
+ fileURL = Substring(aData, fPos, end - fPos);
+ nsString dataURL;
+ nsresult rv = DataURLForFileURL(fileURL, dataURL);
+ // If this one failed, maybe because the file wasn't found,
+ // continue to process the next one.
+ if (NS_SUCCEEDED(rv)) {
+ aData.Replace(fPos, end - fPos, dataURL);
+ }
+ if (fPos == 0)
+ break;
+ offset = fPos - 1;
}
return NS_OK;
}