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 2014/05/17 14:23:09 UTC
svn commit: r1595452 - in /jena/trunk/jena-arq/src/main/java:
arq/cmdline/ModSymbol.java com/hp/hpl/jena/sparql/util/MappingRegistry.java
Author: andy
Date: Sat May 17 12:23:09 2014
New Revision: 1595452
URL: http://svn.apache.org/r1595452
Log:
Remove unused non-feature.
Modified:
jena/trunk/jena-arq/src/main/java/arq/cmdline/ModSymbol.java
jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/util/MappingRegistry.java
Modified: jena/trunk/jena-arq/src/main/java/arq/cmdline/ModSymbol.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/arq/cmdline/ModSymbol.java?rev=1595452&r1=1595451&r2=1595452&view=diff
==============================================================================
--- jena/trunk/jena-arq/src/main/java/arq/cmdline/ModSymbol.java (original)
+++ jena/trunk/jena-arq/src/main/java/arq/cmdline/ModSymbol.java Sat May 17 12:23:09 2014
@@ -32,11 +32,8 @@ public class ModSymbol implements ArgMod
{
protected final ArgDecl setDecl = new ArgDecl(ArgDecl.HasValue, "set", "define", "defn", "def") ;
Context context = new Context() ;
- private String namespace ;
- public ModSymbol() { this(ARQ.arqSymbolPrefix) ; }
-
- public ModSymbol(String namespace) { this.namespace = namespace ; }
+ public ModSymbol() { }
@Override
public void registerWith(CmdGeneral cmdLine)
Modified: jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/util/MappingRegistry.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/util/MappingRegistry.java?rev=1595452&r1=1595451&r2=1595452&view=diff
==============================================================================
--- jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/util/MappingRegistry.java (original)
+++ jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/util/MappingRegistry.java Sat May 17 12:23:09 2014
@@ -23,7 +23,7 @@ import org.apache.jena.riot.system.Prefi
/** Assist with naming symbols by URI.
* This class keeps a prefix mapping so that subsystems can register their
- * prefix and
+ * prefix and keep code shorter.
*/
public class MappingRegistry
{
Re: svn commit: r1595452 - in /jena/trunk/jena-arq/src/main/java:
arq/cmdline/ModSymbol.java com/hp/hpl/jena/sparql/util/MappingRegistry.java
Posted by Andy Seaborne <an...@apache.org>.
On 19/05/14 11:59, Rob Vesse wrote:
> Looks like the unused non-feature is used in SDB, getting build failures
> from SDB with this commit
Bother. Sorry about that. Fixed in svn.
Andy
>
> Rob
>
> On 17/05/2014 13:23, "andy@apache.org" <an...@apache.org> wrote:
>
>> Author: andy
>> Date: Sat May 17 12:23:09 2014
>> New Revision: 1595452
>>
>> URL: http://svn.apache.org/r1595452
>> Log:
>> Remove unused non-feature.
>>
>> Modified:
>> jena/trunk/jena-arq/src/main/java/arq/cmdline/ModSymbol.java
>>
>> jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/util/MappingRegis
>> try.java
>>
>> Modified: jena/trunk/jena-arq/src/main/java/arq/cmdline/ModSymbol.java
>> URL:
>> http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/arq/cmdline
>> /ModSymbol.java?rev=1595452&r1=1595451&r2=1595452&view=diff
>> ==========================================================================
>> ====
>> --- jena/trunk/jena-arq/src/main/java/arq/cmdline/ModSymbol.java
>> (original)
>> +++ jena/trunk/jena-arq/src/main/java/arq/cmdline/ModSymbol.java Sat May
>> 17 12:23:09 2014
>> @@ -32,11 +32,8 @@ public class ModSymbol implements ArgMod
>> {
>> protected final ArgDecl setDecl = new ArgDecl(ArgDecl.HasValue,
>> "set", "define", "defn", "def") ;
>> Context context = new Context() ;
>> - private String namespace ;
>>
>> - public ModSymbol() { this(ARQ.arqSymbolPrefix) ; }
>> -
>> - public ModSymbol(String namespace) { this.namespace = namespace ; }
>> + public ModSymbol() { }
>>
>> @Override
>> public void registerWith(CmdGeneral cmdLine)
>>
>> Modified:
>> jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/util/MappingRegis
>> try.java
>> URL:
>> http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/com/hp/hpl/
>> jena/sparql/util/MappingRegistry.java?rev=1595452&r1=1595451&r2=1595452&vi
>> ew=diff
>> ==========================================================================
>> ====
>> ---
>> jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/util/MappingRegis
>> try.java (original)
>> +++
>> jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/util/MappingRegis
>> try.java Sat May 17 12:23:09 2014
>> @@ -23,7 +23,7 @@ import org.apache.jena.riot.system.Prefi
>>
>> /** Assist with naming symbols by URI.
>> * This class keeps a prefix mapping so that subsystems can register
>> their
>> - * prefix and
>> + * prefix and keep code shorter.
>> */
>> public class MappingRegistry
>> {
>>
>>
>
>
>
>
Re: svn commit: r1595452 - in /jena/trunk/jena-arq/src/main/java:
arq/cmdline/ModSymbol.java com/hp/hpl/jena/sparql/util/MappingRegistry.java
Posted by Rob Vesse <rv...@dotnetrdf.org>.
Looks like the unused non-feature is used in SDB, getting build failures
from SDB with this commit
Rob
On 17/05/2014 13:23, "andy@apache.org" <an...@apache.org> wrote:
>Author: andy
>Date: Sat May 17 12:23:09 2014
>New Revision: 1595452
>
>URL: http://svn.apache.org/r1595452
>Log:
>Remove unused non-feature.
>
>Modified:
> jena/trunk/jena-arq/src/main/java/arq/cmdline/ModSymbol.java
>
>jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/util/MappingRegis
>try.java
>
>Modified: jena/trunk/jena-arq/src/main/java/arq/cmdline/ModSymbol.java
>URL:
>http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/arq/cmdline
>/ModSymbol.java?rev=1595452&r1=1595451&r2=1595452&view=diff
>==========================================================================
>====
>--- jena/trunk/jena-arq/src/main/java/arq/cmdline/ModSymbol.java
>(original)
>+++ jena/trunk/jena-arq/src/main/java/arq/cmdline/ModSymbol.java Sat May
>17 12:23:09 2014
>@@ -32,11 +32,8 @@ public class ModSymbol implements ArgMod
> {
> protected final ArgDecl setDecl = new ArgDecl(ArgDecl.HasValue,
>"set", "define", "defn", "def") ;
> Context context = new Context() ;
>- private String namespace ;
>
>- public ModSymbol() { this(ARQ.arqSymbolPrefix) ; }
>-
>- public ModSymbol(String namespace) { this.namespace = namespace ; }
>+ public ModSymbol() { }
>
> @Override
> public void registerWith(CmdGeneral cmdLine)
>
>Modified:
>jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/util/MappingRegis
>try.java
>URL:
>http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/com/hp/hpl/
>jena/sparql/util/MappingRegistry.java?rev=1595452&r1=1595451&r2=1595452&vi
>ew=diff
>==========================================================================
>====
>---
>jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/util/MappingRegis
>try.java (original)
>+++
>jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/util/MappingRegis
>try.java Sat May 17 12:23:09 2014
>@@ -23,7 +23,7 @@ import org.apache.jena.riot.system.Prefi
>
> /** Assist with naming symbols by URI.
> * This class keeps a prefix mapping so that subsystems can register
>their
>- * prefix and
>+ * prefix and keep code shorter.
> */
> public class MappingRegistry
> {
>
>