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

Tables not containing table header panic #261

Closed
kokoISnoTarget opened this issue Mar 9, 2024 · 0 comments · Fixed by #279
Closed

Tables not containing table header panic #261

kokoISnoTarget opened this issue Mar 9, 2024 · 0 comments · Fixed by #279
Labels
A-table Area: Dealing with markdown tables C-bug Category: Something isn't working C-crash-report Category: bugs submitted from inlyne's crash reporting

Comments

@kokoISnoTarget
Copy link
Contributor

When loading a file containing a table without a table header, the renderer will panic here: https://github.com/Inlyne-Project/inlyne/blob/15e7d498c6c1c952b05e66714def307e1a45e5d7/src/renderer.rs#L402C1-L402C75.

<table><th></th></table> // works
<table></table> // panics

Crash report produced by inlyne:
https://gist.github.com/kokoISnoTarget/6744e4501e67877f6c8c3a3be98c6e7c

@CosmicHorrorDev CosmicHorrorDev added C-bug Category: Something isn't working A-table Area: Dealing with markdown tables labels Mar 10, 2024
@CosmicHorrorDev CosmicHorrorDev added the C-crash-report Category: bugs submitted from inlyne's crash reporting label Mar 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-table Area: Dealing with markdown tables C-bug Category: Something isn't working C-crash-report Category: bugs submitted from inlyne's crash reporting
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants