diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2019-03-02 21:19:24 +0100 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2019-03-03 11:02:56 +0100 |
| commit | e1ec5eb6dc76d3fcb4e29dc214daf8ee7ca9e547 (patch) | |
| tree | d5dfca6e688235623ca918a6f78611840d5f13fe /sdext | |
| parent | 9b6a377e770cc6455feac0b83da8d9639926b993 (diff) | |
Extend cast to whole expression
...which has no effect on the result, but silences benign Clang
-fsanitize=implicit-signed-integer-truncation warnings
Change-Id: Ic2cf38466f12462d67ceb6268d5197e13cc6143a
Reviewed-on: https://gerrit.libreoffice.org/68627
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'sdext')
| -rw-r--r-- | sdext/source/pdfimport/pdfparse/pdfentries.cxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sdext/source/pdfimport/pdfparse/pdfentries.cxx b/sdext/source/pdfimport/pdfparse/pdfentries.cxx index 919e0964b8d4..92cb45ab5a12 100644 --- a/sdext/source/pdfimport/pdfparse/pdfentries.cxx +++ b/sdext/source/pdfimport/pdfparse/pdfentries.cxx @@ -312,11 +312,11 @@ OString PDFString::getFilteredString() const { sal_Char rResult = 0; if( *pRun >= '0' && *pRun <= '9' ) - rResult = sal_Char( *pRun-'0' ) << 4; + rResult = sal_Char( ( *pRun-'0' ) << 4 ); else if( *pRun >= 'a' && *pRun <= 'f' ) - rResult = sal_Char( *pRun-'a' + 10 ) << 4; + rResult = sal_Char( ( *pRun-'a' + 10 ) << 4 ); else if( *pRun >= 'A' && *pRun <= 'F' ) - rResult = sal_Char( *pRun-'A' + 10 ) << 4; + rResult = sal_Char( ( *pRun-'A' + 10 ) << 4 ); pRun++; if( *pRun != '>' && pRun - pStr < nLen ) { |