Skip to content

Conversation

@eth1platinum
Copy link
Collaborator

Allows lookup of properties directly like so:

    ThingSetReadOnlyProperty<float *> *imaxCellsProp = nullptr;
    if (ThingSetRegistry::findById(TS_ID_MODIINMAX, &imaxCellsProp) ...

@garethpotter
Copy link
Collaborator

As discussed, I don't think we can actually merge this, because the cast is not safe: you can look up a ThingSetProperty<float *> but provide a ThingSetProperty<uint8_t *> as the out value and it will happily comply.

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.

4 participants