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

Finer-grained logging statement DOI #65

Open
khatchad opened this issue Jul 23, 2018 · 2 comments
Open

Finer-grained logging statement DOI #65

khatchad opened this issue Jul 23, 2018 · 2 comments
Assignees
Milestone

Comments

@khatchad
Copy link
Member

khatchad commented Jul 23, 2018

The problem is that logging statements themselves are not associated with a DOI value in the classic Mylyn approach. The closest we have gotten thus far is the enclosing IMethod. However, we could consider things like blocks or even adjacent statements (either above or below the logging statement). Another thing could be ILocalVariables, which may may a DOI value association. Variables could be inside blocks, adjacent statements, or even the statement itself (by referencing them).

To summarize, the options can be as follows:
Use the DOI value of:

  1. ILocalVariables inside the logging statement. Would be like 0th adjacent statement.
  2. ILocalVariables adjacent to the logging statement. These could be variables included in say the above or below statement or statement(s), e.g., 3. 1-3 adjacent statements.
  3. ILocalVariables inside the logging statement's enclosing block.

When we say "inside," we mean either declared or referenced.

The default can be the enclosing method.

┆Issue is synchronized with this Asana task

@khatchad
Copy link
Member Author

We can see which options are the best during our evaluation.

@khatchad khatchad added this to the Future milestone Jul 23, 2018
@khatchad
Copy link
Member Author

#39 (comment) seems related.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants