[PATCH] fdo#57950: Remove last chained appends from avmedia

Marcos Souza (via Code Review) gerrit at gerrit.libreoffice.org
Sat Jan 5 06:27:34 PST 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/1548

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/48/1548/1

fdo#57950: Remove last chained appends from avmedia

Change-Id: I0906bf7493828c5923f3243c6849a2eca5f37bb0
Signed-off-by: Marcos Paulo de Souza <marcos.souza.org at gmail.com>
---
M avmedia/source/framework/mediaitem.cxx
1 file changed, 1 insertion(+), 5 deletions(-)



diff --git a/avmedia/source/framework/mediaitem.cxx b/avmedia/source/framework/mediaitem.cxx
index 2776809..05c29aa 100644
--- a/avmedia/source/framework/mediaitem.cxx
+++ b/avmedia/source/framework/mediaitem.cxx
@@ -470,11 +470,7 @@
             xTransaction->commit();
         }
 
-        OUStringBuffer buf("vnd.sun.star.Package:");
-        buf.append(media);
-        buf.append(sal_Unicode('/'));
-        buf.append(filename);
-        o_rEmbeddedURL = buf.makeStringAndClear();
+        o_rEmbeddedURL = "vnd.sun.star.Package:" + media + "/" + filename;
         return true;
     }
     catch (uno::Exception const&)

-- 
To view, visit https://gerrit.libreoffice.org/1548
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0906bf7493828c5923f3243c6849a2eca5f37bb0
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Marcos Souza <marcos.souza.org at gmail.com>



More information about the LibreOffice mailing list