diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-09-21 15:42:49 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-09-22 08:30:10 +0200 |
| commit | b63609ba5478ed9b020c113f5704f7ea8447dec8 (patch) | |
| tree | e40896646675d8b462cec7a90ca1e94e902b3746 /oox | |
| parent | 4af6c0948be47d7816eb1b6f2137b70aba639f0d (diff) | |
loplugin:flatten in framework..package
Change-Id: Ide8a97eae6e2fdc7d2dcccba1480ac55a9b555bc
Reviewed-on: https://gerrit.libreoffice.org/42593
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'oox')
| -rw-r--r-- | oox/source/ppt/comments.cxx | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/oox/source/ppt/comments.cxx b/oox/source/ppt/comments.cxx index 965a7ef658b0..9f9a8fe7977c 100644 --- a/oox/source/ppt/comments.cxx +++ b/oox/source/ppt/comments.cxx @@ -76,10 +76,11 @@ OUString Comment::getAuthor ( const CommentAuthorList& list ) const Comment& CommentList::getCommentAtIndex (int index) { - if(index < (int)cmLst.size() && index >= 0) - return cmLst.at(index); - else + if(index >= (int)cmLst.size() || index < 0) throw css::lang::IllegalArgumentException(); + + return cmLst.at(index) +; } } } |