diff options
author | Caolán McNamara <caolanm@redhat.com> | 2021-06-30 21:08:46 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2021-07-01 09:47:09 +0200 |
commit | 123a3ccacdb30dce0733212f35dea738bcafb747 (patch) | |
tree | 2ce8127ebc7587b345cc77d478c4c724f7f1f21f /hwpfilter | |
parent | 4a0b40f1be9f6773c8ebc5331c257911a76a5cee (diff) |
getParaShape() adds bounds checking we don't need in this case
Change-Id: I02f917169fcae0dc5bdfc55653cd21fe67eac9c1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118200
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'hwpfilter')
-rw-r--r-- | hwpfilter/source/hwpfile.cxx | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/hwpfilter/source/hwpfile.cxx b/hwpfilter/source/hwpfile.cxx index 4c79a6c1211f..6e04d8f67429 100644 --- a/hwpfilter/source/hwpfile.cxx +++ b/hwpfilter/source/hwpfile.cxx @@ -624,7 +624,6 @@ int HWPFile::compareCharShape(CharShape const *shape) return 0; } - int HWPFile::compareParaShape(ParaShape const *shape) { int count = pslist.size(); @@ -632,7 +631,7 @@ int HWPFile::compareParaShape(ParaShape const *shape) { for(int i = 0; i< count; i++) { - ParaShape *pshape = getParaShape(i); + ParaShape *pshape = pslist[i].get(); if( shape->left_margin == pshape->left_margin && shape->right_margin == pshape->right_margin && shape->pspacing_prev == pshape->pspacing_prev && @@ -661,7 +660,6 @@ int HWPFile::compareParaShape(ParaShape const *shape) return 0; } - HWPFile *GetCurrentDoc() { return HWPFile::cur_doc; |