Skip to content
This repository has been archived by the owner on Jul 17, 2019. It is now read-only.

Commit

Permalink
Merge pull request #43 from JuliaData/nl/isnull
Browse files Browse the repository at this point in the history
Use !isnull() rather than != null
  • Loading branch information
quinnj committed Oct 3, 2017
2 parents 0a54be2 + 38b50de commit d1a806c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/DataStreams.jl
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ function Schema(types=(), header=["Column$i" for i = 1:length(types)], rows::(Un
header2 = String[string(x) for x in header]
cols = length(header2)
cols != length(types2) && throw(ArgumentError("length(header): $(length(header2)) must == length(types): $(length(types2))"))
return Schema{rows != null, Tuple{types2...}}(header2, rows, cols, metadata, Dict(n=>i for (i, n) in enumerate(header2)))
return Schema{!isnull(rows), Tuple{types2...}}(header2, rows, cols, metadata, Dict(n=>i for (i, n) in enumerate(header2)))
end
Schema(types, rows::Union{Integer,Null}, metadata::Dict=Dict()) = Schema(types, ["Column$i" for i = 1:length(types)], rows, metadata)

Expand Down Expand Up @@ -694,4 +694,4 @@ end
end # if isdefined(Core, :NamedTuple)
end # module Data

end # module DataStreams
end # module DataStreams

0 comments on commit d1a806c

Please sign in to comment.