Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(562)

Side by Side Diff: webrtc/base/safe_conversions.h

Issue 1335923002: Add RTC_ prefix to (D)CHECKs and related macros. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Rebase. Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « webrtc/base/rtccertificate.cc ('k') | webrtc/base/stringencode.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2014 The WebRTC Project Authors. All rights reserved. 2 * Copyright 2014 The WebRTC Project Authors. All rights reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 14 matching lines...) Expand all
25 template <typename Dst, typename Src> 25 template <typename Dst, typename Src>
26 inline bool IsValueInRangeForNumericType(Src value) { 26 inline bool IsValueInRangeForNumericType(Src value) {
27 return internal::RangeCheck<Dst>(value) == internal::TYPE_VALID; 27 return internal::RangeCheck<Dst>(value) == internal::TYPE_VALID;
28 } 28 }
29 29
30 // checked_cast<> is analogous to static_cast<> for numeric types, 30 // checked_cast<> is analogous to static_cast<> for numeric types,
31 // except that it CHECKs that the specified numeric conversion will not 31 // except that it CHECKs that the specified numeric conversion will not
32 // overflow or underflow. NaN source will always trigger a CHECK. 32 // overflow or underflow. NaN source will always trigger a CHECK.
33 template <typename Dst, typename Src> 33 template <typename Dst, typename Src>
34 inline Dst checked_cast(Src value) { 34 inline Dst checked_cast(Src value) {
35 CHECK(IsValueInRangeForNumericType<Dst>(value)); 35 RTC_CHECK(IsValueInRangeForNumericType<Dst>(value));
36 return static_cast<Dst>(value); 36 return static_cast<Dst>(value);
37 } 37 }
38 38
39 // saturated_cast<> is analogous to static_cast<> for numeric types, except 39 // saturated_cast<> is analogous to static_cast<> for numeric types, except
40 // that the specified numeric conversion will saturate rather than overflow or 40 // that the specified numeric conversion will saturate rather than overflow or
41 // underflow. NaN assignment to an integral will trigger a CHECK condition. 41 // underflow. NaN assignment to an integral will trigger a RTC_CHECK condition.
42 template <typename Dst, typename Src> 42 template <typename Dst, typename Src>
43 inline Dst saturated_cast(Src value) { 43 inline Dst saturated_cast(Src value) {
44 // Optimization for floating point values, which already saturate. 44 // Optimization for floating point values, which already saturate.
45 if (std::numeric_limits<Dst>::is_iec559) 45 if (std::numeric_limits<Dst>::is_iec559)
46 return static_cast<Dst>(value); 46 return static_cast<Dst>(value);
47 47
48 switch (internal::RangeCheck<Dst>(value)) { 48 switch (internal::RangeCheck<Dst>(value)) {
49 case internal::TYPE_VALID: 49 case internal::TYPE_VALID:
50 return static_cast<Dst>(value); 50 return static_cast<Dst>(value);
51 51
52 case internal::TYPE_UNDERFLOW: 52 case internal::TYPE_UNDERFLOW:
53 return std::numeric_limits<Dst>::min(); 53 return std::numeric_limits<Dst>::min();
54 54
55 case internal::TYPE_OVERFLOW: 55 case internal::TYPE_OVERFLOW:
56 return std::numeric_limits<Dst>::max(); 56 return std::numeric_limits<Dst>::max();
57 57
58 // Should fail only on attempting to assign NaN to a saturated integer. 58 // Should fail only on attempting to assign NaN to a saturated integer.
59 case internal::TYPE_INVALID: 59 case internal::TYPE_INVALID:
60 FATAL(); 60 FATAL();
61 return std::numeric_limits<Dst>::max(); 61 return std::numeric_limits<Dst>::max();
62 } 62 }
63 63
64 FATAL(); 64 FATAL();
65 return static_cast<Dst>(value); 65 return static_cast<Dst>(value);
66 } 66 }
67 67
68 } // namespace rtc 68 } // namespace rtc
69 69
70 #endif // WEBRTC_BASE_SAFE_CONVERSIONS_H_ 70 #endif // WEBRTC_BASE_SAFE_CONVERSIONS_H_
OLDNEW
« no previous file with comments | « webrtc/base/rtccertificate.cc ('k') | webrtc/base/stringencode.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698