gwr commented on this pull request.
> @@ -34,7 +34,6 @@ */ #define tolower(C) (((C) >= 'A' && (C) <= 'Z') ? (C) - 'A' + 'a' : (C)) #define toupper(C) (((C) >= 'a' && (C) <= 'z') ? (C) - 'a' + 'A': (C)) -#define iscntrl(C) ((((C) >= 0) && ((C) <= 0x1f)) || ((C) == 0x7f)) @yuripv Are you satisfied with this review? Igor, I generally lean toward "the less change the better", so I'd probably go as Yuri suggests and just remove the test for zero. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/418#discussion_r178436906 ------------------------------------------ openzfs: openzfs-developer Permalink: https://openzfs.topicbox.com/groups/developer/discussions/Tf974af07ec408e5e-M64f42bf585919655dfe4ddf9 Delivery options: https://openzfs.topicbox.com/groups
