GGLinnk/bees
1
0
Fork 0
mirror of https://github.com/Zygo/bees.git synced 2025-05-18 05:45:45 +02:00
Code Packages

error: drop redundant CHECK_CONSTRAINT

Browse Source 
CHECK_CONSTRAINT is just THROW_CHECK1 with an inconsistent name.
Remove it.

Signed-off-by: Zygo Blaxell <bees@furryterror.org>
This commit is contained in:
Zygo Blaxell 2017-12-21 14:00:17 -05:00
parent 77614a0e99
commit 07751885d2
2 changed files with 1 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
include/crucible/error.h
View File

@ -100,12 +100,6 @@ namespace crucible {
} while (0) } while (0)
// macros for checking a constraint // macros for checking a constraint
#define CHECK_CONSTRAINT(value, expr) do { \
if (!(expr)) { \
THROW_ERROR(out_of_range, #value << " = " << value << " failed constraint check (" << #expr << ")"); \
} \
} while(0)
#define THROW_CHECK0(type, expr) do { \ #define THROW_CHECK0(type, expr) do { \
if (!(expr)) { \ if (!(expr)) { \
THROW_ERROR(type, "failed constraint check (" << #expr << ")"); \ THROW_ERROR(type, "failed constraint check (" << #expr << ")"); \

2
lib/fs.cc
View File

@ -625,7 +625,7 @@ namespace crucible {
void void
Fiemap::do_ioctl(int fd) Fiemap::do_ioctl(int fd)
{ {
CHECK_CONSTRAINT(m_min_count, m_min_count <= m_max_count); THROW_CHECK1(out_of_range, m_min_count, m_min_count <= m_max_count);
auto extent_count = m_min_count; auto extent_count = m_min_count;
vector<char> ioctl_arg = vector_copy_struct<fiemap>(this); vector<char> ioctl_arg = vector_copy_struct<fiemap>(this);