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

Proper Audio/Video Dumping #1220

Merged
merged 1 commit into from Oct 12, 2014
Merged

Proper Audio/Video Dumping #1220

merged 1 commit into from Oct 12, 2014

Conversation

RisingFog
Copy link
Member

Screwed up the previous PR, replacing with new PR

Old notes: #1183

@RachelBryk
Copy link
Member

Still need to handle starting during emulation.


void DumpSamples (const short *buff, int nsamp, int srate);
void DumpSamplesBE (const short *buff, int nsamp, int srate);
};

This comment was marked as off-topic.

@ghost
Copy link

ghost commented Oct 4, 2014

Still storing duplicate frames when there's no need to.

@@ -0,0 +1,62 @@
#include <sstream>

This comment was marked as off-topic.

@neobrain
Copy link
Member

neobrain commented Oct 4, 2014

Still has a vast majority of coding style issues remaining.

Also, please review the usage of static local variables, I think this is extremely error prone as it is right now.

u64 tickCountTemp = s_tickCountStart;
s_tickCountStart = CoreTiming::GetTicks();
p.Do(s_tickCountStart);
s_tickCountStart = tickCountTemp;

This comment was marked as off-topic.

@skidau
Copy link
Contributor

skidau commented Oct 6, 2014

@dolphin-emu-bot rebuild

#include "VideoCommon/VideoBackendBase.h"
#include "VideoCommon/VideoConfig.h"

This comment was marked as off-topic.

@RisingFog RisingFog changed the title [WIP] Proper Audio/Video Dumping Proper Audio/Video Dumping Oct 8, 2014
void* temp_stored_frame = realloc(s_stored_frame, s_bitmap.biSizeImage);
if (temp_stored_frame)
{
s_stored_frame = temp_stored_frame;

This comment was marked as off-topic.

@JMC47
Copy link
Contributor

JMC47 commented Oct 11, 2014

Tested and confirmed to work on GC, Wii and Virtual Console games. Video dumping still leaves a lot to be desired in terms of user friendliness, but at least it works now.

skidau added a commit that referenced this pull request Oct 12, 2014
Proper Audio/Video Dumping
@skidau skidau merged commit a00ad68 into dolphin-emu:master Oct 12, 2014
@RisingFog RisingFog deleted the avsync branch October 12, 2014 03:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet