Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Alexis Montoison <[email protected]>
  • Loading branch information
gdalle and amontoison authored May 18, 2024
1 parent 27a3ac4 commit 83864e7
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/colpackcoloring.jl
Original file line number Diff line number Diff line change
Expand Up @@ -52,14 +52,14 @@ function ColPackColoring(
verbose::Bool=false,
)
reflen = Vector{Cint}([Cint(0)])
refColPack = Vector{Ptr{Cvoid}}([C_NULL])
refColPack = Ref{Ptr{Cvoid}}(C_NULL)
ret = @ccall libcolpack.build_coloring(
refColPack::Ptr{Cvoid},
reflen::Ptr{Cint},
filename::Cstring,
method.method::Cstring,
order.order::Cstring,
Cint(verbose)::Cint,
verbose::Cint,
)::Cint
if ret == 0
error("ColPack coloring failed.")
Expand Down Expand Up @@ -90,7 +90,7 @@ function ColPackColoring(
push!(csr_mem, vec)
end
nrows = size(M, 2)
reflen = Vector{Cint}([Cint(0)])
reflen = Ref{Cint}(0)
refColPack = Vector{Ptr{Cvoid}}([C_NULL])
ret = @ccall libcolpack.build_coloring_from_csr(
refColPack::Ptr{Cvoid},
Expand All @@ -99,7 +99,7 @@ function ColPackColoring(
nrows::Cint,
method.method::Cstring,
order.order::Cstring,
Cint(verbose)::Cint,
verbose::Cint,
)::Cint
if ret == 0
error("ColPack coloring failed.")
Expand Down

0 comments on commit 83864e7

Please sign in to comment.