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

TAStudio: Selecting a lot of frames + Copying throws an error. #597

Closed
Niamek opened this issue Mar 13, 2016 · 8 comments
Closed

TAStudio: Selecting a lot of frames + Copying throws an error. #597

Niamek opened this issue Mar 13, 2016 · 8 comments
Assignees

Comments

@Niamek
Copy link

Niamek commented Mar 13, 2016

-Load a game(I tried with a GB and SNES game.
-Open TAStudio
-Let it play for a long time(about 10 000 frames)
-Select the frame 00000
-with shift, select the last frame of the movie(so that 0 to that frame is all selected
-Wait a little bit
-Copy all that.

It should throw this exception:

************** Exception Text **************
System.NullReferenceException: Object reference not set to an instance of an object.
at BizHawk.Client.Common.Bk2LogEntryGenerator.CreateLogEntry(Boolean createEmpty)
at BizHawk.Client.Common.Bk2LogEntryGenerator.GenerateLogEntry()
at BizHawk.Client.EmuHawk.TAStudio.CopyMenuItem_Click(Object sender, EventArgs e)
at System.Windows.Forms.ToolStripItem.RaiseEvent(Object key, EventArgs e)
at System.Windows.Forms.ToolStripMenuItem.OnClick(EventArgs e)
at System.Windows.Forms.ToolStripItem.HandleClick(EventArgs e)
at System.Windows.Forms.ToolStripItem.HandleMouseUp(MouseEventArgs e)
at System.Windows.Forms.ToolStripItem.FireEventInteractive(EventArgs e, ToolStripItemEventType met)
at System.Windows.Forms.ToolStripItem.FireEvent(EventArgs e, ToolStripItemEventType met)
at System.Windows.Forms.ToolStrip.OnMouseUp(MouseEventArgs mea)
at System.Windows.Forms.ToolStripDropDown.OnMouseUp(MouseEventArgs mea)
at System.Windows.Forms.Control.WmMouseUp(Message& m, MouseButtons button, Int32 clicks)
at System.Windows.Forms.Control.WndProc(Message& m)
at System.Windows.Forms.ScrollableControl.WndProc(Message& m)
at System.Windows.Forms.ToolStrip.WndProc(Message& m)
at System.Windows.Forms.ToolStripDropDown.WndProc(Message& m)
at System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m)
at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

I just came across this since I had some movies that seemed to be bugged, so I had the idea to copy only the whole movie from TAStudio(probably not a good idea). Curiously, it's the same message than one of my other bug reports(#580), but with a different length... Could be related?

EDIT: Not only copyinh throws the error, but cutting too.

@vadosnaprimer
Copy link
Contributor

Not the same at all, look at all the functions that lead to this exception in both cases.

@vadosnaprimer vadosnaprimer self-assigned this Mar 24, 2016
@vadosnaprimer
Copy link
Contributor

I sped up selection of multiple rows (to be almost instant), but I can't get this crash when I copy, only when I paste.

vadosnaprimer pushed a commit that referenced this issue Apr 17, 2016
quick fix to #597.
fix crash when pasting input beyond movie length. todo: make input log append like PasteInsert does.
@vadosnaprimer
Copy link
Contributor

vadosnaprimer commented Apr 17, 2016

Attempted to fix all these, can you compile?

EDIT: Will close it until it happens again.

vadosnaprimer pushed a commit to vadosnaprimer/BizHawk that referenced this issue May 8, 2016
quick fix to TASEmulators#597.
fix crash when pasting input beyond movie length. todo: make input log append like PasteInsert does.
@Niamek
Copy link
Author

Niamek commented Aug 15, 2016

Apologies for the delay, got carried a little bit.

I just tested this and I can confirm the error message is gone. However, in the last interim build on this post: http://tasvideos.org/forum/viewtopic.php?p=333074#333074
(I used this build in order to try that error), it seems that pasting all of these rows seems to freeze the emulator. But I doubt some people will actually copy and 10050 frames. I can check that freeze again if you need.

@vadosnaprimer
Copy link
Contributor

Reopening, or I'll forget it.

@vadosnaprimer vadosnaprimer reopened this Aug 15, 2016
@vadosnaprimer
Copy link
Contributor

Doesn't freeze for me when I paste.

@Niamek
Copy link
Author

Niamek commented Aug 23, 2016

Will try again.

EDIT: I am having trouble with the last build. So I might need to wait before. I can't paste since it refuses to paste.

@Niamek
Copy link
Author

Niamek commented Sep 11, 2016

Just tried with the last release. I encountered no freeze. Apologies for the delay and for the false bug!

@Niamek Niamek closed this as completed Sep 11, 2016
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

2 participants