Skip to content

The color named Chart_Reuse -> Chartreuse#475

Open
pat-rogers wants to merge 2 commits into
AdaCore:masterfrom
pat-rogers:chart_reuse
Open

The color named Chart_Reuse -> Chartreuse#475
pat-rogers wants to merge 2 commits into
AdaCore:masterfrom
pat-rogers:chart_reuse

Conversation

@pat-rogers
Copy link
Copy Markdown
Contributor

I was tempted to leave it as-is... :-)

@jklmnn
Copy link
Copy Markdown
Member

jklmnn commented May 11, 2026

What is the intention for this change (it may be a language thing I'm missing here)? The problem I see is that it effectively changes the API of HAL.Bitmap. So every alire crate using this interface needs to be updated if it happens to use that constant.

It would also warrant a new version number for the alire crate.

@pat-rogers
Copy link
Copy Markdown
Contributor Author

pat-rogers commented May 11, 2026 via email

@jklmnn
Copy link
Copy Markdown
Member

jklmnn commented May 12, 2026

"Chart_Reuse" is a misspelling of the color "Chartreuse". I assume it
was a (natural) language thing initially because it is a French word.
Maybe the effect of a spelling corrector? I didn't look to see who wrote
that line though.

Oh, my head thought it was about reusing a chart lol.

Your call. We can close the PR if you think it isn't worth it. I don't use that constant myself.

Can you add the correct version but keep the incorrect one with a comment that we leave it for backwards compatibility?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants