Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/337#discussion_r159239355
--- 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 --
Doing just one operation for fluent seems inconsistent. `Context`
manipulation isn't (shouldn't) be that common an operation.
See also the quite recent `Context.mergeCopy`.
---
