Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Null Executor memory reductions #2179

Closed
bdbaddog opened this issue Jan 2, 2018 · 0 comments
Closed

Null Executor memory reductions #2179

bdbaddog opened this issue Jan 2, 2018 · 0 comments

Comments

@bdbaddog
Copy link
Contributor

bdbaddog commented Jan 2, 2018

This issue was originally created at: 2008-08-16 08:23:41.
This issue was reported by: pankrat.
pankrat said at 2008-08-16 08:23:41

The attached patch reduces the memory footprint of Executor.Null by making it a
true stub and not inherit from Executor anymore. A Null executor object is
morphed to a real Executor object if an action is added through
'add_pre_action', 'add_post_action', 'set_action_list'.

Moreover, the singleton pattern is applied for the NullEnvironment object.

This way, Null executor objects can be cut down to 240 Byte (~25% of its
original size). Depending on the project, this can save about 3-4% of memory.

pankrat said at 2008-08-16 08:24:58

Created an attachment (id=474)
Refactored Executor.Null which not inherits from Executor

gregnoel said at 2008-08-19 12:37:37

Bug party triage.

gregnoel said at 2008-09-09 16:05:45

Bug party triage. Go for it.

pankrat said at 2008-09-12 11:18:04

Patch applied in revision 3390.

pankrat attached nullexecutor_stub.patch at 2008-08-16 08:24:58.

Refactored Executor.Null which not inherits from Executor

@bdbaddog bdbaddog added this to the 1.1 milestone Jan 2, 2018
@bdbaddog bdbaddog closed this as completed Jan 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant