Changeset 42779f8

Timestamp:

Aug 16, 2017, 12:53:41 PM (8 years ago)

Author:

Jason Gross <jasongross9@gmail.com>

Branches:

master

Children:

03fbf66

Parents:

4ead7b3

git-author:

Jason Gross <jgross@mit.edu> (07/12/11 17:40:48)

git-committer:

Jason Gross <jasongross9@gmail.com> (08/16/17 12:53:41)

Message:

Moved logging-specific variables to perl

But only the ones that could be moved losslessly, i.e., no special
validsettings and not enums.

Files:

• perl/lib/BarnOwl/Logging.pm (modified) (1 diff)
• variable.c (modified) (2 diffs)

perl/lib/BarnOwl/Logging.pm

@@ -35 +35 @@
 
 $BarnOwl::Hooks::newMessage->add("BarnOwl::Logging::log");
+$BarnOwl::Hooks::startup->add("BarnOwl::Logging::_register_variables");
+
+sub _register_variables {
+    BarnOwl::new_variable_bool('logging',
+        {
+            default     => 0,
+            summary     => 'turn personal logging on or off',
+            description => "If this is set to on, personal messages are\n"
+                         . "logged in the directory specified\n"
+                         . "by the 'logpath' variable.  The filename in that\n"
+                         . "directory is derived from the sender of the message."
+        });
+
+    BarnOwl::new_variable_bool('classlogging',
+        {
+            default     => 0,
+            summary     => 'turn class logging on or off',
+            description => "If this is set to on, class messages are\n"
+                         . "logged in the directory specified by the\n"
+                         . "'classpath' variable.  The filename in that\n"
+                         . "directory is derived from the class to which\n"
+                         . "the message was sent."
+        });
+
+    BarnOwl::new_variable_string('logfilter',
+        {
+            default     => '',
+            summary     => 'name of a filter controlling which messages to log',
+            description => "If non empty, any messages matching the given filter will be logged.\n"
+                         . "This is a completely separate mechanism from the other logging\n"
+                         . "variables like logging, classlogging, loglogins, loggingdirection,\n"
+                         . "etc.  If you want this variable to control all logging, make sure\n"
+                         . "all other logging variables are in their default state."
+        });
+
+    BarnOwl::new_variable_bool('loglogins',
+        {
+            default     => 0,
+            summary     => 'enable logging of login notifications',
+            description => "When this is enabled, BarnOwl will log login and logout notifications\n"
+                         . "for AIM, zephyr, or other protocols.  If disabled BarnOwl will not print\n"
+                         . "login or logout notifications."
+        });
+}
 
 =head2 sanitize_filename BASE_PATH FILENAME

variable.c

@@ -136 +136 @@
                "load logins from .anyone on startup", "" );
 
-  OWLVAR_BOOL( "logging" /* %OwlVarStub */, 0,
-               "turn personal logging on or off", 
-               "If this is set to on, personal messages are\n"
-               "logged in the directory specified\n"
-               "by the 'logpath' variable.  The filename in that\n"
-               "directory is derived from the sender of the message.\n" );
-
-  OWLVAR_BOOL( "classlogging" /* %OwlVarStub */, 0,
-               "turn class logging on or off",
-               "If this is set to on, class messages are\n"
-               "logged in the directory specified\n"
-               "by the 'classlogpath' variable.\n" 
-               "The filename in that directory is derived from\n"
-               "the name of the class to which the message was sent.\n" );
-
   OWLVAR_ENUM( "loggingdirection" /* %OwlVarStub */, OWL_LOGGING_DIRECTION_BOTH,
                "specifies which kind of messages should be logged",
@@ -185 +170 @@
                "Enable printing of login notifications",
                "When this is enabled, BarnOwl will print login and logout notifications\n"
-               "for AIM, zephyr, or other protocols.  If disabled BarnOwl will not print\n"
-               "login or logout notifications.\n");
-
-  OWLVAR_STRING( "logfilter" /* %OwlVarStub */, "",
-                 "name of a filter controlling which messages to log",
-
-                 "If non empty, any messages matching the given filter will be logged.\n"
-                 "This is a completely separate mechanism from the other logging\n"
-                 "variables like logging, classlogging, loglogins, loggingdirection,\n"
-                 "etc.  If you want this variable to control all logging, make sure\n"
-                 "all other logging variables are in their default state.\n");
-
-  OWLVAR_BOOL( "loglogins" /* %OwlVarStub */, 0,
-               "Enable logging of login notifications",
-               "When this is enabled, BarnOwl will log login and logout notifications\n"
                "for AIM, zephyr, or other protocols.  If disabled BarnOwl will not print\n"
                "login or logout notifications.\n");