Make arguments of register keyword only.

name, nargs and isdeterm are now keyword only.

Author: Sean1708

src/SQLite.jl

@@ -54,7 +54,7 @@ function SQLiteDB(file::AbstractString="";UTF16::Bool=false)
     file = isempty(file) ? file : expanduser(file)
     if @OK sqliteopen(utf(file),handle)
         db = SQLiteDB(utf(file),handle[1])
-        register(db, regexp, 2)
+        register(db, regexp, nargs=2)
         finalizer(db,close)
         return db
     else # error

src/UDF.jl

@@ -61,13 +61,8 @@ end
 macro register(db, func)
     :(register($(esc(db)), $(esc(func))))
 end
-# User-facing method with keyword arguments for registering a function
-# to be used within SQLite
-function register(db::SQLiteDB, func::Function; nargs::Int=-1, isdeterm::Bool=true, name::String=string(func))
-    register(db, func, nargs, isdeterm, name)
-end
 # User-facing method for registering a Julia function to be used within SQLite
-function register(db::SQLiteDB, func::Function, nargs::Int=-1, isdeterm::Bool=true, name::String=string(func))
+function register(db::SQLiteDB, func::Function; nargs::Int=-1, name::AbstractString=string(func), isdeterm::Bool=true)
     @assert nargs <= 127 "use -1 if > 127 arguments are needed"
     # assume any negative number means a varargs function
     nargs < -1 && (nargs = -1)
@@ -86,6 +81,6 @@ function register(db::SQLiteDB, func::Function, nargs::Int=-1, isdeterm::Bool=tr
 end
 
 # annotate types because the MethodError makes more sense that way
-regexp(r::String, s::String) = ismatch(Regex(r), s)
+regexp(r::AbstractString, s::AbstractString) = ismatch(Regex(r), s)
 # macro for preserving the special characters in a string
 macro sr_str(s) s end

test/runtests.jl

@@ -131,8 +131,8 @@ r = query(db, sr"SELECT LastName FROM Employee WHERE BirthDate REGEXP '^\d{4}-08
 @test r.values[1][1] == "Peacock"
 
 triple(x) = x * 3
-@test_throws ErrorException SQLite.register(db, triple, 186)
-SQLite.register(db, triple, 1)
+@test_throws ErrorException SQLite.register(db, triple, nargs=186)
+SQLite.register(db, triple, nargs=1)
 r = query(db, "SELECT triple(Total) FROM Invoice ORDER BY InvoiceId LIMIT 5")
 s = query(db, "SELECT Total FROM Invoice ORDER BY InvoiceId LIMIT 5")
 for (i, j) in zip(r.values[1], s.values[1])