copy: rearrange flags field definition
authorLennart Poettering <lennart@poettering.net>
Fri, 23 Mar 2018 17:27:31 +0000 (18:27 +0100)
committerLennart Poettering <lennart@poettering.net>
Fri, 13 Apr 2018 09:32:46 +0000 (11:32 +0200)
Let's use the usual 1U << 0,1,2,3,4 spelling for definiting flags enums.

src/basic/copy.h

index 9ea1049..0c50bce 100644 (file)
@@ -13,9 +13,9 @@
 #include <sys/types.h>
 
 typedef enum CopyFlags {
-        COPY_REFLINK    = 0x1,      /* try to reflink */
-        COPY_MERGE      = 0x2,      /* merge existing trees with our new one to copy */
-        COPY_REPLACE    = 0x4,      /* replace an existing file if there's one */
+        COPY_REFLINK = 1U << 0, /* Try to reflink */
+        COPY_MERGE   = 1U << 1, /* Merge existing trees with our new one to copy */
+        COPY_REPLACE = 1U << 2, /* Replace an existing file if there's one */
 } CopyFlags;
 
 int copy_file_fd(const char *from, int to, CopyFlags copy_flags);