From 7d30ffc907f9dea353da3a116420a7fab1ff985b Mon Sep 17 00:00:00 2001 From: Torsten Hilbrich Date: Wed, 14 Nov 2012 12:20:31 +0100 Subject: [PATCH] base: Fix a number of warnings about unused parameters The warnings are shown if components using the genode base are compiled with -Wextra -Wall enabled. Fixes #489. --- base/include/base/allocator_avl.h | 2 +- base/include/base/capability.h | 2 +- base/include/base/rpc_server.h | 4 ++-- base/include/base/slab.h | 2 +- base/include/cpu_session/cpu_session.h | 2 +- base/include/root/component.h | 6 +++--- base/include/util/meta.h | 22 +++++++++++----------- base/include/x86/cpu/string.h | 2 +- 8 files changed, 21 insertions(+), 21 deletions(-) diff --git a/base/include/base/allocator_avl.h b/base/include/base/allocator_avl.h index 89b705cfb8..c7d06be097 100644 --- a/base/include/base/allocator_avl.h +++ b/base/include/base/allocator_avl.h @@ -241,7 +241,7 @@ namespace Genode { * The 'sizeof(umword_t)' represents the overhead of the meta-data * slab allocator. */ - size_t overhead(size_t size) { return sizeof(Block) + sizeof(umword_t); } + size_t overhead(size_t) { return sizeof(Block) + sizeof(umword_t); } bool need_size_for_free() const { return false; } }; diff --git a/base/include/base/capability.h b/base/include/base/capability.h index 6e2bed4250..a8435e26fe 100644 --- a/base/include/base/capability.h +++ b/base/include/base/capability.h @@ -64,7 +64,7 @@ namespace Genode { Meta::Overload_selector) const; template - void _unmarshal_result(Ipc_client &, T &arg, + void _unmarshal_result(Ipc_client &, T &, Meta::Overload_selector) const { } /** diff --git a/base/include/base/rpc_server.h b/base/include/base/rpc_server.h index 9f4f4754eb..68110c9f9f 100644 --- a/base/include/base/rpc_server.h +++ b/base/include/base/rpc_server.h @@ -61,7 +61,7 @@ namespace Genode { _read_args(is, args._2); } - void _read_args(Ipc_istream &is, Meta::Empty) { } + void _read_args(Ipc_istream &, Meta::Empty) { } template void _write_results(Ipc_ostream &os, ARG_LIST &args) @@ -72,7 +72,7 @@ namespace Genode { _write_results(os, args._2); } - void _write_results(Ipc_ostream &os, Meta::Empty) { } + void _write_results(Ipc_ostream &, Meta::Empty) { } template Rpc_exception_code _do_serve(typename RPC_FUNCTION::Server_args &args, diff --git a/base/include/base/slab.h b/base/include/base/slab.h index 627327ab73..825606d862 100644 --- a/base/include/base/slab.h +++ b/base/include/base/slab.h @@ -248,7 +248,7 @@ namespace Genode { bool alloc(size_t, void **); void free(void *addr, size_t) { free(addr); } size_t consumed(); - size_t overhead(size_t size) { return _block_size/_num_elem; } + size_t overhead(size_t) { return _block_size/_num_elem; } bool need_size_for_free() const { return false; } }; } diff --git a/base/include/cpu_session/cpu_session.h b/base/include/cpu_session/cpu_session.h index aadaeb306e..df397389e3 100644 --- a/base/include/cpu_session/cpu_session.h +++ b/base/include/cpu_session/cpu_session.h @@ -149,7 +149,7 @@ namespace Genode { * \param thread thread to set into single step mode * \param enable true = enable single-step mode; false = disable */ - virtual void single_step(Thread_capability thread, bool enable) {} + virtual void single_step(Thread_capability, bool) {} /** * Return number of CPUs available via the CPU session diff --git a/base/include/root/component.h b/base/include/root/component.h index 666718e707..e224b9e221 100644 --- a/base/include/root/component.h +++ b/base/include/root/component.h @@ -39,7 +39,7 @@ namespace Genode { Single_client() : _used(0) { } - void aquire(const char *args) + void aquire(const char *) { if (_used) throw Root::Unavailable(); @@ -56,7 +56,7 @@ namespace Genode { */ struct Multiple_clients { - void aquire(const char *args) { } + void aquire(const char *) { } void release() { } }; @@ -146,7 +146,7 @@ namespace Genode { * \param args description of additional resources in the * same format as used at session creation */ - virtual void _upgrade_session(SESSION_TYPE *session, const char *args) { } + virtual void _upgrade_session(SESSION_TYPE *, const char *) { } virtual void _destroy_session(SESSION_TYPE *session) { destroy(_md_alloc, session); } diff --git a/base/include/util/meta.h b/base/include/util/meta.h index f407667ac4..5016578353 100644 --- a/base/include/util/meta.h +++ b/base/include/util/meta.h @@ -460,22 +460,22 @@ namespace Genode { */ template - static inline void call_member(RET_TYPE &ret, SERVER &server, ARGS &args, + static inline void call_member(RET_TYPE &ret, SERVER &server, ARGS &, RET_TYPE (SERVER::*func)()) { ret = (server.*func)(); } template - static inline void call_member(RET_TYPE &ret, SERVER &server, ARGS &args, + static inline void call_member(RET_TYPE &ret, SERVER &server, ARGS &, RET_TYPE (SERVER::*func)() const) { ret = (server.*func)(); } template - static inline void call_member(Meta::Empty &ret, SERVER &server, ARGS &args, + static inline void call_member(Meta::Empty &, SERVER &server, ARGS &, void (SERVER::*func)()) { (server.*func)(); } template - static inline void call_member(Meta::Empty &ret, SERVER &server, ARGS &args, + static inline void call_member(Meta::Empty &, SERVER &server, ARGS &, void (SERVER::*func)() const) { (server.*func)(); } @@ -486,7 +486,7 @@ namespace Genode { { ret = (server.*func)(args.get()); } template - static inline void call_member(Meta::Empty &ret, SERVER &server, ARGS &args, + static inline void call_member(Meta::Empty &, SERVER &server, ARGS &args, void (SERVER::*func)(typename Type_at::Type)) { (server.*func)(args.get()); } @@ -498,7 +498,7 @@ namespace Genode { { ret = (server.*func)(args.get(), args._2.get()); } template - static inline void call_member(Meta::Empty &ret, SERVER &server, ARGS &args, + static inline void call_member(Meta::Empty &, SERVER &server, ARGS &args, void (SERVER::*func)(typename Type_at::Type, typename Type_at::Type)) { (server.*func)(args.get(), args._2.get()); } @@ -512,7 +512,7 @@ namespace Genode { { ret = (server.*func)(args.get(), args._2.get(), args._2._2.get()); } template - static inline void call_member(Meta::Empty &ret, SERVER &server, ARGS &args, + static inline void call_member(Meta::Empty &, SERVER &server, ARGS &args, void (SERVER::*func)(typename Type_at::Type, typename Type_at::Type, typename Type_at::Type)) @@ -528,7 +528,7 @@ namespace Genode { { ret = (server.*func)(args.get(), args._2.get(), args._2._2.get(), args._2._2._2.get()); } template - static inline void call_member(Meta::Empty &ret, SERVER &server, ARGS &args, + static inline void call_member(Meta::Empty &, SERVER &server, ARGS &args, void (SERVER::*func)(typename Type_at::Type, typename Type_at::Type, typename Type_at::Type, @@ -546,7 +546,7 @@ namespace Genode { { ret = (server.*func)(args.get(), args._2.get(), args._2._2.get(), args._2._2._2.get(), args._2._2._2._2.get()); } template - static inline void call_member(Meta::Empty &ret, SERVER &server, ARGS &args, + static inline void call_member(Meta::Empty &, SERVER &server, ARGS &args, void (SERVER::*func)(typename Type_at::Type, typename Type_at::Type, typename Type_at::Type, @@ -567,7 +567,7 @@ namespace Genode { args._2._2._2._2.get(), args._2._2._2._2._2.get()); } template - static inline void call_member(Meta::Empty &ret, SERVER &server, ARGS &args, + static inline void call_member(Meta::Empty &, SERVER &server, ARGS &args, void (SERVER::*func)(typename Type_at::Type, typename Type_at::Type, typename Type_at::Type, @@ -591,7 +591,7 @@ namespace Genode { args._2._2._2._2.get(), args._2._2._2._2._2.get(), args._2._2._2._2._2._2.get()); } template - static inline void call_member(Meta::Empty &ret, SERVER &server, ARGS &args, + static inline void call_member(Meta::Empty &, SERVER &server, ARGS &args, void (SERVER::*func)(typename Type_at::Type, typename Type_at::Type, typename Type_at::Type, diff --git a/base/include/x86/cpu/string.h b/base/include/x86/cpu/string.h index a25e7b096d..ceb9eb1880 100644 --- a/base/include/x86/cpu/string.h +++ b/base/include/x86/cpu/string.h @@ -25,7 +25,7 @@ namespace Genode * * \return Number of bytes not copied */ - inline size_t memcpy_cpu(void *dst, const void *src, size_t size) { + inline size_t memcpy_cpu(void *, const void *, size_t size) { return size; } }