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

Incomplete XML output when closing Visual Studio #41

Open
clayness opened this issue Dec 4, 2019 · 5 comments
Open

Incomplete XML output when closing Visual Studio #41

clayness opened this issue Dec 4, 2019 · 5 comments
Assignees

Comments

@clayness
Copy link

clayness commented Dec 4, 2019

I recently ran a trace with the Visual Studio plugin, and after I closed everything down, I noticed that the last of my output files did not include the closing </gaze> or </itrace_plugin> tags.

I reproduced it on a different computer using these steps:

  1. Open iTrace Core
  2. Setup the session with an output directory
  3. Run Visual Studio Community 2017 (as administrator)
  4. Open iTrace window
  5. Connect to core
  6. Start session in core
  7. Wiggle mouse around, etc.
  8. Stop session in core
  9. Close Visual Studio

It appears from the code that there is no management of the lifecycle for the plugin/tool window, so the closing tag is only written when (a) the user disconnects from core or (b) a new session is started.

@shbonita shbonita added the bug label Dec 4, 2019
@shbonita
Copy link
Contributor

shbonita commented Dec 4, 2019

@clayness thanks for submitting this issue. The repro steps are very helpful. We will get to assigning this to a developer soon.

@clayness
Copy link
Author

clayness commented Dec 6, 2019

For reference, there is some sample code available on this StackOverflow question that demonstrates how to wire up some of the relevant lifecycle events.

Be warned that the garbage collector will try to collect these COM objects if they are local variables, so you'll likely need to save the DTE object and the relevant events object as members of the package class.

@dtg3 dtg3 removed their assignment Dec 9, 2019
@dtg3
Copy link
Member

dtg3 commented Dec 9, 2019

@cabryant20 Review this issue.
If the issue is not resolved by the Weds meeting, be prepared to discuss your approaches/ideas to resolve the problem at that time.

If there will be no Weds meeting (12/11), plan for the next meeting time (SDML or iTrace whichever is first).

@dtg3 dtg3 added the High label May 14, 2020
@dtg3 dtg3 assigned cabryant20 and unassigned cabryant20 May 14, 2020
@cabryant20
Copy link
Contributor

cabryant20 commented May 19, 2020

Added Issue-41 branch. OnClose event added that checks if the writer/timer are still active and calls the xmlClose function if they are.

@shbonita
Copy link
Contributor

@Joshua2000B is this fixed?

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

5 participants