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

Included, but potentially uninitialized state #15

Open
PragTob opened this issue Jan 12, 2013 · 5 comments
Open

Included, but potentially uninitialized state #15

PragTob opened this issue Jan 12, 2013 · 5 comments

Comments

@PragTob
Copy link
Collaborator

PragTob commented Jan 12, 2013

  • programmers include and execute initialization of trait state explicitly: since we can't reliably/easily detect which methods or parts of them is actually necessary initialization
  • Idea: Warning: if you include state that seems to initialized somewhere, warn the programmer
@PragTob
Copy link
Collaborator Author

PragTob commented Jan 12, 2013

I don't think that it is very feasible right now to check if something is seemingly initialized somewhere - I believe that you should generally be warned if you include state that this might have to be initialized.

@antongulenko
Copy link
Collaborator

So you want to always warn the programmer whenever state is included?

@PragTob
Copy link
Collaborator Author

PragTob commented Jan 12, 2013

Yeah that was kind of my idea for the first run through, it doesn't need to work perfectly and we'd save the work to eventually figure out if some variabel is initialized somewhere.

Of course we could do the obvious 'check if it has an assignment in the initialize' but imo it's not too important for a prototype.

@lauritzthamsen
Copy link
Owner

@PragTob warnings in any case isn't so different from warnings in no case...

@lauritzthamsen
Copy link
Owner

methods to search for initialization code: initialize*

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

3 participants