You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jena.apache.org by an...@apache.org on 2013/02/10 17:16:17 UTC

svn commit: r1444560 - in /jena/trunk/jena-core/src: main/java/com/hp/hpl/jena/rdf/model/ main/java/com/hp/hpl/jena/rdf/model/impl/ main/java/com/hp/hpl/jena/reasoner/ main/java/com/hp/hpl/jena/util/ test/java/com/hp/hpl/jena/rdf/model/test/ test/java/...

Author: andy
Date: Sun Feb 10 16:16:16 2013
New Revision: 1444560

URL: http://svn.apache.org/r1444560
Log:
Tidy up - remove (very!) old commentry.

Modified:
    jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/rdf/model/AnonId.java
    jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/rdf/model/impl/ModelCom.java
    jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/reasoner/BaseInfGraph.java
    jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/util/PrintUtil.java
    jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/rdf/model/test/TestContainers.java
    jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/rdf/model/test/TestSelectorUse.java
    jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/rdf/model/test/TestStatements.java
    jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/test/JenaTestBase.java
    jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/xmloutput/TestPackage.java

Modified: jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/rdf/model/AnonId.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/rdf/model/AnonId.java?rev=1444560&r1=1444559&r2=1444560&view=diff
==============================================================================
--- jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/rdf/model/AnonId.java (original)
+++ jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/rdf/model/AnonId.java Sun Feb 10 16:16:16 2013
@@ -60,8 +60,7 @@ public class AnonId extends java.lang.Ob
     /** 
         Creates new AnonId. Normally this id is guaranteed to be unique on this 
         machine: it is time-dependant. However, sometimes [incorrect] code is
-        sensitive to bnode ordering and produces bizarre bugs (both Dave and
-        Chris have been bitten by this, as have some users, I think). Hence the
+        sensitive to bnode ordering and produces bizarre bugs. Hence the
         disableBNodeUIDGeneration flag, which allows bnode IDs to be predictable.
     */
     public AnonId() {

Modified: jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/rdf/model/impl/ModelCom.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/rdf/model/impl/ModelCom.java?rev=1444560&r1=1444559&r2=1444560&view=diff
==============================================================================
--- jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/rdf/model/impl/ModelCom.java (original)
+++ jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/rdf/model/impl/ModelCom.java Sun Feb 10 16:16:16 2013
@@ -45,8 +45,6 @@ import com.hp.hpl.jena.vocabulary.RDF ;
  * <P>This class implements common methods, mainly convenience methods, for
  *    model implementations.  It is intended use is as a base class from which
  *    model implemenations can be derived.</P>
- *
- * hacked by Jeremy, tweaked by Chris (May 2002 - October 2002)
  */
 
 public class ModelCom extends EnhGraph

Modified: jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/reasoner/BaseInfGraph.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/reasoner/BaseInfGraph.java?rev=1444560&r1=1444559&r2=1444560&view=diff
==============================================================================
--- jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/reasoner/BaseInfGraph.java (original)
+++ jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/reasoner/BaseInfGraph.java Sun Feb 10 16:16:16 2013
@@ -365,8 +365,6 @@ public abstract class BaseInfGraph exten
      * was implemented as sameValueAs rather than equals. If it turns out that
      * the filter is needed, it can be commented back in, AND a corresponding
      * filter added to find(Node x 3) -- and test cases, of course.
-     *
-     * <p>[Chris, after discussion with Dave]
      */
     @Override
     public ExtendedIterator<Triple> graphBaseFind(TripleMatch m) {

Modified: jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/util/PrintUtil.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/util/PrintUtil.java?rev=1444560&r1=1444559&r2=1444560&view=diff
==============================================================================
--- jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/util/PrintUtil.java (original)
+++ jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/util/PrintUtil.java Sun Feb 10 16:16:16 2013
@@ -31,9 +31,6 @@ import com.hp.hpl.jena.shared.PrefixMapp
  * A collection of small utilites for pretty printing nodes, triples
  * and associated things. The core functionality here is a static
  * prefix map which is preloaded with known prefixes.
- * 
- * <p>updated by Chris March 2004 to use a PrefixMapping rather than the
- * specialised tables.
  */
 public class PrintUtil {
     

Modified: jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/rdf/model/test/TestContainers.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/rdf/model/test/TestContainers.java?rev=1444560&r1=1444559&r2=1444560&view=diff
==============================================================================
--- jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/rdf/model/test/TestContainers.java (original)
+++ jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/rdf/model/test/TestContainers.java Sun Feb 10 16:16:16 2013
@@ -37,9 +37,6 @@ public class TestContainers extends Abst
 		super(modelFactory, name);
 	}
 
-	/**
-	 * Contributed by Damian, turned into a test case by Chris.
-	 */
 	public void testCanAsContainer()
 	{
 		final String seqUri = "http://example.com/#seq";

Modified: jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/rdf/model/test/TestSelectorUse.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/rdf/model/test/TestSelectorUse.java?rev=1444560&r1=1444559&r2=1444560&view=diff
==============================================================================
--- jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/rdf/model/test/TestSelectorUse.java (original)
+++ jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/rdf/model/test/TestSelectorUse.java Sun Feb 10 16:16:16 2013
@@ -166,7 +166,6 @@ public class TestSelectorUse extends Abs
 		/*
 		 * the _null_ argument to LiteralImpl was preserved only for backward
 		 * compatability. It was be logged and has now become an exception.
-		 * (Brian and Chris had a discussion about this and agreed).
 		 */
 		try
 		{

Modified: jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/rdf/model/test/TestStatements.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/rdf/model/test/TestStatements.java?rev=1444560&r1=1444559&r2=1444560&view=diff
==============================================================================
--- jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/rdf/model/test/TestStatements.java (original)
+++ jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/rdf/model/test/TestStatements.java Sun Feb 10 16:16:16 2013
@@ -130,8 +130,7 @@ public class TestStatements extends Abst
 	}
 
 	/**
-	 * this case came up when Chris was sorting out ReifedStatement and
-	 * had mishacked Model.createStatement. A resource created in one
+	 * A resource created in one
 	 * model and incorporated into a statement asserted constructed by a
 	 * different model should test equal to the resource extracted from that
 	 * statement, even if it's a bnode.

Modified: jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/test/JenaTestBase.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/test/JenaTestBase.java?rev=1444560&r1=1444559&r2=1444560&view=diff
==============================================================================
--- jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/test/JenaTestBase.java (original)
+++ jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/test/JenaTestBase.java Sun Feb 10 16:16:16 2013
@@ -27,8 +27,7 @@ import com.hp.hpl.jena.util.iterator.*;
 
 /**
     A basis for Jena test cases which provides assertFalse and assertDiffer.
-    Often the logic of the names is clearer than using a negation (well, Chris
-    thinks so anyway).
+    Often the logic of the names is clearer than using a negation.
 */
 public class JenaTestBase extends TestCase
     {

Modified: jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/xmloutput/TestPackage.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/xmloutput/TestPackage.java?rev=1444560&r1=1444559&r2=1444560&view=diff
==============================================================================
--- jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/xmloutput/TestPackage.java (original)
+++ jena/trunk/jena-core/src/test/java/com/hp/hpl/jena/xmloutput/TestPackage.java Sun Feb 10 16:16:16 2013
@@ -62,8 +62,7 @@ public class TestPackage extends TestCas
     
     /**
          Added as a place to put the test(s) which ensure that thrown URI exceptions
-         carry the bad URI with them. I (Chris) would embed them in the other tests,
-         but I can't work out how to do so ...
+         carry the bad URI with them.
     */
     public static class TestURIExceptions extends TestCase
         {