[eXist-commits] [eXist-db/exist] 04bf63: [refactor] rename Function.overloaded to variadic

[Patrick R. <no...@gi...>]

  Branch: refs/heads/develop
  Home:   https://github.com/eXist-db/exist
  Commit: 04bf6335d719ad96ec1441e628ce7d21a4868b26
      https://github.com/eXist-db/exist/commit/04bf6335d719ad96ec1441e628ce7d21a4868b26
  Author: Juri Leino <gi...@li...>
  Date:   2023-08-22 (Tue, 22 Aug 2023)

  Changed paths:
    M exist-core/src/main/java/org/exist/xquery/Function.java
    M exist-core/src/main/java/org/exist/xquery/FunctionSignature.java
    M exist-core/src/main/java/org/exist/xquery/functions/fn/FunHigherOrderFun.java
    M exist-core/src/main/java/org/exist/xquery/functions/fn/LoadXQueryModule.java

  Log Message:
  -----------
  [refactor] rename Function.overloaded to variadic

The overloaded flag is set to true for functions like fn:concat, which has an unlimited arity.
The getter function and constructor parameter names were renamed to reflect this.
It is therefore better to call it variadic (see https://en.wikipedia.org/wiki/Variadic_function).
The term is also used in the XQuery specification to describe the nature of such functions.


  Commit: 2281c66c0a74ebbdbf531492a5231078344ad927
      https://github.com/eXist-db/exist/commit/2281c66c0a74ebbdbf531492a5231078344ad927
  Author: Juri Leino <gi...@li...>
  Date:   2023-08-22 (Tue, 22 Aug 2023)

  Changed paths:
    M exist-core/src/main/java/org/exist/xquery/functions/fn/FunNamespaceURI.java
    M exist-core/src/main/java/org/exist/xquery/functions/response/StreamBinary.java
    M exist-core/src/main/java/org/exist/xquery/functions/util/GetFragmentBetween.java

  Log Message:
  -----------
  [fix] remove variadic flag from functions

response:stream-binary and util:get-fragment-between are both
non-variadic.

fn:namespace-uri were flagged as non-variadic, which is the default.

Note: there are variadic functions in the range module. Removing the flag results in failing tests, so it seems they
really need to be variadic.


  Commit: e04765f92c789c4b6d0dbe3dd14818251aa4edc8
      https://github.com/eXist-db/exist/commit/e04765f92c789c4b6d0dbe3dd14818251aa4edc8
  Author: Juri Leino <gi...@li...>
  Date:   2023-08-26 (Sat, 26 Aug 2023)

  Changed paths:
    M extensions/indexes/range/src/test/xquery/range/range.xql

  Log Message:
  -----------
  [test] multi value field lookup


  Commit: 7fa1d00f62441a8f86480dab1bd3ef2b7c384b46
      https://github.com/eXist-db/exist/commit/7fa1d00f62441a8f86480dab1bd3ef2b7c384b46
  Author: Patrick Reinhart <pa...@re...>
  Date:   2023-08-26 (Sat, 26 Aug 2023)

  Changed paths:
    M exist-core/src/main/java/org/exist/xquery/Function.java
    M exist-core/src/main/java/org/exist/xquery/FunctionSignature.java
    M exist-core/src/main/java/org/exist/xquery/functions/fn/FunHigherOrderFun.java
    M exist-core/src/main/java/org/exist/xquery/functions/fn/FunNamespaceURI.java
    M exist-core/src/main/java/org/exist/xquery/functions/fn/LoadXQueryModule.java
    M exist-core/src/main/java/org/exist/xquery/functions/response/StreamBinary.java
    M exist-core/src/main/java/org/exist/xquery/functions/util/GetFragmentBetween.java
    M extensions/indexes/range/src/test/xquery/range/range.xql

  Log Message:
  -----------
  Merge pull request #5021 from line-o/overloaded

Rename Function flag overloaded to variadic


Compare: https://github.com/eXist-db/exist/compare/4fff15af0f6d...7fa1d00f6244