search

DoomRater / DrizzleScript

an open source LSL diaper script
Other
2 stars 2 forks source link

Code Audit in June #43

Closed DoomRater closed 8 years ago

DoomRater commented 9 years ago

I'm going to be doing a sweep of the code to see if there is anything currently in the core that could be rewritten better, turned into plugins or functions, or the like starting June 1st. During that time features will be frozen until the audit is complete.

Some things absolutely need to be turned into plugins, for example skins for more than just the main diaper itself. Also this will force me to build an API for other scripts to communicate with a menu script. I may peek at OpenCollar's source to see how it achieves this effect.

Some checks and evaluations are redundant and could be better rewritten.

Some functions were implemented by the skin of their teeth and could use rewriting.

DoomRater commented 9 years ago

According to someone who works with GPL 2.0 I am unable to borrow from OpenCollar in order to work with DrizzleScript due to the RPL license. It looks like I'll have to write those functions myself if I want them.

DoomRater commented 9 years ago

Something for consideration: do we need to save the wet level and mess level as a saved setting in the memory core, or can we alter this as we need to? And can we just send what values we want saved to the memory core one by one instead of having to send everything all at once? Being able to send specific values or retrieve specific values would be nice.

DoomRater commented 9 years ago

I'm two days late to beginning the code audit, but my current roadmap will have to be delayed a few days more. I'm still in the middle of things that couldn't have been done at another time, plus I'd like to get the code running in OpenSim as well due to my usage of Mobius Grid.

DoomRater commented 8 years ago

The code audit simply didn't happen as expected. It's time to close this and schedule a different time for a code audit.