summaryrefslogtreecommitdiff
path: root/unotools
diff options
context:
space:
mode:
authorStephan Bergmann <sbergman@redhat.com>2015-04-16 16:34:34 +0200
committerStephan Bergmann <sbergman@redhat.com>2015-04-16 17:11:40 +0200
commitbfedaea4921a3bba10e1cdb566901978b6dbffc2 (patch)
tree2715dfeb1318b1454f634004c70257c8430afd8d /unotools
parent3e55cf49f406076e1bdb491f8b3d484b766d5c72 (diff)
improve comment
Change-Id: I451ab0fb0e7ee1a6d20091caa9c1142b16796dc7
Diffstat (limited to 'unotools')
-rw-r--r--unotools/source/config/historyoptions.cxx11
1 files changed, 5 insertions, 6 deletions
diff --git a/unotools/source/config/historyoptions.cxx b/unotools/source/config/historyoptions.cxx
index e7c28067823d..a9acd9aa8ae0 100644
--- a/unotools/source/config/historyoptions.cxx
+++ b/unotools/source/config/historyoptions.cxx
@@ -315,12 +315,11 @@ Sequence< Sequence<PropertyValue> > SvtHistoryOptions_Impl::GetList(EHistoryType
xOrderList->getByName(OUString::number(nItem)) >>= xSet;
xSet->getPropertyValue(s_sHistoryItemRef) >>= sUrl;
- // Check if file is openable and on a local filesystem.
- // Windows UNC pathes like \\server.domain\file.odt map to
- // file://server.domain/file.odt. Therefore, we require a beginning
- // slash which supprisingly also works for local files on Windows
- // as they map to file:///C:/folder/file.odt. Remote files may
- // cause hangs if the share is unavailable. See tdf#89394
+ // Check if file is openable, but for performance reasons try to
+ // only do so for files on a local filesystem. For Windows,
+ // checking for "file:///" nicely filters out UNC paths (that only
+ // have two slashes), but of course misses to filter out remote
+ // mounts on Unix-like systems:
if (!sUrl.startsWith("file:///") || lcl_fileOpenable(sUrl))
{
xItemList->getByName(sUrl) >>= xSet;