Skip to content

Fix missing scope-resolution colon in bIsRef template variable#11

Closed
MrKarver wants to merge 1 commit into
DeanRoddey:developfrom
MrKarver:fix-bisref-scope-typo
Closed

Fix missing scope-resolution colon in bIsRef template variable#11
MrKarver wants to merge 1 commit into
DeanRoddey:developfrom
MrKarver:fix-bisref-scope-typo

Conversation

@MrKarver

Copy link
Copy Markdown

Small fix while studying the kernel. In CIDKernel_BaseInlines.hpp, the bIsRef template variable uses a single colon instead of the scope-resolution operator:

template <typename T> constexpr tCIDLib::TBoolean bIsRef = IsRef<T>:bState;
//                                                              ^ should be ::

Every sibling trait variable in the same file uses ::bState (bIsConst, bIsPointer, bIsNumeric, etc.), so this is just a typo — corrected to IsRef<T>::bState.

Thanks for open-sourcing CIDLib and CQC. I'm building on it actively and have a lot of respect for the work. More to come.

@MrKarver

MrKarver commented Jun 11, 2026

Copy link
Copy Markdown
Author

Dean, I mostly opened this as an excuse to say thank you.

I've been building on CIDLib and CQC seriously. Driver work, a native outbound WebSocket client, integrations with UniFi and others. The more time I spend in the code, the more I understand what you meant when you said it could be more than an automation system. It's a remarkable body of work.

If you're still around and open to it, I'd love to contribute more some genuine bug fixes in the WebSocket engine, and new capability I think has value for anyone on the codebase. And if you've stepped away, I completely understand. I just wanted you to know the work is still being used and respected, and that someone's picking up the torch.

Either way: thank you for putting decades into this and then giving it to the world.

-- Kristian

@DeanRoddey

DeanRoddey commented Jun 12, 2026 via email

Copy link
Copy Markdown
Owner

@MrKarver MrKarver closed this by deleting the head repository Jun 21, 2026
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