PR #13473 Cjian/c4244 round 2

Cjian/c4244 round 2 #13473

jchen351 merged 8 commits into main from cjian/C4244_round_2

First round of fixes.

f5e5f309

Using static_cast<size_t>

39518e39

Merge branch 'main' into cjian/C4244_round_2

44219078

Fixes round 2

8da8507f

Merge branch 'main' into cjian/C4244_round_2

64840e85

snnn commented on 2022-10-27

snnn dismissed these changes on 2022-10-27

snnn commented on 2022-10-27

Updating with SafeInt and gsl::narrow

61dbce30

jchen351 dismissed their stale review via 61dbce30 3 years ago

Updating with SafeInt and gsl::narrow

55de90a1

Updating with SafeInt and gsl::narrow

d9cff9c0

snnn commented on 2022-10-27

snnn approved these changes on 2022-10-27

jchen351 merged f9378c5c into main 3 years ago

jchen351 deleted the cjian/C4244_round_2 branch 3 years ago

Reviewers

snnn

Assignees

No one assigned

Labels

None yet

Milestone

No milestone