Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

Make core Ticker abstract and public #116

Merged
merged 7 commits into from Apr 25, 2016
Merged

Make core Ticker abstract and public #116

merged 7 commits into from Apr 25, 2016

Conversation

hartez
Copy link
Contributor

@hartez hartez commented Apr 20, 2016

Description of Change

Make the core Ticker abstract and public so it can be inherited by platform
implementations; remove now-unused parts of original Ticker; add CreateTicker
to IPlatformServices; remove unused CreateTimer methods from IPlatformServices

No tests; this is mostly just removing old unused code.

API Changes

None

Behavioral Changes

None

PR Checklist

  • Has tests (if omitted, state reason in description)
  • Rebased on top of master at time of PR
  • Changes adhere to coding standard
  • Consolidate commits as makes sense

Make the core Ticker abstract and public so it can be inherited by platform
implementations; remove now-unused parts of original Ticker; add CreateTicker
to IPlatformServices; remove unused CreateTimer methods from IPlatformServices
@hartez hartez changed the title Make core Ticker abstract and public Make core Ticker abstract and public [DO NOT MERGE] Apr 20, 2016
@hartez hartez changed the title Make core Ticker abstract and public [DO NOT MERGE] Make core Ticker abstract and public Apr 20, 2016
//ITimer CreateTimer(Action<object> callback, object state, int dueTime, int period);
//ITimer CreateTimer(Action<object> callback, object state, long dueTime, long period);
//ITimer CreateTimer(Action<object> callback, object state, TimeSpan dueTime, TimeSpan period);
//ITimer CreateTimer(Action<object> callback, object state, uint dueTime, uint period);

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not just delete it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whoops.

@kingces95
Copy link
Contributor

👍

@rmarinho rmarinho merged commit 843bc47 into master Apr 25, 2016
@rmarinho rmarinho deleted the ivt-ticker branch April 25, 2016 19:57
@samhouts samhouts added this to the 2.3.0 milestone Jun 27, 2018
mattleibow pushed a commit that referenced this pull request Jan 28, 2021
* Add initial compass API with samples

* Update Compass to new API and add CompassData return types.

* Add xunit tests

* Add compass tests and documentation

* Made adjustments based on feedback to cleanup code.

* Fix samples! Rename everywhere seemed to have failed :(

* Fix build

* Fix build again.

* Tweaked the sample VM a bit
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants