Make dynamic_annotations build with -Wconstant-conversion from newer clangs.

Make dynamic_annotations build with -Wconstant-conversion from newer clangs.

After http://llvm.org/viewvc/llvm-project?rev=259271&view=rev, clang points
out that lineno is sometimes assigned a value > SHRT_MAX, which causes
an overflow. All values are smaller than USHRT_MAX, so just make lineno
an unsigned short.

No behavior change.

BUG=none

Committed: https://crrev.com/6ef06b461ccae125f8c8d6e33dde10c550ff3780
Cr-Commit-Position: refs/heads/master@{#372952}

[Nico, 2016-02-02 14:53:14]

thakis@chromium.org changed reviewers:
+ glider@chromium.org

[Alexander Potapenko, 2016-02-02 15:02:31]

On 2016/02/02 14:53:14, Nico wrote:

Well, at least for third_party/sqlite/amalgamation/sqlite3.c the number of lines
does not fit into unsigned short.

[Nico, 2016-02-02 15:03:04]

This is a .c file, not a .h file (?)

[Alexander Potapenko, 2016-02-02 15:03:37]

On 2016/02/02 15:02:31, Alexander Potapenko wrote:
> On 2016/02/02 14:53:14, Nico wrote:
> 
> Well, at least for third_party/sqlite/amalgamation/sqlite3.c the number of
lines
> does not fit into unsigned short.

(however the previous version of code didn't address this problem either)

[Alexander Potapenko, 2016-02-02 15:04:31]

Oh, sorry, you are right.
LGTM

[Alexander Potapenko, 2016-02-02 15:04:35]

Oh, sorry, you are right.
LGTM

[Nico, 2016-02-02 15:05:48]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2016-02-02 15:07:25]

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

[commit-bot: I haz the power, 2016-02-02 16:00:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-02 16:01:30]

Description was changed from

==========
Make dynamic_annotations build with -Wconstant-conversion from newer clangs.

After http://llvm.org/viewvc/llvm-project?rev=259271&view=rev, clang points
out that lineno is sometimes assigned a value > SHRT_MAX, which causes
an overflow. All values are smaller than USHRT_MAX, so just make lineno
an unsigned short.

No behavior change.

BUG=none
==========

to

==========
Make dynamic_annotations build with -Wconstant-conversion from newer clangs.

After http://llvm.org/viewvc/llvm-project?rev=259271&view=rev, clang points
out that lineno is sometimes assigned a value > SHRT_MAX, which causes
an overflow. All values are smaller than USHRT_MAX, so just make lineno
an unsigned short.

No behavior change.

BUG=none

Committed: https://crrev.com/6ef06b461ccae125f8c8d6e33dde10c550ff3780
Cr-Commit-Position: refs/heads/master@{#372952}
==========

[commit-bot: I haz the power, 2016-02-02 16:01:31]

Patchset 1 (id:??) landed as
https://crrev.com/6ef06b461ccae125f8c8d6e33dde10c550ff3780
Cr-Commit-Position: refs/heads/master@{#372952}