[GitHub] jena pull request #337: JENA-1391: Dataset dyadic views, Model and Dataset S...

[afs]

Github user afs commented on a diff in the pull request:

    https://github.com/apache/jena/pull/337#discussion_r159527518
  
    --- Diff: jena-arq/src/main/java/org/apache/jena/sparql/util/Context.java 
---
    @@ -199,13 +193,12 @@ public long getLong(Symbol symbol, long defaultValue) 
{
             }
         }
         
    -    public void putAll(Context other) {
    +    public Context putAll(Context other) {
             if ( readonly )
    --- End diff --
    
    Doesn't seem much point IMO.  Changing the context, let alone changing 
several setting, is not a common thing to do.


---