]> git.netwichtig.de Git - user/henk/code/inspircd.git/commitdiff
Comments (lots of)
authorbrain <brain@e03df62e-2008-0410-955e-edbf42e46eb7>
Sun, 17 Jun 2007 18:45:22 +0000 (18:45 +0000)
committerbrain <brain@e03df62e-2008-0410-955e-edbf42e46eb7>
Sun, 17 Jun 2007 18:45:22 +0000 (18:45 +0000)
git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@7377 e03df62e-2008-0410-955e-edbf42e46eb7

include/dynamic.h
include/exitcodes.h
include/globals.h
include/hash_map.h
include/hashcomp.h

index 12ad61aed64a9b2e06955ad3f58938387b1c356e..db46291c4b888f34acec0d1101ec725431c5383f 100644 (file)
@@ -46,14 +46,16 @@ class CoreExport DLLManager
        bool GetSymbol(void **v, const char *sym_name);
 
        /** Get the last error from dlopen() or dlsym().
-        * @return The last error string, or NULL if no error has occured
+        * @return The last error string, or NULL if no error has occured.
         */
        char* LastError() 
        {
                 return err;
        }
 
-       /** The module handle
+       /** The module handle.
+        * This is OS dependent, on POSIX platforms it is a pointer to a function
+        * pointer (yes, really!) and on windows it is a library handle.
         */
        void *h;
 
@@ -77,11 +79,11 @@ class CoreExport DLLFactoryBase : public DLLManager
         */
        DLLFactoryBase(InspIRCd* Instance, const char *fname, const char *func_name = 0);
 
-       /** Default destructor
+       /** Default destructor.
         */
        virtual ~DLLFactoryBase();
 
-       /** A function pointer to the factory function
+       /** A function pointer to the factory function.
         */
        void * (*factory_func)(void);   
 };
@@ -122,3 +124,4 @@ template <class T> class CoreExport DLLFactory : public DLLFactoryBase
 };
 
 #endif
+
index 692e731e0923da6de24e25e9dd370d04f63fd2f6..d6830198448723e0283667159cc20dc2995a5dd6 100644 (file)
  */
 enum ExitStatus
 {
-       EXIT_STATUS_NOERROR = 0,
-       EXIT_STATUS_DIE = 1,
-       EXIT_STATUS_FAILED_EXEC = 2,
-       EXIT_STATUS_INTERNAL = 3,
-       EXIT_STATUS_CONFIG = 4,
-       EXIT_STATUS_LOG = 5,
-       EXIT_STATUS_FORK = 6,
-       EXIT_STATUS_ARGV = 7,
-       EXIT_STATUS_BIND = 8,
-       EXIT_STATUS_PID = 9,
-       EXIT_STATUS_SOCKETENGINE = 10,
-       EXIT_STATUS_ROOT = 11,
-       EXIT_STATUS_DIETAG = 12,
-       EXIT_STATUS_MODULE = 13,
+       EXIT_STATUS_NOERROR = 0,        /* No error */
+       EXIT_STATUS_DIE = 1,            /* Operator issued DIE */
+       EXIT_STATUS_FAILED_EXEC = 2,    /* execv() failed */
+       EXIT_STATUS_INTERNAL = 3,       /* Internal error */
+       EXIT_STATUS_CONFIG = 4,         /* Config error */
+       EXIT_STATUS_LOG = 5,            /* Log file error */
+       EXIT_STATUS_FORK = 6,           /* fork() failed */
+       EXIT_STATUS_ARGV = 7,           /* Invalid program arguments */
+       EXIT_STATUS_BIND = 8,           /* Port binding failed on all ports */
+       EXIT_STATUS_PID = 9,            /* Couldn't write PID file */
+       EXIT_STATUS_SOCKETENGINE = 10,  /* Couldn't start socket engine */
+       EXIT_STATUS_ROOT = 11,          /* Refusing to start as root */
+       EXIT_STATUS_DIETAG = 12,        /* Found a die tag in the config file */
+       EXIT_STATUS_MODULE = 13,        /* Couldn't load a required module */
+       EXIT_STATUS_CREATEPROCESS = 14, /* CreateProcess failed (windows) */
        EXIT_STATUS_SIGTERM = 15        /* Note: dont move this value. It corresponds with the value of #define SIGTERM. */
 };
 
@@ -41,3 +42,4 @@ enum ExitStatus
 extern const char * ExitCodes[];
 
 #endif
+
index bd5fdb621ac115a5967560b6491c70c5fe19f0c9..4a01e454eed81eef909734d8f9b0c99a36708bc1 100644 (file)
 #include <map>
 #include <vector>
 
+/** A cached text file stored with its contents as lines
+ */
 typedef std::deque<std::string> file_cache;
+
+/** A configuration key and value pair
+ */
 typedef std::pair< std::string, std::string > KeyVal;
+
+/** A list of related configuration keys and values
+ */
 typedef std::vector< KeyVal > KeyValList;
+
+/** An entire config file, built up of KeyValLists
+ */
 typedef std::multimap< std::string, KeyValList > ConfigDataHash;
 
 #endif
+
index ad38d426aac39abe7e3d4446acb82667f2115a76..784a8bc3b2a21b7925d7786cffebdf035297a473 100644 (file)
 
 #include "inspircd_config.h"
 
+/** Where hash_map is varies from compiler to compiler
+ * as it is not standard.
+ */
 #ifndef WIN32
 #include <ext/hash_map>
+/** Oddball linux namespace for hash_map */
 #define nspace __gnu_cxx
 #else
 #include <hash_map>
 #define nspace stdext
+/** Oddball windows namespace for hash_map */
 using stdext::hash_map;
 #endif
 
index 338e23a09d24939c57958bc7b44ad0c42746cfa8..0556f4399aefda9fb46042f0a0f24942fbfd3755 100644 (file)
  * backwards compatible with other code which is not
  * aware of irc::string.
  *******************************************************/
+
+/** Required namespaces and symbols */
 using namespace std;
+
+/** aton() */
 using irc::sockets::insp_aton;
+
+/** nota() */
 using irc::sockets::insp_ntoa;
 
 #ifndef LOWERMAP
@@ -80,30 +85,53 @@ namespace irc
         */
        struct irc_char_traits : std::char_traits<char> {
 
-               /** Check if two chars match
+               /** Check if two chars match.
+                * @param c1st First character
+                * @param c2nd Second character
+                * @return true if the characters are equal
                 */
                static bool eq(char c1st, char c2nd);
 
-               /** Check if two chars do NOT match
+               /** Check if two chars do NOT match.
+                * @param c1st First character
+                * @param c2nd Second character
+                * @return true if the characters are unequal
                 */
                static bool ne(char c1st, char c2nd);
 
-               /** Check if one char is less than another
+               /** Check if one char is less than another.
+                * @param c1st First character
+                * @param c2nd Second character
+                * @return true if c1st is less than c2nd
                 */
                static bool lt(char c1st, char c2nd);
 
-               /** Compare two strings of size n
+               /** Compare two strings of size n.
+                * @param str1 First string
+                * @param str2 Second string
+                * @param n Length to compare to
+                * @return similar to strcmp, zero for equal, less than zero for str1
+                * being less and greater than zero for str1 being greater than str2.
                 */
                static CoreExport int compare(const char* str1, const char* str2, size_t n);
 
-               /** Find a char within a string up to position n
+               /** Find a char within a string up to position n.
+                * @param s1 String to find in
+                * @param n Position to search up to
+                * @param c Character to search for
+                * @return Pointer to the first occurance of c in s1
                 */
                static CoreExport const char* find(const char* s1, int  n, char c);
        };
 
+       /** Compose a hex string from raw data.
+        * @param raw The raw data to compose hex from
+        * @pram rawsz The size of the raw data buffer
+        * @return The hex string.
+        */
        CoreExport std::string hex(const unsigned char *raw, size_t rawsz);
 
-       /** This typedef declares irc::string based upon irc_char_traits
+       /** This typedef declares irc::string based upon irc_char_traits.
         */
        typedef basic_string<char, irc_char_traits, allocator<char> > string;
 
@@ -194,7 +222,7 @@ namespace irc
                void PushMinus();
                /** Return zero or more elements which form the
                 * mode line. This will be clamped to a max of
-                * MAXMODES+1 items (MAXMODES mode parameters and
+                * MAXMODES items (MAXMODES-1 mode parameters and
                 * one mode sequence string), and max_line_size
                 * characters. As specified below, this function
                 * should be called in a loop until it returns zero,
@@ -244,12 +272,28 @@ namespace irc
                tokenstream(const std::string &source);
                ~tokenstream();
 
-               /** Fetch the next token from the stream
-                * @return The next token is returned, or an empty string if none remain
+               /** Fetch the next token from the stream as a std::string
+                * @param token The next token available, or an empty string if none remain
+                * @return True if tokens are left to be read, false if the last token was just retrieved.
                 */
                bool GetToken(std::string &token);
+
+               /** Fetch the next token from the stream as an irc::string
+                * @param token The next token available, or an empty string if none remain
+                * @return True if tokens are left to be read, false if the last token was just retrieved.
+                */
                bool GetToken(irc::string &token);
+
+               /** Fetch the next token from the stream as an integer
+                * @param token The next token available, or undefined if none remain
+                * @return True if tokens are left to be read, false if the last token was just retrieved.
+                */
                bool GetToken(int &token);
+
+               /** Fetch the next token from the stream as a long integer
+                * @param token The next token available, or undefined if none remain
+                * @return True if tokens are left to be read, false if the last token was just retrieved.
+                */
                bool GetToken(long &token);
        };
 
@@ -261,7 +305,7 @@ namespace irc
        class CoreExport sepstream : public classbase
        {
         private:
-               /** Original string
+               /** Original string.
                 */
                std::string tokens;
                /** Last position of a seperator token
@@ -300,6 +344,8 @@ namespace irc
        class CoreExport commasepstream : public sepstream
        {
         public:
+               /** Initialize with comma seperator
+                */
                commasepstream(const std::string &source) : sepstream(source, ',')
                {
                }
@@ -310,6 +356,8 @@ namespace irc
        class CoreExport spacesepstream : public sepstream
        {
         public:
+               /** Initialize with space seperator
+                */
                spacesepstream(const std::string &source) : sepstream(source, ' ')
                {
                }
@@ -491,18 +539,32 @@ namespace irc
                 */
                unsigned char GetSize();
 
+               /** Get free bits mask
+                */
                virtual unsigned char* GetFreeBits() { return NULL; }
 
+               /** Set free bits mask
+                */
                virtual void SetFreeBits(unsigned char* freebits) { }
        };
 
+       /** Turn _ characters in a string into spaces
+        * @param n String to translate
+        * @return The new value with _ translated to space.
+        */
        CoreExport const char* Spacify(const char* n);
 }
 
 /* Define operators for using >> and << with irc::string to an ostream on an istream. */
 /* This was endless fun. No. Really. */
 /* It was also the first core change Ommeh made, if anyone cares */
+
+/** Operator << for irc::string
+ */
 inline std::ostream& operator<<(std::ostream &os, const irc::string &str) { return os << str.c_str(); }
+
+/** Operator >> for irc::string
+ */
 inline std::istream& operator>>(std::istream &is, irc::string &str)
 {
        std::string tmp;
@@ -512,30 +574,55 @@ inline std::istream& operator>>(std::istream &is, irc::string &str)
 }
 
 /* Define operators for + and == with irc::string to std::string for easy assignment
- * and comparison - Brain
+ * and comparison
+ *
+ * Operator +
  */
 inline std::string operator+ (std::string& leftval, irc::string& rightval)
 {
        return leftval + std::string(rightval.c_str());
 }
 
+/* Define operators for + and == with irc::string to std::string for easy assignment
+ * and comparison
+ *
+ * Operator +
+ */
 inline irc::string operator+ (irc::string& leftval, std::string& rightval)
 {
        return leftval + irc::string(rightval.c_str());
 }
 
+/* Define operators for + and == with irc::string to std::string for easy assignment
+ * and comparison
+ *
+ * Operator ==
+ */
 inline bool operator== (const std::string& leftval, const irc::string& rightval)
 {
        return (leftval.c_str() == rightval);
 }
 
+/* Define operators for + and == with irc::string to std::string for easy assignment
+ * and comparison
+ *
+ * Operator ==
+ */
 inline bool operator== (const irc::string& leftval, const std::string& rightval)
 {
        return (leftval == rightval.c_str());
 }
 
+/** Assign an irc::string to a std::string.
+ */
 inline std::string assign(const irc::string &other) { return other.c_str(); }
+
+/** Assign a std::string to an irc::string.
+ */
 inline irc::string assign(const std::string &other) { return other.c_str(); }
+
+/** Trim the leading and trailing spaces from a std::string.
+ */
 inline std::string& trim(std::string &str)
 {
        std::string::size_type start = str.find_first_not_of(" ");
@@ -548,9 +635,9 @@ inline std::string& trim(std::string &str)
        return str;
 }
 
-/* Hashing stuff is totally different on vc++'s hash_map implementation, so to save a buttload of #ifdefs we'll just
-   do it all at once - Burlex */
-
+/** Hashing stuff is totally different on vc++'s hash_map implementation, so to save a buttload of
+ * #ifdefs we'll just do it all at once
+ */
 namespace nspace
 {
        /** Hashing function to hash irc::string
@@ -559,21 +646,28 @@ namespace nspace
        template<> class CoreExport hash_compare<irc::string, std::less<irc::string> >
        {
        public:
-               enum { bucket_size = 4, min_buckets = 8 };                      // Got these numbers from the CRT source,
-                                                                                                                       // if anyone wants to change them feel free.
+               enum { bucket_size = 4, min_buckets = 8 }; /* Got these numbers from the CRT source, if anyone wants to change them feel free. */
+
+               /** Compare two irc::string values for hashing in hash_map
+                */
                bool operator()(const irc::string & s1, const irc::string & s2) const
                {
                        if(s1.length() != s2.length()) return true;
                        return (irc::irc_char_traits::compare(s1.c_str(), s2.c_str(), s1.length()) < 0);
                }
                
+               /** Hash an irc::string value for hash_map
+                */
                size_t operator()(const irc::string & s) const;
        };
 
        template<> class CoreExport hash_compare<std::string, std::less<std::string> >
        {
        public:
-               enum { bucket_size = 4, min_buckets = 8 };
+               enum { bucket_size = 4, min_buckets = 8 }; /* Again, from the CRT source */
+
+               /** Compare two std::string values for hashing in hash_map
+                */
                bool operator()(const std::string & s1, const std::string & s2) const
                {
                        if(s1.length() != s2.length()) return true;
@@ -589,6 +683,10 @@ namespace nspace
 #else
        template<> struct hash<irc::string>
        {
+               /** Hash an irc::string using RFC1459 case sensitivity rules
+                * @param s A string to hash
+                * @return The hash value
+                */
                size_t operator()(const irc::string &s) const;
        };
 
@@ -609,3 +707,4 @@ namespace nspace
 }
 
 #endif
+