chiark / gitweb /
Prep v228: Substitute declaration masks (1/4)
authorSven Eden <yamakuzure@gmx.net>
Tue, 11 Apr 2017 15:05:07 +0000 (17:05 +0200)
committerSven Eden <yamakuzure@gmx.net>
Wed, 26 Apr 2017 10:59:13 +0000 (12:59 +0200)
Although it looks very ugly, substitute all declaration masks of the
form:

  // UNNEEDED type foo(...);

with:

  /// UNNEEDED by elogind
  #if 0
  type foo(...);
  #endif // 0

to make future merging of upstream pathes easier, as the relevant
lines themselves are not changed any more.

14 files changed:
src/basic/capability-util.h
src/basic/cgroup-util.h
src/basic/copy.c
src/basic/copy.h
src/basic/errno-list.h
src/basic/extract-word.h
src/basic/fd-util.h
src/basic/fileio-label.h
src/basic/fileio.h
src/basic/fs-util.h
src/basic/hashmap.h
src/basic/hostname-util.h
src/basic/label.h [new file with mode: 0644]
src/basic/log.h

index 3f6c988..d835e3f 100644 (file)
 #include "util.h"
 
 unsigned long cap_last_cap(void);
-// UNNEEDED int have_effective_cap(int value);
-// UNNEEDED int capability_bounding_set_drop(uint64_t drop, bool right_now);
-// UNNEEDED int capability_bounding_set_drop_usermode(uint64_t drop);
+/// UNNEEDED by elogind
+#if 0
+int have_effective_cap(int value);
+int capability_bounding_set_drop(uint64_t drop, bool right_now);
+int capability_bounding_set_drop_usermode(uint64_t drop);
 
-// UNNEEDED int drop_privileges(uid_t uid, gid_t gid, uint64_t keep_capabilities);
+int drop_privileges(uid_t uid, gid_t gid, uint64_t keep_capabilities);
 
-// UNNEEDED int drop_capability(cap_value_t cv);
+int drop_capability(cap_value_t cv);
 
-/// UNNEEDED by elogind
-#if 0
 DEFINE_TRIVIAL_CLEANUP_FUNC(cap_t, cap_free);
 #define _cleanup_cap_free_ _cleanup_(cap_freep)
 
index b82f2a8..6fd7727 100644 (file)
@@ -125,11 +125,13 @@ int cg_attach_fallback(const char *controller, const char *path, pid_t pid);
 int cg_create_and_attach(const char *controller, const char *path, pid_t pid);
 
 int cg_set_attribute(const char *controller, const char *path, const char *attribute, const char *value);
-// UNNEEDED int cg_get_attribute(const char *controller, const char *path, const char *attribute, char **ret);
-
-// UNNEEDED int cg_set_group_access(const char *controller, const char *path, mode_t mode, uid_t uid, gid_t gid);
-// UNNEEDED int cg_set_task_access(const char *controller, const char *path, mode_t mode, uid_t uid, gid_t gid);
+/// UNNEEDED by elogind
+#if 0
+int cg_get_attribute(const char *controller, const char *path, const char *attribute, char **ret);
 
+int cg_set_group_access(const char *controller, const char *path, mode_t mode, uid_t uid, gid_t gid);
+int cg_set_task_access(const char *controller, const char *path, mode_t mode, uid_t uid, gid_t gid);
+#endif // 0
 int cg_install_release_agent(const char *controller, const char *agent);
 int cg_uninstall_release_agent(const char *controller);
 
@@ -139,54 +141,65 @@ int cg_is_empty_recursive(const char *controller, const char *path);
 int cg_get_root_path(char **path);
 
 int cg_path_get_session(const char *path, char **session);
