Kaydet (Commit) a18fc918 authored tarafından Andrea Gelmini's avatar Andrea Gelmini Kaydeden (comit) Oliver Specht

Fix typos

Change-Id: I051caee9ec6181074201ac4a8cb6fcec4fdf2293
Reviewed-on: https://gerrit.libreoffice.org/18951Reviewed-by: 's avatarOliver Specht <oliver.specht@cib.de>
Tested-by: 's avatarOliver Specht <oliver.specht@cib.de>
üst 1f03430e
...@@ -56,7 +56,7 @@ import static org.junit.Assert.*; ...@@ -56,7 +56,7 @@ import static org.junit.Assert.*;
* This <CODE>ComplexTest</CODE> checks the interface * This <CODE>ComplexTest</CODE> checks the interface
* <CODE>XUserInputInterception</CODE>. Therefore it creates a document, * <CODE>XUserInputInterception</CODE>. Therefore it creates a document,
* adds a mouse and a key listener onto the interface and fire the * adds a mouse and a key listener onto the interface and fire the
* correspond events. If all listener works as expected the test resluts in * correspond events. If all listener works as expected the test results in
* <CODE>OK</CODE> status. * <CODE>OK</CODE> status.
* @short Check the interface XUserInputIntercaption * @short Check the interface XUserInputIntercaption
* @descr checks is a simple way the interface XUserInputInteraction * @descr checks is a simple way the interface XUserInputInteraction
...@@ -307,8 +307,8 @@ public class EventTest { ...@@ -307,8 +307,8 @@ public class EventTest {
xUII.removeMouseClickHandler(mouseListener); xUII.removeMouseClickHandler(mouseListener);
} }
/** /*
* returns the <CODE>XUserInputInterception</CODE> from the <CODE>XMdoel</CODE> * returns the <CODE>XUserInputInterception</CODE> from the <CODE>XModel</CODE>
* @param xModel the XModel of a document * @param xModel the XModel of a document
* @return the <CODE>XUserInputInterception</CODE> of the document * @return the <CODE>XUserInputInterception</CODE> of the document
*/ */
......
...@@ -355,14 +355,14 @@ public class checkdispatchapi ...@@ -355,14 +355,14 @@ public class checkdispatchapi
{ {
// Error // Error
fail("At least one DispatchInformation item does not match the requested group.\n\trequested group=[" + nGroup fail("At least one DispatchInformation item does not match the requested group.\n\trequested group=[" + nGroup
+ "] returned groupd=[" + aInfo.GroupId + "] command=\"" + aInfo.Command + "\""); // true => dont break this test + "] returned groupd=[" + aInfo.GroupId + "] command=\"" + aInfo.Command + "\""); // true => don't break this test
continue; continue;
} }
if (aCheckMap.containsKey(aInfo.Command)) if (aCheckMap.containsKey(aInfo.Command))
{ {
// Error // Error
fail("Found a duplicate item: group=[" + aInfo.GroupId + "] command=\"" + aInfo.Command + "\""); // true => dont break this test fail("Found a duplicate item: group=[" + aInfo.GroupId + "] command=\"" + aInfo.Command + "\""); // true => don't break this test
continue; continue;
} }
......
...@@ -129,10 +129,10 @@ public class StatusIndicator implements com.sun.star.task.XStatusIndicator ...@@ -129,10 +129,10 @@ public class StatusIndicator implements com.sun.star.task.XStatusIndicator
/** /*
* Internal helper to show the status. * Internal helper to show the status.
* Currently it's implement as normal text out on stdout. * Currently it's implement as normal text out on stdout.
* But of course other thimngs are possible here too. * But of course other things are possible here too.
* e.g. a dialog * e.g. a dialog
*/ */
private void impl_show() private void impl_show()
......
...@@ -178,7 +178,7 @@ css::uno::Sequence< css::uno::Any > SAL_CALL XMLBasedAcceleratorConfiguration::g ...@@ -178,7 +178,7 @@ css::uno::Sequence< css::uno::Any > SAL_CALL XMLBasedAcceleratorConfiguration::g
sal_Int32 i = 0; sal_Int32 i = 0;
sal_Int32 c = lCommandList.getLength(); sal_Int32 c = lCommandList.getLength();
css::uno::Sequence< css::uno::Any > lPreferredOnes (c); // dont pack list! css::uno::Sequence< css::uno::Any > lPreferredOnes (c); // don't pack list!
AcceleratorCache& rCache = impl_getCFG(); AcceleratorCache& rCache = impl_getCFG();
for (i=0; i<c; ++i) for (i=0; i<c; ++i)
...@@ -746,7 +746,7 @@ css::uno::Sequence< css::uno::Any > SAL_CALL XCUBasedAcceleratorConfiguration::g ...@@ -746,7 +746,7 @@ css::uno::Sequence< css::uno::Any > SAL_CALL XCUBasedAcceleratorConfiguration::g
sal_Int32 i = 0; sal_Int32 i = 0;
sal_Int32 c = lCommandList.getLength(); sal_Int32 c = lCommandList.getLength();
css::uno::Sequence< css::uno::Any > lPreferredOnes (c); // dont pack list! css::uno::Sequence< css::uno::Any > lPreferredOnes (c); // don't pack list!
AcceleratorCache& rCache = impl_getCFG(true); AcceleratorCache& rCache = impl_getCFG(true);
for (i=0; i<c; ++i) for (i=0; i<c; ++i)
......
...@@ -176,7 +176,7 @@ void DocumentAcceleratorConfiguration::fillCache() ...@@ -176,7 +176,7 @@ void DocumentAcceleratorConfiguration::fillCache()
if (!xDocumentRoot.is()) if (!xDocumentRoot.is())
return; return;
// get current office locale ... but dont cache it. // get current office locale ... but don't cache it.
// Otherwise we must be listener on the configuration layer // Otherwise we must be listener on the configuration layer
// which seems to superflous for this small implementation .-) // which seems to superflous for this small implementation .-)
LanguageTag aLanguageTag( impl_ts_getLocale()); LanguageTag aLanguageTag( impl_ts_getLocale());
......
...@@ -99,7 +99,7 @@ void GlobalAcceleratorConfiguration::fillCache() ...@@ -99,7 +99,7 @@ void GlobalAcceleratorConfiguration::fillCache()
/** read all data into the cache. */ /** read all data into the cache. */
#if 0 #if 0
// get current office locale ... but dont cache it. // get current office locale ... but don't cache it.
// Otherwise we must be listener on the configuration layer // Otherwise we must be listener on the configuration layer
// which seems to superflous for this small implementation .-) // which seems to superflous for this small implementation .-)
// XXX: what is this good for? it was a comphelper::Locale but unused // XXX: what is this good for? it was a comphelper::Locale but unused
......
...@@ -138,7 +138,7 @@ void ModuleAcceleratorConfiguration::fillCache() ...@@ -138,7 +138,7 @@ void ModuleAcceleratorConfiguration::fillCache()
} }
#if 0 #if 0
// get current office locale ... but dont cache it. // get current office locale ... but don't cache it.
// Otherwise we must be listener on the configuration layer // Otherwise we must be listener on the configuration layer
// which seems to superflous for this small implementation .-) // which seems to superflous for this small implementation .-)
// XXX: what is this good for? it was a comphelper::Locale but unused // XXX: what is this good for? it was a comphelper::Locale but unused
......
...@@ -493,7 +493,7 @@ void PresetHandler::connectToResource( PresetHandler::EConfigType ...@@ -493,7 +493,7 @@ void PresetHandler::connectToResource( PresetHandler::EConfigType
void PresetHandler::copyPresetToTarget(const OUString& sPreset, void PresetHandler::copyPresetToTarget(const OUString& sPreset,
const OUString& sTarget) const OUString& sTarget)
{ {
// dont check our preset list, if element exists // don't check our preset list, if element exists
// We try to open it and forward all errors to the user! // We try to open it and forward all errors to the user!
css::uno::Reference< css::embed::XStorage > xWorkingShare; css::uno::Reference< css::embed::XStorage > xWorkingShare;
...@@ -638,7 +638,7 @@ void PresetHandler::addStorageListener(IStorageListener* pListener) ...@@ -638,7 +638,7 @@ void PresetHandler::addStorageListener(IStorageListener* pListener)
EConfigType eCfgType; EConfigType eCfgType;
{ {
SolarMutexGuard g; SolarMutexGuard g;
sRelPath = m_sRelPathUser; // use user path ... because we dont work directly on the share layer! sRelPath = m_sRelPathUser; // use user path ... because we don't work directly on the share layer!
eCfgType = m_eConfigType; eCfgType = m_eConfigType;
} }
...@@ -668,7 +668,7 @@ void PresetHandler::removeStorageListener(IStorageListener* pListener) ...@@ -668,7 +668,7 @@ void PresetHandler::removeStorageListener(IStorageListener* pListener)
EConfigType eCfgType; EConfigType eCfgType;
{ {
SolarMutexGuard g; SolarMutexGuard g;
sRelPath = m_sRelPathUser; // use user path ... because we dont work directly on the share layer! sRelPath = m_sRelPathUser; // use user path ... because we don't work directly on the share layer!
eCfgType = m_eConfigType; eCfgType = m_eConfigType;
} }
......
...@@ -157,7 +157,7 @@ css::uno::Reference< css::embed::XStorage > StorageHolder::openPath(const OUStri ...@@ -157,7 +157,7 @@ css::uno::Reference< css::embed::XStorage > StorageHolder::openPath(const OUStri
sRelPath += sChild + PATH_SEPARATOR; sRelPath += sChild + PATH_SEPARATOR;
} }
// TODO think about return last storage as working storage ... but dont caching it inside this holder! // TODO think about return last storage as working storage ... but don't caching it inside this holder!
// => otherwise the same storage is may be commit more than once. // => otherwise the same storage is may be commit more than once.
return xChild; return xChild;
...@@ -426,7 +426,7 @@ css::uno::Reference< css::embed::XStorage > StorageHolder::openSubStorageWithFal ...@@ -426,7 +426,7 @@ css::uno::Reference< css::embed::XStorage > StorageHolder::openSubStorageWithFal
throw exResult; throw exResult;
// c) try it readonly // c) try it readonly
// dont catch exception here! Outside code wish to know, if operation failed or not. // don't catch exception here! Outside code wish to know, if operation failed or not.
// Otherwhise they work on NULL references ... // Otherwhise they work on NULL references ...
sal_Int32 eNewMode = (eOpenMode & ~css::embed::ElementModes::WRITE); sal_Int32 eNewMode = (eOpenMode & ~css::embed::ElementModes::WRITE);
css::uno::Reference< css::embed::XStorage > xSubStorage = xBaseStorage->openStorageElement(sSubStorage, eNewMode); css::uno::Reference< css::embed::XStorage > xSubStorage = xBaseStorage->openStorageElement(sSubStorage, eNewMode);
...@@ -465,7 +465,7 @@ css::uno::Reference< css::io::XStream > StorageHolder::openSubStreamWithFallback ...@@ -465,7 +465,7 @@ css::uno::Reference< css::io::XStream > StorageHolder::openSubStreamWithFallback
throw exResult; throw exResult;
// c) try it readonly // c) try it readonly
// dont catch exception here! Outside code wish to know, if operation failed or not. // don't catch exception here! Outside code wish to know, if operation failed or not.
// Otherwhise they work on NULL references ... // Otherwhise they work on NULL references ...
sal_Int32 eNewMode = (eOpenMode & ~css::embed::ElementModes::WRITE); sal_Int32 eNewMode = (eOpenMode & ~css::embed::ElementModes::WRITE);
css::uno::Reference< css::io::XStream > xSubStream = xBaseStorage->openStreamElement(sSubStream, eNewMode); css::uno::Reference< css::io::XStream > xSubStream = xBaseStorage->openStreamElement(sSubStream, eNewMode);
......
...@@ -41,7 +41,7 @@ FrameContainer::FrameContainer() ...@@ -41,7 +41,7 @@ FrameContainer::FrameContainer()
/**-*************************************************************************************************************** /**-***************************************************************************************************************
@short deinitialize may a filled container @short deinitialize may a filled container
@descr Special features (if the currently are running) will be dsiabled and we free all used other resources. @descr Special features (if the currently are running) will be disabled and we free all used other resources.
@threadsafe not necessary - its not a singleton @threadsafe not necessary - its not a singleton
*****************************************************************************************************************/ *****************************************************************************************************************/
......
...@@ -281,7 +281,7 @@ IMPL_LINK_NOARG_TYPED(CloseDispatcher, impl_asyncCallback, LinkParamNone*, void) ...@@ -281,7 +281,7 @@ IMPL_LINK_NOARG_TYPED(CloseDispatcher, impl_asyncCallback, LinkParamNone*, void)
// Analyze the environment a first time. // Analyze the environment a first time.
// If we found some special cases, we can // If we found some special cases, we can
// make some decisions erlier! // make some decisions earlier!
css::uno::Reference< css::frame::XFramesSupplier > xDesktop( css::frame::Desktop::create(xContext), css::uno::UNO_QUERY_THROW); css::uno::Reference< css::frame::XFramesSupplier > xDesktop( css::frame::Desktop::create(xContext), css::uno::UNO_QUERY_THROW);
FrameListAnalyzer aCheck1(xDesktop, xCloseFrame, FrameListAnalyzer::E_HELP | FrameListAnalyzer::E_BACKINGCOMPONENT); FrameListAnalyzer aCheck1(xDesktop, xCloseFrame, FrameListAnalyzer::E_HELP | FrameListAnalyzer::E_BACKINGCOMPONENT);
...@@ -430,7 +430,7 @@ bool CloseDispatcher::implts_prepareFrameForClosing(const css::uno::Reference< c ...@@ -430,7 +430,7 @@ bool CloseDispatcher::implts_prepareFrameForClosing(const css::uno::Reference< c
return true; return true;
// Close all views to the same document ... if forced to do so. // Close all views to the same document ... if forced to do so.
// But dont touch our own frame here! // But don't touch our own frame here!
// We must do so ... because the may be following controller->suspend() // We must do so ... because the may be following controller->suspend()
// will show the "save/discard/cancel" dialog for the last view only! // will show the "save/discard/cancel" dialog for the last view only!
if (bCloseAllOtherViewsToo) if (bCloseAllOtherViewsToo)
...@@ -458,7 +458,7 @@ bool CloseDispatcher::implts_prepareFrameForClosing(const css::uno::Reference< c ...@@ -458,7 +458,7 @@ bool CloseDispatcher::implts_prepareFrameForClosing(const css::uno::Reference< c
if (bAllowSuspend) if (bAllowSuspend)
{ {
css::uno::Reference< css::frame::XController > xController = xFrame->getController(); css::uno::Reference< css::frame::XController > xController = xFrame->getController();
if (xController.is()) // some views dont uses a controller .-( (e.g. the help window) if (xController.is()) // some views don't uses a controller .-( (e.g. the help window)
{ {
bControllerSuspended = xController->suspend(sal_True); bControllerSuspended = xController->suspend(sal_True);
if (! bControllerSuspended) if (! bControllerSuspended)
...@@ -466,7 +466,7 @@ bool CloseDispatcher::implts_prepareFrameForClosing(const css::uno::Reference< c ...@@ -466,7 +466,7 @@ bool CloseDispatcher::implts_prepareFrameForClosing(const css::uno::Reference< c
} }
} }
// dont remove the component really by e.g. calling setComponent(null, null). // don't remove the component really by e.g. calling setComponent(null, null).
// It's enough to suspend the controller. // It's enough to suspend the controller.
// If we close the frame later this controller doesn't show the same dialog again. // If we close the frame later this controller doesn't show the same dialog again.
return true; return true;
...@@ -484,7 +484,7 @@ bool CloseDispatcher::implts_closeFrame() ...@@ -484,7 +484,7 @@ bool CloseDispatcher::implts_closeFrame()
if ( ! xFrame.is() ) if ( ! xFrame.is() )
return true; return true;
// dont deliver ownership; our "UI user" will try it again if it failed. // don't deliver ownership; our "UI user" will try it again if it failed.
// OK - he will get an empty frame then. But normally an empty frame // OK - he will get an empty frame then. But normally an empty frame
// should be closeable always :-) // should be closeable always :-)
if (!fpf::closeIt(xFrame, false)) if (!fpf::closeIt(xFrame, false))
......
...@@ -219,7 +219,7 @@ css::uno::Reference< css::frame::XDispatch > DispatchProvider::implts_queryDeskt ...@@ -219,7 +219,7 @@ css::uno::Reference< css::frame::XDispatch > DispatchProvider::implts_queryDeskt
// I.IV) no further special targets exist // I.IV) no further special targets exist
// Now we have to search for the right target frame by calling findFrame() - but should provide our code // Now we have to search for the right target frame by calling findFrame() - but should provide our code
// against creation of a new task if no frame could be found. // against creation of a new task if no frame could be found.
// I said it b efore - it's allowed for dispatch() only. // I said it before - it's allowed for dispatch() only.
else else
{ {
...@@ -234,7 +234,7 @@ css::uno::Reference< css::frame::XDispatch > DispatchProvider::implts_queryDeskt ...@@ -234,7 +234,7 @@ css::uno::Reference< css::frame::XDispatch > DispatchProvider::implts_queryDeskt
xDispatcher = xProvider->queryDispatch(aURL,SPECIALTARGET_SELF,0); xDispatcher = xProvider->queryDispatch(aURL,SPECIALTARGET_SELF,0);
} }
// if it couldn't be found - but creation was allowed // if it couldn't be found - but creation was allowed
// use special dispatcher for creatio or froward it to the browser // use special dispatcher for creation or forwarding to the browser
else if (nSearchFlags & css::frame::FrameSearchFlag::CREATE) else if (nSearchFlags & css::frame::FrameSearchFlag::CREATE)
xDispatcher = implts_getOrCreateDispatchHelper( E_CREATEDISPATCHER, xDesktop, sTargetFrameName, nSearchFlags ); xDispatcher = implts_getOrCreateDispatchHelper( E_CREATEDISPATCHER, xDesktop, sTargetFrameName, nSearchFlags );
} }
......
...@@ -97,7 +97,7 @@ void SAL_CALL SystemExec::dispatchWithNotification( const css::util::URL& ...@@ -97,7 +97,7 @@ void SAL_CALL SystemExec::dispatchWithNotification( const css::util::URL&
{ {
// convert "systemexec:file:///c:/temp/test.html" => "file:///c:/temp/test.html" // convert "systemexec:file:///c:/temp/test.html" => "file:///c:/temp/test.html"
sal_Int32 c = aURL.Complete.getLength()-PROTOCOL_LENGTH; sal_Int32 c = aURL.Complete.getLength()-PROTOCOL_LENGTH;
if (c<1) // we dont check for valid URLs here! The system will show an error message ... if (c<1) // we don't check for valid URLs here! The system will show an error message ...
{ {
impl_notifyResultListener(xListener, css::frame::DispatchResultState::FAILURE); impl_notifyResultListener(xListener, css::frame::DispatchResultState::FAILURE);
return; return;
......
...@@ -59,10 +59,10 @@ FrameListAnalyzer::~FrameListAnalyzer() ...@@ -59,10 +59,10 @@ FrameListAnalyzer::~FrameListAnalyzer()
Of course we can't access frames, which stands outside of this tree. Of course we can't access frames, which stands outside of this tree.
But it's necessary to collect top frames here only. Otherwise we interpret closing of last But it's necessary to collect top frames here only. Otherwise we interpret closing of last
frame wrong. Further we analyze this list and split into different parts. frame wrong. Further we analyze this list and split into different parts.
E.g. for "CloseDoc" we must know, which frames of the given list referr to the same model. E.g. for "CloseDoc" we must know, which frames of the given list refer to the same model.
These frames must be closed then. But all other frames must be untouched. These frames must be closed then. But all other frames must be untouched.
In case the request was "CloseWin" these splitted lists can be used too, to decide if the last window In case the request was "CloseWin" these splitted lists can be used too, to decide if the last window
or document was closed. Then we have to initialize the backing window ... or document was closed. Then we have to initialize the backing window...
Last but not least we must know something about our special help frame. It must be handled Last but not least we must know something about our special help frame. It must be handled
separately. And last but not least - the backing component frame must be detected too. separately. And last but not least - the backing component frame must be detected too.
*/ */
......
...@@ -33,9 +33,9 @@ namespace framework{ ...@@ -33,9 +33,9 @@ namespace framework{
@attention This implementation could be used one times only. We don't support a resetable continuation yet! @attention This implementation could be used one times only. We don't support a resetable continuation yet!
Why? Normally interaction should show a filter selection dialog and ask user for his decision. Why? Normally interaction should show a filter selection dialog and ask user for his decision.
He can select any filter - then instances of these class will be called by handler ... or user He can select any filter - then instances of these class will be called by handler... or user
close dialog without any selection. Then another continuation should be slected by handler to close dialog without any selection. Then another continuation should be selected by handler to
abort continuations ... Retrying isn't very useful here ... I think. abort continuations... Retrying isn't very useful here... I think.
@implements XInteractionFilterSelect @implements XInteractionFilterSelect
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment