]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - include/xline.h
Improve X-line text consistency.
[user/henk/code/inspircd.git] / include / xline.h
index 7c102c8825c7df6469fb22d3e52b61ab22e880ed..dbb89277b95ab328926befe67780dfa15757f00b 100644 (file)
@@ -22,7 +22,7 @@
 
 #pragma once
 
-/** XLine is the base class for ban lines such as G lines and K lines.
+/** XLine is the base class for ban lines such as G-lines and K-lines.
  * Modules may derive from this, and their xlines will automatically be
  * handled as expected by any protocol modules (e.g. m_spanningtree will
  * propogate them using AddLine). The process of translating a type+pattern
@@ -49,7 +49,7 @@ class CoreExport XLine : public classbase
         * @param re The reason of the xline
         * @param t The line type, should be set by the derived class constructor
         */
-       XLine(time_t s_time, long d, std::string src, std::string re, const std::string &t)
+       XLine(time_t s_time, unsigned long d, const std::string& src, const std::string& re, const std::string& t)
                : set_time(s_time)
                , duration(d)
                , source(src)
@@ -67,7 +67,7 @@ class CoreExport XLine : public classbase
        }
 
        /** Change creation time of an xline. Updates expiry
-        * to be after the creation time
+        * to be after the creation time.
         */
        virtual void SetCreateTime(time_t created)
        {
@@ -102,7 +102,7 @@ class CoreExport XLine : public classbase
        virtual void Unset() { }
 
        /** Called when the expiry message is to be displayed for the
-        * line. Usually a line in the form 'expiring Xline blah, set by...'
+        * line. Usually a line in the form 'expiring X-line blah, set by...'
         * see the DisplayExpiry methods of GLine, ELine etc.
         */
        virtual void DisplayExpiry();
@@ -111,7 +111,7 @@ class CoreExport XLine : public classbase
         * e.g. '*\@foo' or '*baz*'. This must always return the full pattern
         * in a form which can be used to construct an entire derived xline,
         * even if it is stored differently internally (e.g. GLine stores the
-        * ident and host parts seperately but will still return ident\@host
+        * ident and host parts separately but will still return ident\@host
         * for its Displayable() method).
         */
        virtual const std::string& Displayable() = 0;
@@ -126,7 +126,7 @@ class CoreExport XLine : public classbase
 
        /** The duration of the ban, or 0 if permenant
         */
-       long duration;
+       unsigned long duration;
 
        /** Source of the ban. This can be a servername or an oper nickname
         */
@@ -157,7 +157,7 @@ class CoreExport KLine : public XLine
 {
   public:
 
-       /** Create a K-Line.
+       /** Create a K-line.
         * @param s_time The set time
         * @param d The duration of the xline
         * @param src The sender of the xline
@@ -165,7 +165,7 @@ class CoreExport KLine : public XLine
         * @param ident Ident to match
         * @param host Host to match
         */
-       KLine(time_t s_time, long d, std::string src, std::string re, std::string ident, std::string host)
+       KLine(time_t s_time, unsigned long d, const std::string& src, const std::string& re, const std::string& ident, const std::string& host)
                : XLine(s_time, d, src, re, "K"), identmask(ident), hostmask(host)
        {
                matchtext = this->identmask;
@@ -203,7 +203,7 @@ class CoreExport KLine : public XLine
 class CoreExport GLine : public XLine
 {
   public:
-       /** Create a G-Line.
+       /** Create a G-line.
         * @param s_time The set time
         * @param d The duration of the xline
         * @param src The sender of the xline
@@ -211,7 +211,7 @@ class CoreExport GLine : public XLine
         * @param ident Ident to match
         * @param host Host to match
         */
-       GLine(time_t s_time, long d, std::string src, std::string re, std::string ident, std::string host)
+       GLine(time_t s_time, unsigned long d, const std::string& src, const std::string& re, const std::string& ident, const std::string& host)
                : XLine(s_time, d, src, re, "G"), identmask(ident), hostmask(host)
        {
                matchtext = this->identmask;
@@ -247,7 +247,7 @@ class CoreExport GLine : public XLine
 class CoreExport ELine : public XLine
 {
   public:
-       /** Create an E-Line.
+       /** Create an E-line.
         * @param s_time The set time
         * @param d The duration of the xline
         * @param src The sender of the xline
@@ -255,7 +255,7 @@ class CoreExport ELine : public XLine
         * @param ident Ident to match
         * @param host Host to match
         */
-       ELine(time_t s_time, long d, std::string src, std::string re, std::string ident, std::string host)
+       ELine(time_t s_time, unsigned long d, const std::string& src, const std::string& re, const std::string& ident, const std::string& host)
                : XLine(s_time, d, src, re, "E"), identmask(ident), hostmask(host)
        {
                matchtext = this->identmask;
@@ -291,14 +291,14 @@ class CoreExport ELine : public XLine
 class CoreExport ZLine : public XLine
 {
   public:
-       /** Create a Z-Line.
+       /** Create a Z-line.
         * @param s_time The set time
         * @param d The duration of the xline
         * @param src The sender of the xline
         * @param re The reason of the xline
         * @param ip IP to match
         */
-       ZLine(time_t s_time, long d, std::string src, std::string re, std::string ip)
+       ZLine(time_t s_time, unsigned long d, const std::string& src, const std::string& re, const std::string& ip)
                : XLine(s_time, d, src, re, "Z"), ipaddr(ip)
        {
        }
@@ -327,14 +327,14 @@ class CoreExport ZLine : public XLine
 class CoreExport QLine : public XLine
 {
   public:
-       /** Create a G-Line.
+       /** Create a Q-line.
         * @param s_time The set time
         * @param d The duration of the xline
         * @param src The sender of the xline
         * @param re The reason of the xline
         * @param nickname Nickname to match
         */
-       QLine(time_t s_time, long d, std::string src, std::string re, std::string nickname)
+       QLine(time_t s_time, unsigned long d, const std::string& src, const std::string& re, const std::string& nickname)
                : XLine(s_time, d, src, re, "Q"), nick(nickname)
        {
        }
@@ -390,7 +390,7 @@ class CoreExport XLineFactory
         * @param xline_specific_mask The mask string for the line, specific to the XLine type being created.
         * @return A specialized XLine class of the given type for this factory.
         */
-       virtual XLine* Generate(time_t set_time, long duration, std::string source, std::string reason, std::string xline_specific_mask) = 0;
+       virtual XLine* Generate(time_t set_time, unsigned long duration, const std::string& source, const std::string& reason, const std::string& xline_specific_mask) = 0;
 
        virtual bool AutoApplyToUserList(XLine* x) { return true; }
 
@@ -399,7 +399,7 @@ class CoreExport XLineFactory
        virtual ~XLineFactory() { }
 };
 
-/** XLineManager is a class used to manage glines, klines, elines, zlines and qlines,
+/** XLineManager is a class used to manage G-lines, K-lines, E-lines, Z-lines and Q-lines,
  * or any other line created by a module. It also manages XLineFactory classes which
  * can generate a specialized XLine for use by another module.
  */
@@ -436,7 +436,7 @@ class CoreExport XLineManager
         */
        IdentHostPair IdentSplit(const std::string &ident_and_host);
 
-       /** Checks what users match e:lines and sets their ban exempt flag accordingly.
+       /** Checks what users match E-lines and sets their ban exempt flag accordingly.
         */
        void CheckELines();
 
@@ -476,7 +476,7 @@ class CoreExport XLineManager
        /** Registers an xline factory.
         * An xline factory is a class which when given a particular xline type,
         * will generate a new XLine specialized to that type. For example if you
-        * pass the XLineFactory that handles glines some data it will return a
+        * pass the XLineFactory that handles G-lines some data it will return a
         * pointer to a GLine, polymorphically represented as XLine. This is used where
         * you do not know the full details of the item you wish to create, e.g. in a
         * server protocol module like m_spanningtree, when you receive xlines from other