-// UNNEEDED int cg_path_get_owner_uid(const char *path, uid_t *uid);
-// UNNEEDED int cg_path_get_unit(const char *path, char **unit);
-// UNNEEDED int cg_path_get_user_unit(const char *path, char **unit);
-// UNNEEDED int cg_path_get_machine_name(const char *path, char **machine);
-// UNNEEDED int cg_path_get_slice(const char *path, char **slice);
-// UNNEEDED int cg_path_get_user_slice(const char *path, char **slice);
-
+/// UNNEEDED by elogind
+#if 0
+int cg_path_get_owner_uid(const char *path, uid_t *uid);
+int cg_path_get_unit(const char *path, char **unit);
+int cg_path_get_user_unit(const char *path, char **unit);
+int cg_path_get_machine_name(const char *path, char **machine);
+int cg_path_get_slice(const char *path, char **slice);
+int cg_path_get_user_slice(const char *path, char **slice);
+#endif // 0
 int cg_shift_path(const char *cgroup, const char *cached_root, const char **shifted);
 int cg_pid_get_path_shifted(pid_t pid, const char *cached_root, char **cgroup);
 
 int cg_pid_get_session(pid_t pid, char **session);
-// UNNEEDED int cg_pid_get_owner_uid(pid_t pid, uid_t *uid);
-// UNNEEDED int cg_pid_get_unit(pid_t pid, char **unit);
-// UNNEEDED int cg_pid_get_user_unit(pid_t pid, char **unit);
-// UNNEEDED int cg_pid_get_machine_name(pid_t pid, char **machine);
-// UNNEEDED int cg_pid_get_slice(pid_t pid, char **slice);
-// UNNEEDED int cg_pid_get_user_slice(pid_t pid, char **slice);
-
-// UNNEEDED int cg_path_decode_unit(const char *cgroup, char **unit);
-
+/// UNNEEDED by elogind
+#if 0
+int cg_pid_get_owner_uid(pid_t pid, uid_t *uid);
+int cg_pid_get_unit(pid_t pid, char **unit);
+int cg_pid_get_user_unit(pid_t pid, char **unit);
+int cg_pid_get_machine_name(pid_t pid, char **machine);
+int cg_pid_get_slice(pid_t pid, char **slice);
+int cg_pid_get_user_slice(pid_t pid, char **slice);
+
+int cg_path_decode_unit(const char *cgroup, char **unit);
+#endif // 0
 char *cg_escape(const char *p);
 char *cg_unescape(const char *p) _pure_;
 
 bool cg_controller_is_valid(const char *p);
-
-// UNNEEDED int cg_slice_to_path(const char *unit, char **ret);
-
-// UNNEEDED typedef const char* (*cg_migrate_callback_t)(CGroupMask mask, void *userdata);
-
-// UNNEEDED int cg_create_everywhere(CGroupMask supported, CGroupMask mask, const char *path);
-// UNNEEDED int cg_attach_everywhere(CGroupMask supported, const char *path, pid_t pid, cg_migrate_callback_t callback, void *userdata);
-// UNNEEDED int cg_attach_many_everywhere(CGroupMask supported, const char *path, Set* pids, cg_migrate_callback_t callback, void *userdata);
-// UNNEEDED int cg_migrate_everywhere(CGroupMask supported, const char *from, const char *to, cg_migrate_callback_t callback, void *userdata);
-// UNNEEDED int cg_trim_everywhere(CGroupMask supported, const char *path, bool delete_root);
-// UNNEEDED int cg_enable_everywhere(CGroupMask supported, CGroupMask mask, const char *p);
-
+/// UNNEEDED by elogind
+#if 0
+int cg_slice_to_path(const char *unit, char **ret);
+
+typedef const char* (*cg_migrate_callback_t)(CGroupMask mask, void *userdata);
+
+int cg_create_everywhere(CGroupMask supported, CGroupMask mask, const char *path);
+int cg_attach_everywhere(CGroupMask supported, const char *path, pid_t pid, cg_migrate_callback_t callback, void *userdata);
+int cg_attach_many_everywhere(CGroupMask supported, const char *path, Set* pids, cg_migrate_callback_t callback, void *userdata);
+int cg_migrate_everywhere(CGroupMask supported, const char *from, const char *to, cg_migrate_callback_t callback, void *userdata);
+int cg_trim_everywhere(CGroupMask supported, const char *path, bool delete_root);
+int cg_enable_everywhere(CGroupMask supported, CGroupMask mask, const char *p);
+#endif // 0
 int cg_mask_supported(CGroupMask *ret);
