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.

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

Add API synopsis titles and cross references #562

Conversation

gmlueck
Copy link
Contributor

@gmlueck gmlueck commented Jun 9, 2024

Add a title to each [source] block that is a synopsis. The title uses
[apidef] if the title is a unique name for a SYCL identifier,
otherwise it uses [apititle].

Also add an extension which creates automatic cross references to the
[apidef] synopses. In order to make a cross reference, use the
Asciidoc role [api]. This is styled like [code] but also creates a
cross reference. For example, [api]#foo# creates a cross reference to
the synopsis whose title is [apidef]#foo#.

Replace some uses of [code] with [api] in the sections defining the
math builtins.

gmlueck added 2 commits June 9, 2024 13:52
Add a title to each `[source]` block that is a synopsis.  The title
uses `[apidef]` if the title is a unique name for a SYCL identifier,
otherwise it uses `[apititle]`.  My plan is to add tooling that creates
automatic cross references for `[apidef]`.
Add an extension which creates automatic links from `[api]#NAME#` to
`[apidef]#NAME#`.  The `[api]` role is styled the same as `[code]`,
which means that we can replace some instances of `[code]` with `[api]`
when that role is used to refer to a SYCL identifier.  By using `[api]`
the identifier will also be a link to the synopsis that defines the
identifier.

Also change some instances of `[code]` to `[api]` in the math builtin
sections to make use of this new ability.
@gmlueck gmlueck added the editorial Some purely editorial problem label Jun 9, 2024
@TApplencourt
Copy link
Contributor

For fun, in ruby, we have the in-famous flip-flop operator; in theory, it makes dealing with scope easier. In practice, it's unreadable.

@gmlueck
Copy link
Contributor Author

gmlueck commented Jun 13, 2024

Merging as editorial.

@gmlueck gmlueck merged commit 61fe3e2 into KhronosGroup:SYCL-2020/master Jun 13, 2024
2 checks passed
@gmlueck gmlueck deleted the gmlueck/synopsis-titles-and-xrefs branch June 13, 2024 20:40
keryell pushed a commit that referenced this pull request Sep 10, 2024
Add API synopsis titles and cross references
gmlueck added a commit that referenced this pull request Nov 7, 2024
Add API synopsis titles and cross references

(cherry picked from commit 61fe3e2)
gmlueck added a commit that referenced this pull request Nov 7, 2024
Add API synopsis titles and cross references

(cherry picked from commit 61fe3e2)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editorial Some purely editorial problem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants