Skip to content
This repository was archived by the owner on Aug 25, 2025. It is now read-only.
This repository was archived by the owner on Aug 25, 2025. It is now read-only.

Missing Metro::reset() in the example code #3

@wei-peng

Description

@wei-peng

The library and example code is very brief and clear. But I think the Metro::reset() should be called once at the begaining of every example. Maybe in setup().

IMHO, the consequence of not calling Metro::reset() is not severe as the code repeats the HIGH/LOW sequence anyway. But not initilizing previous_millis by calling Metro::reset() causes the first round of the loop being excuted immediately rather than waiting for 250ms.

Or the constructor function could initialize previous_millis by calling Metro::reset().

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions