Remove one warning from PDFium compilation in Cros

Remove one warning from PDFium compilation in Cros

The original code caused an optimization warning when
gcc tries to get rid of 'count > m_Length - index'
in CFX_ByteStringC::Mid() while assuming there is no signed
integer overflow.

This could also be avoided once FX_STRSIZE is no longer a
signed integer type.

Committed: https://pdfium.googlesource.com/pdfium/+/6f5325e538f31a37bb97a6a71f5c8cd64c6f7639

[Wei Li, 2016-04-21 21:38:14]

weili@chromium.org changed reviewers:
+ thestig@chromium.org

[Wei Li, 2016-04-21 21:38:14]

PTAL, thanks

[Wei Li, 2016-04-21 21:38:37]

Description was changed from

==========
Remove one warning from PDFium compilation in Cros

The original code caused an optimization warning when
gcc tries to get rid of 'count > m_Length - index'
in CFX_ByteStringC::Mid() while assuming there is no signed
integer overflow.

This could also be avoided once FX_STRSIZE is no longer an
signed integer type.
==========

to

==========
Remove one warning from PDFium compilation in Cros

The original code caused an optimization warning when
gcc tries to get rid of 'count > m_Length - index'
in CFX_ByteStringC::Mid() while assuming there is no signed
integer overflow.

This could also be avoided once FX_STRSIZE is no longer a
signed integer type.
==========

[Lei Zhang, 2016-04-21 22:02:10]

lgtm

[Wei Li, 2016-04-21 22:19:27]

The CQ bit was checked by weili@chromium.org

[commit-bot: I haz the power, 2016-04-21 22:19:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1906213002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1906213002/1

[commit-bot: I haz the power, 2016-04-21 22:31:32]

Description was changed from

==========
Remove one warning from PDFium compilation in Cros

The original code caused an optimization warning when
gcc tries to get rid of 'count > m_Length - index'
in CFX_ByteStringC::Mid() while assuming there is no signed
integer overflow.

This could also be avoided once FX_STRSIZE is no longer a
signed integer type.
==========

to

==========
Remove one warning from PDFium compilation in Cros

The original code caused an optimization warning when
gcc tries to get rid of 'count > m_Length - index'
in CFX_ByteStringC::Mid() while assuming there is no signed
integer overflow.

This could also be avoided once FX_STRSIZE is no longer a
signed integer type.

Committed:
https://pdfium.googlesource.com/pdfium/+/6f5325e538f31a37bb97a6a71f5c8cd64c6f...
==========

[commit-bot: I haz the power, 2016-04-21 22:31:33]

Committed patchset #1 (id:1) as
https://pdfium.googlesource.com/pdfium/+/6f5325e538f31a37bb97a6a71f5c8cd64c6f...