strepsirrhini/libreoffice-online
3
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Minor cleanups in DocumentBroker

Browse source 
documentLastModifiedTime was an unnecessary value parameter, filename
was copy-initialized without a reason.

Change-Id: I5e5a2fe9d9fcde61db539e30689492b8c191b598
This commit is contained in:
Miklos Vajna 2019-01-02 09:07:06 +01:00
parent 4866ec47bf
commit 3d38a71d77
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
wsd/DocumentBroker.cpp
View file

@ -61,7 +61,7 @@ namespace
void sendLastModificationTime(const std::shared_ptr<Session>& session,
DocumentBroker* documentBroker,
Poco::Timestamp documentLastModifiedTime)
const Poco::Timestamp& documentLastModifiedTime)
{
if (!session)
return;
@ -878,7 +878,7 @@ bool DocumentBroker::saveToStorageInternal(const std::string& sessionId,
// normalize the url (mainly to " " -> "%20")
const std::string url = Poco::URI(storageSaveResult.getSaveAsUrl()).toString();
const std::string filename = storageSaveResult.getSaveAsName();
const std::string& filename = storageSaveResult.getSaveAsName();
// encode the name
std::string encodedName;