Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Provide better API for libclang's command line arguments #83

Open
edsko opened this issue Aug 8, 2024 · 2 comments
Open

Provide better API for libclang's command line arguments #83

edsko opened this issue Aug 8, 2024 · 2 comments
Assignees
@phadej @edsko
Milestone
1: `Storable` ins...

Comments

@edsko
Copy link
Collaborator

edsko commented Aug 8, 2024 •
edited
Loading

We currently configure libclang by passing a raw set of command line arguments (of type [String]). We should have a proper Haskell datatype that lists the libclang options we are interested in (and then internally translate this to [String]).

This is part of #10 and #71.
@edsko edsko added this to the 1: `Storable` instances milestone Aug 8, 2024
phadej added a commit that referenced this issue Aug 19, 2024
@phadej
Issue #83: Better API for libclang command line arguments
3669c5b
@phadej phadej mentioned this issue Aug 19, 2024
Closed
@phadej phadej mentioned this issue Sep 1, 2024
Merged
@edsko
Copy link
Collaborator Author

edsko commented Sep 14, 2024

Perhaps some of the options we should support are the ones mentioned in the docs of 'CXDiagnosticDisplayOptions' (added in #176). However, this is only useful if we don't write our own renderer for libclang options (#174), and instead use diagnosticFormatted/clang_formatDiagnostic.

@edsko
Copy link
Collaborator Author

edsko commented Sep 25, 2024

I'm going to define a record with some fields as a starting point for this.

@edsko edsko self-assigned this Sep 25, 2024
edsko added a commit that referenced this issue Sep 25, 2024
@edsko
Use C23 for the golden tests
d018b56 
This takes a first step towards #83 by introducing a `ClangArgs` record type.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@phadej phadej

@edsko edsko

Labels
None yet
Projects
None yet
Milestone
1: `Storable` instances
Development

No branches or pull requests
2 participants
@phadej @edsko