search

lep / jassdoc

Document the WarCraft 3 API
52 stars 20 forks source link

Add more info for timer functions #37

Closed Prometheus3375 closed 3 years ago

Prometheus3375 commented 3 years ago

Conclusions are from https://www.hiveworkshop.com/threads/issues-with-timer-functions.309433/

lep commented 3 years ago

Big thanks first of all. Can you add the same doc to GetElapsedTimer being null if it was destroyed while still running? Otherwise just tell me when you're happy with the changes and i will merge it :)

On 10.10.21 15:58, Prometheus3375 wrote:

Conclusions are from https://www.hiveworkshop.com/threads/issues-with-timer-functions.309433/ https://www.hiveworkshop.com/threads/issues-with-timer-functions.309433/

    You can view, comment on, or merge this pull request online at:

https://github.com/lep/jassdoc/pull/37 https://github.com/lep/jassdoc/pull/37

    Commit Summary

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/lep/jassdoc/pull/37, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAA3SWOWCTCZ2QQOZGO6A2LUGGLXDANCNFSM5FWQAWIA. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Prometheus3375 commented 3 years ago

I added. If such note is OK, feel free to merge changes.