diff options
| author | Muhammet Kara <muhammet.kara@pardus.org.tr> | 2016-12-26 13:43:15 +0300 |
|---|---|---|
| committer | Julien Nabet <serval2412@yahoo.fr> | 2017-01-02 10:39:04 +0000 |
| commit | 8885343a33365d2570513073c3666428aa79dbdf (patch) | |
| tree | 814cbc1b7a2be7df4abfec2b3b432f225985497a /libreofficekit | |
| parent | 136f5a209e1ca19f7d7b8eb69bb76e7d1b2c07e6 (diff) | |
Fix inefficient usage of string::find() in condition (CWE597)
string::compare() will be faster when string::find's result is
compared with 0, because it will not scan the whole string.
Change-Id: I78596a6d796fe9779f88b7c7b91da09aa27b7035
Reviewed-on: https://gerrit.libreoffice.org/32430
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Diffstat (limited to 'libreofficekit')
| -rw-r--r-- | libreofficekit/source/gtk/lokdocview.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libreofficekit/source/gtk/lokdocview.cxx b/libreofficekit/source/gtk/lokdocview.cxx index a08d93e8e3fa..cb88bb1b48f7 100644 --- a/libreofficekit/source/gtk/lokdocview.cxx +++ b/libreofficekit/source/gtk/lokdocview.cxx @@ -1136,7 +1136,7 @@ callback (gpointer pData) { case LOK_CALLBACK_INVALIDATE_TILES: { - if (pCallback->m_aPayload.find("EMPTY") != 0) // payload doesn't start with "EMPTY" + if (pCallback->m_aPayload.compare(0, 5, "EMPTY") != 0) // payload doesn't start with "EMPTY" { GdkRectangle aRectangle = payloadToRectangle(pDocView, pCallback->m_aPayload.c_str()); setTilesInvalid(pDocView, aRectangle); |