2002/css-validator/org/w3c/css/properties/css1 CssFontFamily.java,1.3,1.4 CssFontFamilyCSS1.java,1.3,1.4 CssFontFamilyCSS2.java,1.3,1.4

• From: Olivier Thereaux via cvs-syncmail <cvsmail@w3.org>
• Date: Wed, 13 Dec 2006 07:09:11 +0000
• To: www-validator-cvs@w3.org
• Message-Id: <E1GuOEp-0005ni-Nh@lionel-hutz.w3.org>

Update of /sources/public/2002/css-validator/org/w3c/css/properties/css1
In directory hutz:/tmp/cvs-serv22268

Modified Files:
	CssFontFamily.java CssFontFamilyCSS1.java 
	CssFontFamilyCSS2.java 
Log Message:
tentatively removing duplicate, obsolete code

Index: CssFontFamilyCSS1.java
===================================================================
RCS file: /sources/public/2002/css-validator/org/w3c/css/properties/css1/CssFontFamilyCSS1.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -d -r1.3 -r1.4
--- CssFontFamilyCSS1.java	14 Sep 2005 15:14:31 -0000	1.3
+++ CssFontFamilyCSS1.java	13 Dec 2006 07:09:09 -0000	1.4
@@ -181,9 +181,12 @@
 	if(check && !expression.end()) {
 	    throw new InvalidParamException("unrecognize", ac);
 	}
-	if (!containsGenericFamily()) {
-	    ac.getFrame().addWarning("no-generic-family", "font-family");
-	}
+	// This looks like obsolete code: (no context, no level, and duplicate 
+	// of a warning handled already in CSS1Style.java
+	// olivier 2006-12-13
+	//if (!containsGenericFamily()) {
+	//    ac.getFrame().addWarning("no-generic-family", "font-family");
+	//}
 
     }
 

Index: CssFontFamilyCSS2.java
===================================================================
RCS file: /sources/public/2002/css-validator/org/w3c/css/properties/css1/CssFontFamilyCSS2.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -d -r1.3 -r1.4
--- CssFontFamilyCSS2.java	14 Sep 2005 15:14:31 -0000	1.3
+++ CssFontFamilyCSS2.java	13 Dec 2006 07:09:09 -0000	1.4
@@ -188,9 +188,12 @@
 	if(check && !expression.end()) {
 	    throw new InvalidParamException("unrecognize", ac);
 	}
-	if (!containsGenericFamily()) {
-	    ac.getFrame().addWarning("no-generic-family", "font-family");
-	}
+	// This looks like obsolete code: (no context, no level, and duplicate 
+	// of a warning handled already in CSS1Style.java
+	// olivier 2006-12-13
+	//if (!containsGenericFamily()) {
+	//    ac.getFrame().addWarning("no-generic-family", "font-family");
+	//}
     }
 
     public CssFontFamilyCSS2(ApplContext ac, CssExpression expression)

Index: CssFontFamily.java
===================================================================
RCS file: /sources/public/2002/css-validator/org/w3c/css/properties/css1/CssFontFamily.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -d -r1.3 -r1.4
--- CssFontFamily.java	14 Sep 2005 15:14:31 -0000	1.3
+++ CssFontFamily.java	13 Dec 2006 07:09:09 -0000	1.4
@@ -206,9 +206,12 @@
 	if(check && !expression.end()) {
 	    throw new InvalidParamException("unrecognize", ac);
 	}
-	if (!containsGenericFamily()) {
-	    ac.getFrame().addWarning("no-generic-family", "font-family");
-	}
+	// This looks like obsolete code: (no context, no level, and duplicate 
+	// of a warning handled already in CSS1Style.java
+	// olivier 2006-12-13
+	//if (!containsGenericFamily()) {
+	//    ac.getFrame().addWarning("no-generic-family", "font-family");
+	//}
     }
 
     public CssFontFamily(ApplContext ac, CssExpression expression)

Received on Wednesday, 13 December 2006 07:09:20 UTC