diff options
| author | brain <brain@e03df62e-2008-0410-955e-edbf42e46eb7> | 2007-10-23 21:00:35 +0000 |
|---|---|---|
| committer | brain <brain@e03df62e-2008-0410-955e-edbf42e46eb7> | 2007-10-23 21:00:35 +0000 |
| commit | 2e1f86fd0c9911210b79e6ac346672441eef18c4 (patch) | |
| tree | 9151566c4c38be05335dbbe907d64bc3085c664b /src/modules/m_services.cpp | |
| parent | 7240c926c8e9aeacf95c2bd4803c8879963cca3f (diff) | |
All of insp now builds with -pedantic (theres some warnings to squash in modules, and in spanningtree, but its all 'unused parameter').
I suggest we actually go through and satisfy these unused parameter errors for two reasons:
(1) it acts as a strong compiler hint leading to better optimization
(2) it will give us a good clue of what parameters are NEVER used and should therefore be removed from use (like i just did with AddMode)
git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@8323 e03df62e-2008-0410-955e-edbf42e46eb7
Diffstat (limited to 'src/modules/m_services.cpp')
| -rw-r--r-- | src/modules/m_services.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/modules/m_services.cpp b/src/modules/m_services.cpp index 88a6b8fb9..07d10aad5 100644 --- a/src/modules/m_services.cpp +++ b/src/modules/m_services.cpp @@ -179,8 +179,8 @@ class ModuleServices : public Module m4 = new User_r(ServerInstance); m5 = new User_R(ServerInstance); - if (!ServerInstance->AddMode(m1, 'r') || !ServerInstance->AddMode(m2, 'R') || !ServerInstance->AddMode(m3, 'M') - || !ServerInstance->AddMode(m4, 'r') || !ServerInstance->AddMode(m5, 'R')) + if (!ServerInstance->AddMode(m1) || !ServerInstance->AddMode(m2) || !ServerInstance->AddMode(m3) + || !ServerInstance->AddMode(m4) || !ServerInstance->AddMode(m5)) { throw ModuleException("Could not add user and channel modes!"); } |