diff options
author | Mike Kaganski <mike.kaganski@collabora.com> | 2023-04-02 01:18:42 +0300 |
---|---|---|
committer | Mike Kaganski <mike.kaganski@collabora.com> | 2023-04-02 18:46:47 +0200 |
commit | 116b9d6ddf2b61186b29f0370234eec9c1bbe306 (patch) | |
tree | 7f90a12333274086e33c4e0fabfd96a8e54e6b6c /include/vcl/jsdialog | |
parent | d7ba78e9c7be835a1e2ecdacd25995663e96862f (diff) |
Avoid conversions between OUString and OString in VCL
Standardize on OUString, which is the main internal string class.
Convert from/to OUString only when communicating with respective
external APIs.
Removes about 200 conversions from the code.
Change-Id: I96ecee7c6fd271bb76639220e96d69d2964bed26
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149930
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'include/vcl/jsdialog')
-rw-r--r-- | include/vcl/jsdialog/executor.hxx | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/include/vcl/jsdialog/executor.hxx b/include/vcl/jsdialog/executor.hxx index 0c47066da9fa..c781f1ea1150 100644 --- a/include/vcl/jsdialog/executor.hxx +++ b/include/vcl/jsdialog/executor.hxx @@ -49,7 +49,7 @@ public: rIconView.signal_item_activated(); } - static void trigger_clicked(weld::Toolbar& rToolbar, const OString& rIdent) + static void trigger_clicked(weld::Toolbar& rToolbar, const OUString& rIdent) { rToolbar.signal_clicked(rIdent); } @@ -88,12 +88,12 @@ public: rArea.m_aCommandHdl.Call(rCmd); } - static void enter_page(weld::Notebook& rNotebook, const OString& rPage) + static void enter_page(weld::Notebook& rNotebook, const OUString& rPage) { rNotebook.m_aEnterPageHdl.Call(rPage); } - static void leave_page(weld::Notebook& rNotebook, const OString& rPage) + static void leave_page(weld::Notebook& rNotebook, const OUString& rPage) { rNotebook.m_aLeavePageHdl.Call(rPage); } @@ -102,15 +102,15 @@ public: namespace jsdialog { // type used to store key-value pairs to put in the generated messages -typedef std::unordered_map<std::string, OUString> ActionDataMap; +typedef std::unordered_map<OString, OUString> ActionDataMap; /// execute action on a widget -VCL_DLLPUBLIC bool ExecuteAction(const std::string& nWindowId, const OString& rWidget, +VCL_DLLPUBLIC bool ExecuteAction(const OUString& nWindowId, const OUString& rWidget, StringMap& rData); /// send full update message to the client -VCL_DLLPUBLIC void SendFullUpdate(const std::string& nWindowId, const OString& rWidget); +VCL_DLLPUBLIC void SendFullUpdate(const OUString& nWindowId, const OUString& rWidget); /// send action message to the client -VCL_DLLPUBLIC void SendAction(const std::string& nWindowId, const OString& rWidget, +VCL_DLLPUBLIC void SendAction(const OUString& nWindowId, const OUString& rWidget, std::unique_ptr<ActionDataMap> pData); VCL_DLLPUBLIC StringMap jsonToStringMap(const char* pJSON); }; |