(02-02-2010, 12:59 AM)James333 Wrote:this doesn't seem right either :p, you can just remove the "!g_Config.m_EnableAFCLoopingHack ||" portion(02-01-2010, 05:40 AM)Xtreme2damax Wrote: Instead of:Why ?
if (g_Config.m_EnableAFCLooping || PB.RepeatMode == 0)
It should be:
if (!g_Config.m_EnableAFCLoopingHack || g_Config.m_EnableAFCLoopingHack && PB.RepeatMode == 0)
AFC Looping Checkbox for DSP Plugin
|
02-02-2010, 05:03 AM
02-02-2010, 05:51 AM
(02-02-2010, 05:03 AM)LuisR14 Wrote:(02-02-2010, 12:59 AM)James333 Wrote:this doesn't seem right either :p, you can just remove the "!g_Config.m_EnableAFCLoopingHack ||" portion(02-01-2010, 05:40 AM)Xtreme2damax Wrote: Instead of:Why ? It works just fine as it is, the way you mentioned causes problems and with James333 way the option is on all the time and can't be toggled. Can anyone give me some hints how to hook this to ISOProperties.cpp and make the option functional in the GameConfig? I'm already half way or almost finished, I just need to know how to do this. Any developers that can lend any insight or feedback? (02-02-2010, 05:03 AM)LuisR14 Wrote:(02-02-2010, 12:59 AM)James333 Wrote:this doesn't seem right either :p, you can just remove the "!g_Config.m_EnableAFCLoopingHack ||" portion(02-01-2010, 05:40 AM)Xtreme2damax Wrote: Instead of:Why ? At the moment of writing th patch I didn't knew which was the hack ( and actually , I didn't care xD ) so I leaved it like that. Actually I can't see why the original was wrong ( I read the Xtreme2damax posts but I didn't understand ) I would keep it like if (!g_Config.m_EnableAFCLoopingHack || PB.RepeatMode == 0) EDIT : and use && is not a good idea without the "!g_Config.m_EnableAFCLoopingHack ||" portion (02-02-2010, 05:51 AM)Xtreme2damax Wrote: It works just fine as it is, the way you mentioned causes problems and with James333 way the option is on all the time and can't be toggled.Now I understand , yeah , apparently it doesn't change in game ( but it does if you stop the game, at least It did in my tests )
MB : MSI Z68A-GD80
CPU : Intel Core i7 2600k @ 3.4 GHz (stock clock) Gfx : ATI Raedon HD 6850 Ram : 4 GB DDR3-1333 OS : Windows XP x86 I speak Spanish ( And a little English ) 02-02-2010, 05:57 AM
(02-02-2010, 05:57 AM)LuisR14 Wrote:Do you know how the ISOProperties.cpp overrides the main settings?? ( We can't make it override the settings of the DSP plugin D: )(02-02-2010, 05:51 AM)Xtreme2damax Wrote: Any developers that can lend any insight or feedback?well i'm currently looking at it
MB : MSI Z68A-GD80
CPU : Intel Core i7 2600k @ 3.4 GHz (stock clock) Gfx : ATI Raedon HD 6850 Ram : 4 GB DDR3-1333 OS : Windows XP x86 I speak Spanish ( And a little English ) 02-02-2010, 09:58 AM
Yes it should on a per game basis, so if you have it checked in the DSP options and unchecked in the Gameconfig AFC sound won't loop. I think I'm close to figuring it out, but I can seem to compile without errors. It looks like the Gameini code in ISOProperties was commented out for the RE0 hack and some looping hack that is likely unrelated to the AFC looping hack. I just started editing any files that were related to the DSP and Gameini configuration, adding the RE0 and AFCLooping hack under files that have EnableDTKMusic and AudioThrottle lines. I almost have it compiling correctly except for one or two small errors.
Here is the error I am receiving, perhaps someone can tell me what to do. God I hope this works.. Here are the errors: Code: Error 7 error C2661: 'IniFile::Set' : no overloaded function takes 4 arguments d:\My Documents\Emulator SVN\Dolphin-Modified\Source\Plugins\Plugin_DSP_HLE\Src\Config.cpp 72 Here is the code contained in Config.cpp: Code: // Copyright (C) 2003 Dolphin Project. 02-02-2010, 11:57 AM
(02-02-2010, 09:58 AM)Xtreme2damax Wrote: It looks like the Gameini code in ISOProperties was commented out for the RE0 hack and some looping hack that is likely unrelated to the AFC looping hack.actually it is related and maybe you should read the comments :p HLE's Config.cpp Code: void CConfig::GameIniLoad(const char *game_ini) 02-02-2010, 02:04 PM
Well I was trying to work around that and fix it, already saw that comment..
02-02-2010, 04:02 PM
Once proper looping is implemented, a checkbox to enable/disable looping would be nice for testing. It's seems like looping needs a lot of work in Dolphin, I notice that other games besides the games that use the Mario, Zelda, Pikmin, Luigi, SMS UCodes suffer from audio looping issues. Resident Evil Zero, Remake, RE2, RE3, and RE4 suffer from this as do some other games..
I have to figure out how to be able to override and enable/disable the AFC looping hack in the GameConfig. It seems editing ISOPropeties.cpp and ISOProperties.h is not enough. If I could find where all the other options in the GameConfig are hooked to that would be a step in the right direction. 02-02-2010, 07:30 PM
took me some time to improve your patch and figure out what was wrong with ISOProperties but here's v2 of the patch :p
(also i was testing it and it seemed that ZTP was ok with it disabled o_o) |
« Next Oldest | Next Newest »
|
Users browsing this thread: 1 Guest(s)