JuliaGraphs / JuliaGraphs/GraphViz.jl
Wrong conversion of String to C String?
Nobody has claimed this yet.
- Dominant language
- Julia
- Stars
- 69
- Forks
- 24
- PR merge metrics
- No merged PRs in 30d
Description
I noticed this line of code in GraphViz.jl package, but later on a String object being used here (note the format="julia:cairo"). It doesn't seem right? The function expects C string, not Julia string? Could it be the reason for https://github.com/JuliaLang/julia/issues/56150?
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by inspecting src/cairo.jl at the linked lines 49 and 82, comparing the argument expected by the called function with the String passed for format="julia:cairo". Reproduce or verify whether this relates to Julia issue 56150 and review the related Julia pull request 51275. Done means the conversion concern is resolved or clearly ruled out with appropriate verification.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- c, julia
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100