- More TODO items

- Update comments

git-svn-id: http://htmlpurifier.org/svnroot/htmlpurifier/trunk@1549 48356398-32a2-884e-a903-53898d9a118a

library/HTMLPurifier/AttrDef/CSS/FontFamily.php

@@ -1,9 +1,8 @@
 <?php
 
-// whitelisting allowed fonts would be nice
-
 /**
  * Validates a font family list according to CSS spec
+ * @todo whitelisting allowed fonts would be nice
  */
 class HTMLPurifier_AttrDef_CSS_FontFamily extends HTMLPurifier_AttrDef
 {

library/HTMLPurifier/AttrDef/URI.php

@@ -1,9 +1,5 @@
 <?php
 
-// special case filtering directives 
-
-// disabling directives
-
 /**
  * Validates a URI as defined by RFC 3986.
  * @note Scheme-specific mechanics deferred to HTMLPurifier_URIScheme

library/HTMLPurifier/Bootstrap.php

@@ -1,10 +1,25 @@
 <?php
 
-// constants are slow, but we'll make one exception
+// constants are slow, so we use as few as possible
 if (!defined('HTMLPURIFIER_PREFIX')) {
     define('HTMLPURIFIER_PREFIX', realpath(dirname(__FILE__) . '/..'));
 }
 
+// accomodations for versions earlier than 5.0.2
+// borrowed from PHP_Compat, LGPL licensed, by Aidan Lister <aidan@php.net>
+if (!defined('PHP_EOL')) {
+    switch (strtoupper(substr(PHP_OS, 0, 3))) {
+        case 'WIN':
+            define('PHP_EOL', "\r\n");
+            break;
+        case 'DAR':
+            define('PHP_EOL', "\r");
+            break;
+        default:
+            define('PHP_EOL', "\n");
+    }
+}
+
 /**
  * Bootstrap class that contains meta-functionality for HTML Purifier such as
  * the autoload function.

library/HTMLPurifier/ChildDef.php

@@ -1,10 +1,5 @@
 <?php
 
-// HTMLPurifier_ChildDef and inheritance have three types of output:
-// true = leave nodes as is
-// false = delete parent node and all children
-// array(...) = replace children nodes with these
-
 /**
  * Defines allowed child nodes and validates tokens against it.
  */

library/HTMLPurifier/Config.php

@@ -1,21 +1,5 @@
 <?php
 
-// member variables
-// accomodations for versions earlier than 4.3.10 and 5.0.2
-// borrowed from PHP_Compat, LGPL licensed, by Aidan Lister <aidan@php.net>
-if (!defined('PHP_EOL')) {
-    switch (strtoupper(substr(PHP_OS, 0, 3))) {
-        case 'WIN':
-            define('PHP_EOL', "\r\n");
-            break;
-        case 'DAR':
-            define('PHP_EOL', "\r");
-            break;
-        default:
-            define('PHP_EOL', "\n");
-    }
-}
-
 /**
  * Configuration object that triggers customizable behavior.
  *

library/HTMLPurifier/DoctypeRegistry.php

@@ -1,6 +1,5 @@
 <?php
 
-// Legacy directives for doctype specification
 class HTMLPurifier_DoctypeRegistry
 {
     

library/HTMLPurifier/Generator.php

@@ -1,6 +1,5 @@
 <?php
 
-// extension constraints could be factored into ConfigSchema
 /**
  * Generates HTML from tokens.
  * @todo Refactor interface so that configuration/context is determined

library/HTMLPurifier/HTMLDefinition.php

@@ -1,8 +1,5 @@
 <?php
 
-// this definition and its modules MUST NOT define configuration directives
-// outside of the HTML or Attr namespaces
-
 /**
  * Definition of the purified HTML that describes allowed children,
  * attributes, and many other things.
@@ -22,6 +19,9 @@
  * 
  * @note This class is inspected by Printer_HTMLDefinition; please
  *       update that class if things here change.
+ *
+ * @warning Directives that change this object's structure must be in
+ *          the HTML or Attr namespace!
  */
 class HTMLPurifier_HTMLDefinition extends HTMLPurifier_Definition
 {

library/HTMLPurifier/URIDefinition.php

@@ -1,7 +1,5 @@
 <?php
 
-// informative URI directives
-
 class HTMLPurifier_URIDefinition extends HTMLPurifier_Definition
 {