JamalAbuDayyeh / winff

Automatically exported from code.google.com/p/winff
0 stars 0 forks source link

Can't convert more than 99 audio files at once #209

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. I am trying to convert audio .wav files to .ogg. I drag and drop more than 
99 files into Winff, for instance, 102 .wav files. The output details are: 
Convert to=Audio; Preset=OGG; the output folder is different than the original 
one.
2.
3.

What is the expected output? What do you see instead?
Winff should display the list of the 102 files - instead I get an error 
message: "Access violation. Press OK to ignore and risk data corruption. Press 
Cancel to kill the program."

What version of the product are you using? On what operating system?
I tried this on 3 different settings and got the same issue each time:
- Version 1.5.3, Win7, 32 bit
- Version 1.5.3, Win7, 64 bit
- Version 1.5.3, Linux Mint 16 Petra, Mate Edition

Please provide any additional information below.

Original issue reported on code.google.com by bakirdj...@gmail.com on 19 Feb 2014 at 9:13

GoogleCodeExporter commented 9 years ago
I get the same issue with Version 1.5.3, 32-bit on Windows 8.1.

Original comment by mastermo...@gmail.com on 16 Mar 2014 at 2:35

GoogleCodeExporter commented 9 years ago
And I also confirm it under Linux.

Probably this is also reported in Ubuntu in LP 1295226.

I have to verify, but chances are big that we can only work around this in 
Winff by not allowing more than 99 files as I think this is broken in one of 
the libraries that we use.

Original comment by poipodec...@hotmail.com on 21 Mar 2014 at 7:51

GoogleCodeExporter commented 9 years ago
BUG:

Due to the extra info stored about each job, the system allocates space for 100 
items by default.  The list was being expanded properly.

New behaviour is that when the job list nears the limit, the job limit is 
incremented by 50.  

Tested over 300.

Original comment by istoff@gmail.com on 19 May 2014 at 2:46