-
-// UNNEEDED int cg_kernel_controllers(Set *controllers);
-
+/// UNNEEDED by elogind
+#if 0
+int cg_kernel_controllers(Set *controllers);
+#endif // 0
 int cg_unified(void);
-// UNNEEDED void cg_unified_flush(void);
+/// UNNEEDED by elogind
+#if 0
+void cg_unified_flush(void);
 
-// UNNEEDED bool cg_is_unified_wanted(void);
+bool cg_is_unified_wanted(void);
+#endif // 0
 bool cg_is_legacy_wanted(void);
 
 const char* cgroup_controller_to_string(CGroupController c) _const_;
 CGroupController cgroup_controller_from_string(const char *s) _pure_;
-
-// UNNEEDED int cg_cpu_shares_parse(const char *s, uint64_t *ret);
-// UNNEEDED int cg_blkio_weight_parse(const char *s, uint64_t *ret);
+/// UNNEEDED by elogind
+#if 0
+int cg_cpu_shares_parse(const char *s, uint64_t *ret);
+int cg_blkio_weight_parse(const char *s, uint64_t *ret);
+#endif // 0
index c5cbbb7..cff446d 100644 (file)
@@ -45,7 +45,7 @@ int copy_bytes(int fdf, int fdt, uint64_t max_bytes, bool try_reflink) {
 
         assert(fdf >= 0);
         assert(fdt >= 0);
-// UNNEEDED by elogind
+/// UNNEEDED by elogind
 #if 0
         /* Try btrfs reflinks first. */
         if (try_reflink &&
@@ -129,7 +129,7 @@ int copy_bytes(int fdf, int fdt, uint64_t max_bytes, bool try_reflink) {
         return 0; /* return 0 if we hit EOF earlier than the size limit */
 }
 
-// UNNEEDED by elogind
+/// UNNEEDED by elogind
 #if 0
 static int fd_copy_symlink(int df, const char *from, const struct stat *st, int dt, const char *to) {
         _cleanup_free_ char *target = NULL;
index f5d8a1d..2cc7ad1 100644 (file)
 #include <stdbool.h>
 #include <sys/types.h>
 
-// UNNEEDED int copy_file_fd(const char *from, int to, bool try_reflink);
-// UNNEEDED int copy_file(const char *from, const char *to, int flags, mode_t mode, unsigned chattr_flags);
-// UNNEEDED int copy_file_atomic(const char *from, const char *to, mode_t mode, bool replace, unsigned chattr_flags);
-// UNNEEDED int copy_tree(const char *from, const char *to, bool merge);
-// UNNEEDED int copy_tree_at(int fdf, const char *from, int fdt, const char *to, bool merge);
-// UNNEEDED int copy_directory_fd(int dirfd, const char *to, bool merge);
+/// UNNEEDED by elogind
+#if 0
+int copy_file_fd(const char *from, int to, bool try_reflink);
+int copy_file(const char *from, const char *to, int flags, mode_t mode, unsigned chattr_flags);
+int copy_file_atomic(const char *from, const char *to, mode_t mode, bool replace, unsigned chattr_flags);
+int copy_tree(const char *from, const char *to, bool merge);
+int copy_tree_at(int fdf, const char *from, int fdt, const char *to, bool merge);
+int copy_directory_fd(int dirfd, const char *to, bool merge);
+#endif // 0
 int copy_bytes(int fdf, int fdt, uint64_t max_bytes, bool try_reflink);
-// UNNEEDED int copy_times(int fdf, int fdt);
-// UNNEEDED int copy_xattr(int fdf, int fdt);
+/// UNNEEDED by elogind
+#if 0
+int copy_times(int fdf, int fdt);
+int copy_xattr(int fdf, int fdt);
+#endif // 0
index 0b819fc..752e139 100644 (file)
@@ -24,4 +24,7 @@
 const char *errno_to_name(int id);
 int errno_from_name(const char *name);
 
-// UNNEEDED int errno_max(void);
+/// UNNEEDED by elogind
+#if 0
+int errno_max(void);
+#endif // 0
index eb33224..9944825 100644 (file)
@@ -33,5 +33,8 @@ typedef enum ExtractFlags {
 } ExtractFlags;
 
 int extract_first_word(const char **p, char **ret, const char *separators, ExtractFlags flags);
-// UNNEEDED int extract_first_word_and_warn(const char **p, char **ret, const char *separators, ExtractFlags flags, const char *unit, const char *filename, unsigned line, const char *rvalue);
-// UNNEEDED int extract_many_words(const char **p, const char *separators, ExtractFlags flags, ...) _sentinel_;
+/// UNNEEDED by elogind
+#if 0
+int extract_first_word_and_warn(const char **p, char **ret, const char *separators, ExtractFlags flags, const char *unit, const char *filename, unsigned line, const char *rvalue);
+int extract_many_words(const char **p, const char *separators, ExtractFlags flags, ...) _sentinel_;
+#endif // 0
index 0d742b5..0e03e26 100644 (file)
@@ -40,7 +40,10 @@ void close_many(const int fds[], unsigned n_fd);
 
 int fclose_nointr(FILE *f);
 FILE* safe_fclose(FILE *f);
-// UNNEEDED DIR* safe_closedir(DIR *f);
+/// UNNEEDED by elogind
+#if 0
+DIR* safe_closedir(DIR *f);
+#endif // 0
 
 static inline void closep(int *fd) {
         safe_close(*fd);
@@ -68,8 +71,11 @@ int fd_cloexec(int fd, bool cloexec);
 
 int close_all_fds(const int except[], unsigned n_except);
 
-// UNNEEDED int same_fd(int a, int b);
+/// UNNEEDED by elogind
+#if 0
+int same_fd(int a, int b);
 
-// UNNEEDED void cmsg_close_all(struct msghdr *mh);
+void cmsg_close_all(struct msghdr *mh);
 
-// UNNEEDED bool fdname_is_valid(const char *s);
+bool fdname_is_valid(const char *s);
+#endif // 0
index 81f6205..9fef650 100644 (file)
 #include "fileio.h"
 
 int write_string_file_atomic_label(const char *fn, const char *line);
-// UNNEEDED int write_env_file_label(const char *fname, char **l);
-// UNNEEDED int fopen_temporary_label(const char *target,
-// UNNEEDED                           const char *path, FILE **f, char **temp_path);
+
+/// UNNEEDED by elogind
+#if 0
+int write_env_file_label(const char *fname, char **l);
+int fopen_temporary_label(const char *target,
+                          const char *path, FILE **f, char **temp_path);
+#endif // 0
index 23b76cd..f45b0d5 100644 (file)
@@ -51,16 +51,19 @@ int load_env_file(FILE *f, const char *fname, const char *separator, char ***l);
 int load_env_file_pairs(FILE *f, const char *fname, const char *separator, char ***l);
 
 int write_env_file(const char *fname, char **l);
-
-// UNNEEDED int executable_is_script(const char *path, char **interpreter);
-
+/// UNNEEDED by elogind
+#if 0
+int executable_is_script(const char *path, char **interpreter);
+#endif // 0
 int get_proc_field(const char *filename, const char *pattern, const char *terminator, char **field);
 
 DIR *xopendirat(int dirfd, const char *name, int flags);
 
 int search_and_fopen(const char *path, const char *mode, const char *root, const char **search, FILE **_f);
-// UNNEEDED int search_and_fopen_nulstr(const char *path, const char *mode, const char *root, const char *search, FILE **_f);
-
+/// UNNEEDED by elogind
+#if 0
+int search_and_fopen_nulstr(const char *path, const char *mode, const char *root, const char *search, FILE **_f);
+#endif // 0
 #define FOREACH_LINE(line, f, on_error)                         \
         for (;;)                                                \
                 if (!fgets(line, sizeof(line), f)) {            \
@@ -74,11 +77,17 @@ int fflush_and_check(FILE *f);
 
 int fopen_temporary(const char *path, FILE **_f, char **_temp_path);
 int mkostemp_safe(char *pattern, int flags);
-// UNNEEDED int open_tmpfile(const char *path, int flags);
+/// UNNEEDED by elogind
+#if 0
+int open_tmpfile(const char *path, int flags);
+#endif // 0
 
 int tempfn_xxxxxx(const char *p, const char *extra, char **ret);
 int tempfn_random(const char *p, const char *extra, char **ret);
-// UNNEEDED int tempfn_random_child(const char *p, const char *extra, char **ret);
+/// UNNEEDED by elogind
+#if 0
+int tempfn_random_child(const char *p, const char *extra, char **ret);
 
-// UNNEEDED int write_timestamp_file_atomic(const char *fn, usec_t n);
-// UNNEEDED int read_timestamp_file(const char *fn, usec_t *ret);
+int write_timestamp_file_atomic(const char *fn, usec_t n);
+int read_timestamp_file(const char *fn, usec_t *ret);
+#endif // 0
index d36d2d6..55ee990 100644 (file)
 #include "time-util.h"
 
 int unlink_noerrno(const char *path);
+/// UNNEEDED by elogind
+#if 0
+int rmdir_parents(const char *path, const char *stop);
 
-// UNNEEDED int rmdir_parents(const char *path, const char *stop);
-
-// UNNEEDED int rename_noreplace(int olddirfd, const char *oldpath, int newdirfd, const char *newpath);
-
+int rename_noreplace(int olddirfd, const char *oldpath, int newdirfd, const char *newpath);
+#endif // 0
 int readlinkat_malloc(int fd, const char *p, char **ret);
 int readlink_malloc(const char *p, char **r);
-// UNNEEDED int readlink_value(const char *p, char **ret);
-// UNNEEDED int readlink_and_make_absolute(const char *p, char **r);
-// UNNEEDED int readlink_and_canonicalize(const char *p, char **r);
-// UNNEEDED int readlink_and_make_absolute_root(const char *root, const char *path, char **ret);
-
+/// UNNEEDED by elogind
+#if 0
+int readlink_value(const char *p, char **ret);
+int readlink_and_make_absolute(const char *p, char **r);
+int readlink_and_canonicalize(const char *p, char **r);
+int readlink_and_make_absolute_root(const char *root, const char *path, char **ret);
+#endif // 0
 int chmod_and_chown(const char *path, mode_t mode, uid_t uid, gid_t gid);
-// UNNEEDED int fchmod_and_fchown(int fd, mode_t mode, uid_t uid, gid_t gid);
-
+/// UNNEEDED by elogind
+#if 0
+int fchmod_and_fchown(int fd, mode_t mode, uid_t uid, gid_t gid);
+#endif // 0
 int fchmod_umask(int fd, mode_t mode);
 
 int fd_warn_permissions(const char *path, int fd);
@@ -53,13 +58,14 @@ int fd_warn_permissions(const char *path, int fd);
 
 int touch_file(const char *path, bool parents, usec_t stamp, uid_t uid, gid_t gid, mode_t mode);
 int touch(const char *path);
+/// UNNEEDED by elogind
+#if 0
+int symlink_idempotent(const char *from, const char *to);
 
-// UNNEEDED int symlink_idempotent(const char *from, const char *to);
-
-// UNNEEDED int symlink_atomic(const char *from, const char *to);
-// UNNEEDED int mknod_atomic(const char *path, mode_t mode, dev_t dev);
-// UNNEEDED int mkfifo_atomic(const char *path, mode_t mode);
-
+int symlink_atomic(const char *from, const char *to);
+int mknod_atomic(const char *path, mode_t mode, dev_t dev);
+int mkfifo_atomic(const char *path, mode_t mode);
+#endif // 0
 int get_files_in_directory(const char *path, char ***list);
 
 /// UNNEEDED by elogind
index 54f6025..c194575 100644 (file)
@@ -368,10 +368,11 @@ static inline void *hashmap_first(Hashmap *h) {
 static inline void *ordered_hashmap_first(OrderedHashmap *h) {
         return internal_hashmap_first(HASHMAP_BASE(h));
 }
-
+/// UNNEEDED by elogind
+#if 0
 /* no hashmap_next */
-// UNNEEDED void *ordered_hashmap_next(OrderedHashmap *h, const void *key);
-
+void *ordered_hashmap_next(OrderedHashmap *h, const void *key);
+#endif // 0
 char **internal_hashmap_get_strv(HashmapBase *h);
 static inline char **hashmap_get_strv(Hashmap *h) {
         return internal_hashmap_get_strv(HASHMAP_BASE(h));
index dd0960e..6eba0a0 100644 (file)
 
 #include "macro.h"
 
-// UNNEEDED bool hostname_is_set(void);
-
-// UNNEEDED char* gethostname_malloc(void);
+/// UNNEEDED by elogind
+#if 0
+bool hostname_is_set(void);
 
+char* gethostname_malloc(void);
+#endif // 0
 bool hostname_is_valid(const char *s, bool allow_trailing_dot) _pure_;
-// UNNEEDED char* hostname_cleanup(char *s);
+/// UNNEEDED by elogind
+#if 0
+char* hostname_cleanup(char *s);
+#endif // 0
 
 #define machine_name_is_valid(s) hostname_is_valid(s, false)
 
 bool is_localhost(const char *hostname);
-// UNNEEDED bool is_gateway_hostname(const char *hostname);
+/// UNNEEDED by elogind
+#if 0
+bool is_gateway_hostname(const char *hostname);
 
-// UNNEEDED int sethostname_idempotent(const char *s);
+int sethostname_idempotent(const char *s);
 
-// UNNEEDED int read_hostname_config(const char *path, char **hostname);
+int read_hostname_config(const char *path, char **hostname);
+#endif // 0
diff --git a/src/basic/label.h b/src/basic/label.h
new file mode 100644 (file)
index 0000000..35feb7c
--- /dev/null
@@ -0,0 +1,33 @@
+/*-*- Mode: C; c-basic-offset: 8; indent-tabs-mode: nil -*-*/
+
+#pragma once
+
+/***
+  This file is part of systemd.
+
+  Copyright 2010 Lennart Poettering
+
+  systemd is free software; you can redistribute it and/or modify it
+  under the terms of the GNU Lesser General Public License as published by
+  the Free Software Foundation; either version 2.1 of the License, or
+  (at your option) any later version.
+
+  systemd is distributed in the hope that it will be useful, but
+  WITHOUT ANY WARRANTY; without even the implied warranty of
+  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+  Lesser General Public License for more details.
+
+  You should have received a copy of the GNU Lesser General Public License
+  along with systemd; If not, see <http://www.gnu.org/licenses/>.
+***/
+
+#include <stdbool.h>
+#include <sys/types.h>
+
+int label_fix(const char *path, bool ignore_enoent, bool ignore_erofs);
+
+int mkdir_label(const char *path, mode_t mode);
+/// UNNEEDED by elogind
+#if 0
+int symlink_label(const char *old_path, const char *new_path);
+#endif // 0
index e6314e8..4325215 100644 (file)
@@ -70,10 +70,15 @@ int log_get_max_level(void) _pure_;
 
 int log_open(void);
 void log_close(void);
-// UNNEEDED void log_forget_fds(void);
-
+/// UNNEEDED by elogind
+#if 0
+void log_forget_fds(void);
+#endif // 0
 void log_close_syslog(void);
-// UNNEEDED void log_close_journal(void);
+/// UNNEEDED by elogind
+#if 0
+void log_close_journal(void);
+#endif // 0
 void log_close_kmsg(void);
 void log_close_console(void);
 
@@ -215,9 +220,12 @@ LogTarget log_target_from_string(const char *s) _pure_;
 #define LOG_MESSAGE(fmt, ...) "MESSAGE=" fmt, ##__VA_ARGS__
 #define LOG_MESSAGE_ID(x) "MESSAGE_ID=" SD_ID128_FORMAT_STR, SD_ID128_FORMAT_VAL(x)
 
-// UNNEEDED void log_received_signal(int level, const struct signalfd_siginfo *si);
+/// UNNEEDED by elogind
+#if 0
+void log_received_signal(int level, const struct signalfd_siginfo *si);
 
-// UNNEEDED void log_set_upgrade_syslog_to_journal(bool b);
+void log_set_upgrade_syslog_to_journal(bool b);
+#endif // 0
 
 int log_syntax_internal(
                 const char *unit,