[Libreoffice-commits] core.git: Branch 'feature/orcus-update' - 2 commits - framework/source sc/source sfx2/inc sfx2/source starmath/source sw/source
Kohei Yoshida
kohei.yoshida at gmail.com
Tue Apr 9 09:58:11 PDT 2013
framework/source/loadenv/loadenv.cxx | 2 +
sc/source/ui/unoobj/scdetect.cxx | 8 ++--
sfx2/inc/sfx2/docfilt.hxx | 64 ++++++++++++++++++-----------------
sfx2/source/bastyp/fltfnc.cxx | 4 +-
sfx2/source/dialog/filedlghelper.cxx | 2 -
sfx2/source/doc/docfilt.cxx | 60 ++++++++++++++++++--------------
sfx2/source/doc/objcont.cxx | 2 -
starmath/source/document.cxx | 2 -
sw/source/core/doc/docglbl.cxx | 2 -
sw/source/ui/app/docsh.cxx | 2 -
sw/source/ui/dbui/dbmgr.cxx | 2 -
11 files changed, 80 insertions(+), 70 deletions(-)
New commits:
commit b2288e8282bce091c5a31b85e3fe75666140dcda
Author: Kohei Yoshida <kohei.yoshida at gmail.com>
Date: Tue Apr 9 13:00:25 2013 -0400
Get the whole thing to build after the SfxFilter interface change.
Change-Id: I62122e298f1434aa9e44c8f3fdfb7f710513617d
diff --git a/framework/source/loadenv/loadenv.cxx b/framework/source/loadenv/loadenv.cxx
index 33956f1..86646d5 100644
--- a/framework/source/loadenv/loadenv.cxx
+++ b/framework/source/loadenv/loadenv.cxx
@@ -744,6 +744,7 @@ bool queryOrcusTypeAndFilter(const uno::Sequence<beans::PropertyValue>& rDescrip
rFilter = "gnumeric";
return true;
}
+#if 0
else if (aURL.endsWith(".xlsx"))
{
rType = "generic_Text";
@@ -756,6 +757,7 @@ bool queryOrcusTypeAndFilter(const uno::Sequence<beans::PropertyValue>& rDescrip
rFilter = "ods";
return true;
}
+#endif
return false;
}
diff --git a/sc/source/ui/unoobj/scdetect.cxx b/sc/source/ui/unoobj/scdetect.cxx
index e40be82..6d8a7dc 100644
--- a/sc/source/ui/unoobj/scdetect.cxx
+++ b/sc/source/ui/unoobj/scdetect.cxx
@@ -432,9 +432,9 @@ OUString SAL_CALL ScFilterDetect::detect( uno::Sequence<beans::PropertyValue>& l
SvStream* pStream = aMedium.GetInStream();
const SfxFilter* pPreselectedFilter = pFilter;
bool bCsvSelected = (pPreselectedFilter &&
- pPreselectedFilter->GetFilterName().EqualsAscii( pFilterAscii ));
+ pPreselectedFilter->GetFilterName().equalsAscii(pFilterAscii));
bool bExcelSelected = (pPreselectedFilter &&
- (pPreselectedFilter->GetName().SearchAscii("Excel") != STRING_NOTFOUND));
+ (pPreselectedFilter->GetName().indexOf("Excel") >= 0));
bool bIsXLS = (bExcelSelected || (bCsvSelected && !aPreselectedFilterName.Len()));
pFilter = 0;
if ( pStream )
@@ -671,7 +671,7 @@ OUString SAL_CALL ScFilterDetect::detect( uno::Sequence<beans::PropertyValue>& l
pFilter = aMatcher.GetFilter4FilterName( OUString(pFilterAscii) );
bFakeXLS = true;
}
- else if ( pPreselectedFilter->GetName().EqualsAscii(pFilterDBase) && lcl_MayBeDBase( rStr ) )
+ else if ( pPreselectedFilter->GetName().equalsAscii(pFilterDBase) && lcl_MayBeDBase( rStr ) )
pFilter = pPreselectedFilter;
else if ( bCsvSelected && bMaybeText )
pFilter = pPreselectedFilter;
@@ -686,7 +686,7 @@ OUString SAL_CALL ScFilterDetect::detect( uno::Sequence<beans::PropertyValue>& l
// "foo <br> bar" with a preselected CSV
// filter. So keep this detection to the end.
- if ( pPreselectedFilter->GetName().EqualsAscii(pFilterHtml) )
+ if (pPreselectedFilter->GetName().equalsAscii(pFilterHtml))
{
pFilter = pPreselectedFilter;
}
diff --git a/starmath/source/document.cxx b/starmath/source/document.cxx
index 37e0b4d..0e2aa3e 100644
--- a/starmath/source/document.cxx
+++ b/starmath/source/document.cxx
@@ -970,7 +970,7 @@ sal_Bool SmDocShell::ConvertTo( SfxMedium &rMedium )
aEquation.SetFlat(sal_True);
bRet = aEquation.Export(rMedium);
}
- else if( pFlt->GetFilterName().EqualsAscii("MathType 3.x"))
+ else if (pFlt->GetFilterName().equalsAscii("MathType 3.x"))
bRet = WriteAsMathType3( rMedium );
}
return bRet;
diff --git a/sw/source/core/doc/docglbl.cxx b/sw/source/core/doc/docglbl.cxx
index ddac41e..5c0eb98 100644
--- a/sw/source/core/doc/docglbl.cxx
+++ b/sw/source/core/doc/docglbl.cxx
@@ -214,7 +214,7 @@ bool SwDoc::SplitDoc( sal_uInt16 eDocType, const String& rPath, bool bOutline, c
GetIDocumentUndoRedo().DoUndo(false);
SetRedlineMode_intern( (RedlineMode_t)(GetRedlineMode() & ~nsRedlineMode_t::REDLINE_ON));
- String sExt( pFilter->GetSuffixes().GetToken(0, ',') );
+ String sExt = pFilter->GetSuffixes().getToken(0, ',');
if( !sExt.Len() )
sExt.AssignAscii( "sxw" );
if( '.' != sExt.GetChar( 0 ) )
diff --git a/sw/source/ui/app/docsh.cxx b/sw/source/ui/app/docsh.cxx
index e961da1..3a3194a 100644
--- a/sw/source/ui/app/docsh.cxx
+++ b/sw/source/ui/app/docsh.cxx
@@ -193,7 +193,7 @@ Reader* SwDocShell::StartConvertFrom(SfxMedium& rMedium, SwReader** ppRdr,
SFX_ITEMSET_ARG( rMedium.GetItemSet(), pUpdateDocItem, SfxUInt16Item, SID_UPDATEDOCMODE, sal_False);
nUpdateDocMode = pUpdateDocItem ? pUpdateDocItem->GetValue() : document::UpdateDocMode::NO_UPDATE;
- if( pFlt->GetDefaultTemplate().Len() )
+ if (!pFlt->GetDefaultTemplate().isEmpty())
pRead->SetTemplateName( pFlt->GetDefaultTemplate() );
if( pRead == ReadAscii && 0 != rMedium.GetInStream() &&
diff --git a/sw/source/ui/dbui/dbmgr.cxx b/sw/source/ui/dbui/dbmgr.cxx
index c7ce039..51db9f3 100644
--- a/sw/source/ui/dbui/dbmgr.cxx
+++ b/sw/source/ui/dbui/dbmgr.cxx
@@ -1140,7 +1140,7 @@ sal_Bool SwNewDBMgr::MergeMailFiles(SwWrtShell* pSourceShell,
//convert fields to text if we are exporting to PDF
//this prevents a second merge while updating the fields in SwXTextDocument::getRendererCount()
- if( pStoreToFilter && pStoreToFilter->GetFilterName().EqualsAscii("writer_pdf_Export"))
+ if( pStoreToFilter && pStoreToFilter->GetFilterName().equalsAscii("writer_pdf_Export"))
rWorkShell.ConvertFieldsToText();
xWorkDocSh->DoSaveAs(*pDstMed);
xWorkDocSh->DoSaveCompleted(pDstMed);
commit 3443d47a62820e611aaab523f1e405bbb1557f5a
Author: Kohei Yoshida <kohei.yoshida at gmail.com>
Date: Tue Apr 9 12:22:06 2013 -0400
String cleanup inside SfxFilter.
Change-Id: I647b9e4de54fdcfadcf17ac7e161b245f3dea546
diff --git a/sfx2/inc/sfx2/docfilt.hxx b/sfx2/inc/sfx2/docfilt.hxx
index 18ccfc8..8400e5a 100644
--- a/sfx2/inc/sfx2/docfilt.hxx
+++ b/sfx2/inc/sfx2/docfilt.hxx
@@ -42,30 +42,32 @@ class SFX2_DLLPUBLIC SfxFilter
friend class SfxFilterContainer;
WildCard aWildCard;
- sal_uIntPtr lFormat;
- String aTypeName;
- OUString aUserData;
+
+ OUString aTypeName;
+ OUString aUserData;
+ OUString aServiceName;
+ OUString aMimeType;
+ OUString aFilterName;
+ OUString aPattern;
+ OUString aUIName;
+ OUString aDefaultTemplate;
+
SfxFilterFlags nFormatType;
- sal_uInt16 nDocIcon;
- OUString aServiceName;
- OUString aMimeType;
- String aFilterName;
- String aPattern;
sal_uIntPtr nVersion;
- String aUIName;
- String aDefaultTemplate;
+ sal_uIntPtr lFormat;
+ sal_uInt16 nDocIcon;
public:
- SfxFilter( const String &rName,
- const String &rWildCard,
- SfxFilterFlags nFormatType,
- sal_uInt32 lFormat,
- const String &rTypeName,
- sal_uInt16 nDocIcon,
- const String &rMimeType,
- const String &rUserData,
- const String& rServiceName );
- ~SfxFilter();
+ SfxFilter( const OUString &rName,
+ const OUString &rWildCard,
+ SfxFilterFlags nFormatType,
+ sal_uInt32 lFormat,
+ const OUString &rTypeName,
+ sal_uInt16 nDocIcon,
+ const OUString &rMimeType,
+ const OUString &rUserData,
+ const OUString& rServiceName );
+ ~SfxFilter();
bool IsAllowedAsTemplate() const { return nFormatType & SFX_FILTER_TEMPLATE; }
bool IsOwnFormat() const { return nFormatType & SFX_FILTER_OWN; }
@@ -75,26 +77,26 @@ public:
bool CanExport() const { return nFormatType & SFX_FILTER_EXPORT; }
bool IsInternal() const { return nFormatType & SFX_FILTER_INTERNAL; }
SfxFilterFlags GetFilterFlags() const { return nFormatType; }
- const String& GetFilterName() const { return aFilterName; }
+ const OUString& GetFilterName() const { return aFilterName; }
const OUString& GetMimeType() const { return aMimeType; }
- const String& GetName() const { return aFilterName; }
+ const OUString& GetName() const { return aFilterName; }
const WildCard& GetWildcard() const { return aWildCard; }
- const String& GetRealTypeName() const { return aTypeName; }
+ const OUString& GetRealTypeName() const { return aTypeName; }
sal_uIntPtr GetFormat() const { return lFormat; }
- const String& GetTypeName() const { return aTypeName; }
- const String& GetUIName() const { return aUIName; }
+ const OUString& GetTypeName() const { return aTypeName; }
+ const OUString& GetUIName() const { return aUIName; }
sal_uInt16 GetDocIconId() const { return nDocIcon; }
const OUString& GetUserData() const { return aUserData; }
- const String& GetDefaultTemplate() const { return aDefaultTemplate; }
+ const OUString& GetDefaultTemplate() const { return aDefaultTemplate; }
void SetDefaultTemplate( const String& rStr ) { aDefaultTemplate = rStr; }
- sal_Bool UsesStorage() const { return GetFormat() != 0; }
- void SetURLPattern( const String& rStr ) { aPattern = rStr; aPattern.ToLowerAscii(); }
- String GetURLPattern() const { return aPattern; }
+ bool UsesStorage() const { return GetFormat() != 0; }
+ void SetURLPattern( const OUString& rStr );
+ OUString GetURLPattern() const { return aPattern; }
void SetUIName( const String& rName ) { aUIName = rName; }
void SetVersion( sal_uIntPtr nVersionP ) { nVersion = nVersionP; }
sal_uIntPtr GetVersion() const { return nVersion; }
- String GetSuffixes() const;
- String GetDefaultExtension() const;
+ OUString GetSuffixes() const;
+ OUString GetDefaultExtension() const;
const OUString& GetServiceName() const { return aServiceName; }
static const SfxFilter* GetDefaultFilter( const String& rName );
diff --git a/sfx2/source/bastyp/fltfnc.cxx b/sfx2/source/bastyp/fltfnc.cxx
index 83181be..78b3607 100644
--- a/sfx2/source/bastyp/fltfnc.cxx
+++ b/sfx2/source/bastyp/fltfnc.cxx
@@ -809,7 +809,7 @@ const SfxFilter* SfxFilterMatcher::GetFilter4FilterName( const String& rName, Sf
{
const SfxFilter* pFilter = (*pFilterArr)[i];
SfxFilterFlags nFlags = pFilter->GetFilterFlags();
- if ( (nFlags & nMust) == nMust && !(nFlags & nDont ) && pFilter->GetFilterName().CompareIgnoreCaseToAscii( aName ) == COMPARE_EQUAL )
+ if ((nFlags & nMust) == nMust && !(nFlags & nDont) && pFilter->GetFilterName().equalsIgnoreAsciiCase(aName))
return pFilter;
}
}
@@ -826,7 +826,7 @@ const SfxFilter* SfxFilterMatcher::GetFilter4FilterName( const String& rName, Sf
{
const SfxFilter* pFilter = (*pList)[i];
SfxFilterFlags nFlags = pFilter->GetFilterFlags();
- if ( (nFlags & nMust) == nMust && !(nFlags & nDont ) && pFilter->GetFilterName().CompareIgnoreCaseToAscii( aName ) == COMPARE_EQUAL )
+ if ( (nFlags & nMust) == nMust && !(nFlags & nDont ) && pFilter->GetFilterName().equalsIgnoreAsciiCase(aName))
return pFilter;
}
diff --git a/sfx2/source/dialog/filedlghelper.cxx b/sfx2/source/dialog/filedlghelper.cxx
index ee8da3d..88fd6aa 100644
--- a/sfx2/source/dialog/filedlghelper.cxx
+++ b/sfx2/source/dialog/filedlghelper.cxx
@@ -1636,7 +1636,7 @@ void FileDialogHelper_Impl::getRealFilter( String& _rFilter ) const
{
const SfxFilter* pFilter =
mpMatcher->GetFilter4UIName( _rFilter, m_nMustFlags, m_nDontFlags );
- _rFilter = pFilter ? pFilter->GetFilterName() : _rFilter.Erase();
+ _rFilter = pFilter ? String(pFilter->GetFilterName()) : _rFilter.Erase();
}
}
diff --git a/sfx2/source/doc/docfilt.cxx b/sfx2/source/doc/docfilt.cxx
index a6ed5aa..ce95b9b 100644
--- a/sfx2/source/doc/docfilt.cxx
+++ b/sfx2/source/doc/docfilt.cxx
@@ -40,67 +40,73 @@ using namespace ::com::sun::star;
DBG_NAME(SfxFilter)
-SfxFilter::SfxFilter( const String &rName,
- const String &rWildCard,
- SfxFilterFlags nType,
- sal_uInt32 lFmt,
- const String &rTypNm,
- sal_uInt16 nIcon,
- const String &rMimeType,
- const String &rUsrDat,
- const String &rServiceName ):
+SfxFilter::SfxFilter( const OUString &rName,
+ const OUString &rWildCard,
+ SfxFilterFlags nType,
+ sal_uInt32 lFmt,
+ const OUString &rTypNm,
+ sal_uInt16 nIcon,
+ const OUString &rMimeType,
+ const OUString &rUsrDat,
+ const OUString &rServiceName ):
aWildCard(rWildCard, ';'),
- lFormat(lFmt),
aTypeName(rTypNm),
aUserData(rUsrDat),
+ aServiceName(rServiceName),
+ aMimeType(rMimeType),
+ aFilterName(rName),
+ aUIName(aFilterName),
nFormatType(nType),
- nDocIcon(nIcon),
- aServiceName( rServiceName ),
- aMimeType( rMimeType ),
- aFilterName( rName )
+ nVersion(SOFFICE_FILEFORMAT_50),
+ lFormat(lFmt),
+ nDocIcon(nIcon)
{
- String aExts = GetWildcard().getGlob();
- String aShort, aLong;
- String aRet;
+ OUString aExts = GetWildcard().getGlob();
+ OUString aShort, aLong;
+ OUString aRet;
sal_uInt16 nMaxLength = USHRT_MAX;
OUString aTest;
sal_uInt16 nPos = 0;
- while( ( aRet = aExts.GetToken( nPos++, ';' ) ).Len() )
+ while (!(aRet = aExts.getToken(nPos++, ';')).isEmpty() )
{
aTest = aRet;
aTest = aTest.replaceFirst( "*." , "" );
if( aTest.getLength() <= nMaxLength )
{
- if( aShort.Len() ) aShort += ';';
+ if (!aShort.isEmpty())
+ aShort += ";";
aShort += aRet;
}
else
{
- if( aLong.Len() ) aLong += ';';
+ if (!aLong.isEmpty())
+ aLong += ";";
aLong += aRet;
}
}
- if( aShort.Len() && aLong.Len() )
+ if (!aShort.isEmpty() && !aLong.isEmpty())
{
- aShort += ';';
+ aShort += ";";
aShort += aLong;
}
aWildCard.setGlob(aShort);
-
- nVersion = SOFFICE_FILEFORMAT_50;
- aUIName = aFilterName;
}
SfxFilter::~SfxFilter()
{
}
-String SfxFilter::GetDefaultExtension() const
+OUString SfxFilter::GetDefaultExtension() const
{
return comphelper::string::getToken(GetWildcard().getGlob(), 0, ';');
}
-String SfxFilter::GetSuffixes() const
+void SfxFilter::SetURLPattern( const OUString& rStr )
+{
+ aPattern = rStr.toAsciiLowerCase();
+}
+
+OUString SfxFilter::GetSuffixes() const
{
String aRet = GetWildcard().getGlob();
while( aRet.SearchAndReplaceAscii( "*.", String() ) != STRING_NOTFOUND ) ;
diff --git a/sfx2/source/doc/objcont.cxx b/sfx2/source/doc/objcont.cxx
index e6722b7..a03879c 100644
--- a/sfx2/source/doc/objcont.cxx
+++ b/sfx2/source/doc/objcont.cxx
@@ -807,7 +807,7 @@ void SfxObjectShell::UpdateFromTemplate_Impl( )
sal_Bool SfxObjectShell::IsHelpDocument() const
{
const SfxFilter* pFilter = GetMedium()->GetFilter();
- return ( pFilter && pFilter->GetFilterName().CompareToAscii("writer_web_HTML_help") == COMPARE_EQUAL );
+ return (pFilter && pFilter->GetFilterName() == "writer_web_HTML_help");
}
void SfxObjectShell::ResetFromTemplate( const String& rTemplateName, const String& rFileName )
More information about the Libreoffice-commits
mailing list