Replace tick() with os.clock()

RyanLua / Satchel

A modern alternative to Roblox's default backpack. Satchel aims to be more customizable and easier to use than the default backpack while still having a "vanilla" feel.

http://satchel.ryanluu.dev

Mozilla Public License 2.0

35 stars 4 forks source link

Replace tick() with os.clock() #47

Closed Katrist closed 9 months ago

Katrist commented 10 months ago

Description

Improved code by removing deprecated objects

Changes Made

Replaced tick() with os.clock()

Checklist

[X] I have tested these changes thoroughly.
[X] I have reviewed my code for any potential errors or issues.
[X] I have followed the code style guidelines for this project.

Additional Notes

Talks about why tick() is deprecated: https://devforum.roblox.com/t/luau-recap-june-2020/632346 In my testing tick() is also less performant, taking more time to get the current time.

Code of Conduct

By submitting this issue, I agree to follow the Code of Conduct.

Katrist commented 10 months ago

On a side note, could you tell me how to upload files in a commit that fits the format of this codebase? I would like to implement a Signal library to replace BindableEvents for performance reasons, but I'm not sure how I would add the Signal module.

Sources: https://devforum.roblox.com/t/lua-signal-class-comparison-optimal-goodsignal-class/1387063

RyanLua commented 10 months ago

On a side note, could you tell me how to upload files in a commit that fits the format of this codebase? I would like to implement a Signal library to replace BindableEvents for performance reasons, but I'm not sure how I would add the Signal module.

Sources: https://devforum.roblox.com/t/lua-signal-class-comparison-optimal-goodsignal-class/1387063

You would add a .lua file for a module script under src. Files ending in .client.lua are local scripts, .server.lua are normal scripts, and .lua are module scripts.

Additionally, you could just add this to src: https://gist.github.com/stravant/b75a322e0919d60dde8a0316d1f09d2f