diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-10-28 00:13:29 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2018-10-28 07:55:59 +0100 |
| commit | 5f0ddb2d2068a4388a7df12cd0e11f628b91e4c0 (patch) | |
| tree | 2007c4a1f6c21a9f0d5a11aab60a10cd178d1d8b /unoxml | |
| parent | 8522572cdd27f62a031bb07c9cf7f6f85e8453ad (diff) | |
tdf#120703 PVS: V547 Expression is always true/false
Change-Id: Ic92cc594979cac2edac04a085957398672a5dfcc
Reviewed-on: https://gerrit.libreoffice.org/62450
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'unoxml')
| -rw-r--r-- | unoxml/source/dom/attr.cxx | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/unoxml/source/dom/attr.cxx b/unoxml/source/dom/attr.cxx index 22f214dbf384..4ff411d3f62c 100644 --- a/unoxml/source/dom/attr.cxx +++ b/unoxml/source/dom/attr.cxx @@ -156,11 +156,8 @@ namespace DOM if (nullptr == m_aAttrPtr->children) { return OUString(); } - char const*const pContent((m_aAttrPtr->children) - ? reinterpret_cast<char const*>(m_aAttrPtr->children->content) - : ""); - OUString const ret(pContent, strlen(pContent), RTL_TEXTENCODING_UTF8); - return ret; + char const*const pContent(reinterpret_cast<char const*>(m_aAttrPtr->children->content)); + return OUString(pContent, strlen(pContent), RTL_TEXTENCODING_UTF8); } /** |