Skip to content
This repository was archived by the owner on Jan 6, 2022. It is now read-only.

Make MyGridTerminalSystem_ModAPI more intuitive#512

Open
Aeronaut wants to merge 5 commits intoKeenSoftwareHouse:masterfrom
Aeronaut:GridTerminalSystem
Open

Make MyGridTerminalSystem_ModAPI more intuitive#512
Aeronaut wants to merge 5 commits intoKeenSoftwareHouse:masterfrom
Aeronaut:GridTerminalSystem

Conversation

@Aeronaut
Copy link
Copy Markdown

@Aeronaut Aeronaut commented Apr 9, 2016

Added a few Methods to make the GridTerminalSystem more intuitive.

Aeronaut added 5 commits April 9, 2016 13:05
Thanks for your input. I edited the GetBlockGroups a little bit, since it just needs to be converted I simply return a converted List...

Overall I think there should be no Reason for reuse Lists, if it will contain what is needed.
Thanks for your input. I edited the GetBlockGroups a little bit, since it just needs to be converted I simply return a converted List...

Overall I think there should be no Reason for reuse Lists, if it will contain what is needed.
@mze9412
Copy link
Copy Markdown

mze9412 commented Apr 9, 2016

The reason is the following:
You can run a script once per tick. Each tick you call GetBlocks() and get a new list. Instead you could create the list in your script, store it in a member and use it with GetBlocks(myList) each time the block runs. This is not a big thing but can save some memory or garbage collector overhead. Especially if there are many scripts running (i.e. on a DS)

@Aeronaut
Copy link
Copy Markdown
Author

Aeronaut commented Apr 9, 2016

Oky, understood.

@malware-dev
Copy link
Copy Markdown

I'm sorry, but this is highly unlikely to be merged due to the list instantiations.

@Spartan322
Copy link
Copy Markdown

Spartan322 commented Apr 21, 2016

I'll gotta add that I doubt the power of redundancies is gonna get this through.

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.

4 participants