Page 1 of 1

Wrong argument order in fStream::revert

Posted: Sun Jul 22, 2012 10:31 am
by gocha
I found a simple but a fatal bug. Because of this, loading a movie snapshot never succeed.

Code: Select all

9c483d87a4704965e783521f3990fdb808d1c6e4
 stream.cpp |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/stream.cpp b/stream.cpp
index 0d4082c..9bc0fb1 100644
--- a/stream.cpp
+++ b/stream.cpp
@@ -283,7 +283,7 @@ size_t fStream::size (void)
 
 int fStream::revert (size_t from, size_t offset)
 {
-    return (REVERT_FSTREAM(fp, from, offset));
+    return (REVERT_FSTREAM(fp, offset, from));
 }
 
 void fStream::closeStream()

Re: Wrong argument order in fStream::revert

Posted: Sun Jul 22, 2012 1:18 pm
by OV2
Thanks, committed.

For those changes you already have on github it's enough to post a link to the commit or simply the commit hash, I can take them from there. Might save you some time ;)

Re: Wrong argument order in fStream::revert

Posted: Sun Jul 22, 2012 2:37 pm
by gocha
Thank you.
OV2 wrote:For those changes you already have on github it's enough to post a link to the commit or simply the commit hash, I can take them from there. Might save you some time ;)
I see, thanks OV2.