LuisR14
02-03-2010, 11:47 PM
no :p (patch isn't finished yet
)
E:
line above is equal:
so if m_EnableAFCLoopingHack is true and RepeatMode is 0 then the line above would be
which then evaluates to
which then you wouldn't get what you wanted (the AFC loop)
now if m_EnableAFCLoopingHack is false and RepeatMode is 0 then it would be
and then it would evaluate to
this time you wouldn't get the AFC loop (as expected from it being false :p)
then if m_EnableAFCLoopingHack is true and RepeatMode is not 0 then it would come out as
this time you would get it (as expected) but then
when m_EnableAFCLoopingHack is false and RepeatMode is not 0
you wouldn't get it to loop (again for the 3rd tiime, but at least it was expected :p)
so your method wouldn't work very well xD
)E:
(02-03-2010, 07:25 AM)James333 Wrote: [ -> ]well there is just a problem with that line, let me give them as examples :p(02-03-2010, 04:43 AM)Xtreme2damax Wrote: [ -> ]Code:
if (!g_Config.m_EnableAFCLoopingHack || g_Config.m_EnableAFCLoopingHack && PB.RepeatMode == 0)
line above is equal:
Code:
if (!g_Config.m_EnableAFCLoopingHack || (g_Config.m_EnableAFCLoopingHack && PB.RepeatMode == 0))so if m_EnableAFCLoopingHack is true and RepeatMode is 0 then the line above would be
Code:
if (false || (true && true))Code:
if (false || true) which equals truenow if m_EnableAFCLoopingHack is false and RepeatMode is 0 then it would be
Code:
if (true || (false && true))Code:
if (true || false) which is true (as well)then if m_EnableAFCLoopingHack is true and RepeatMode is not 0 then it would come out as
Code:
if (false || (true && false)) ...
if (false || false) equals falsewhen m_EnableAFCLoopingHack is false and RepeatMode is not 0
Code:
if (true || (false && false)) ...
if (true || false) equals trueso your method wouldn't work very well xD
(i also found a less confusing way :p)