Here's the setup.

1) Main Form
2) Options Form
3) Seperate class which handles effecting the main form's status based on the options that were changed.

The idea behind the code was to have 1 property that I could call, rather than writing duplicate code.

For example...

When I change an option in the options form, the main form will have changes that take effect (like the context menu showing the shortcut keys or not, etc.).

Now... the main form also needs to get that option when the initial main form loads (without modifying the options).

It's working fine, but it feels wrong to pass the main form directly to the class...

I wonder if it's more efficient to duplicate the code or continue with this class setup where I pass it the actual main form in addition to any other params?