Martin Guy

Tested on Debian-x86-32bit, give 19 messages like tested ogg: no (compilation failed) and spews lots of compiler warnings like aiff.c:36:44: note: expected ‘char *' but argument is of type ‘const char *’ aiff.c:885:21: warning: assignment discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers] wav.c:548:5: warning: missing initializer for field ‘encoding’ of ‘const struct waveformat’ [-Wmissing-field-initializers] probably because the standard build silences compiler warnings...

That's a hard thing to do because decoding the current frame can require data left over from decoding the previous two. libmpg123 manages even sample-accurate seeking very well - maybe you could try that.

Great tool! I've tried doing resynthesis myself but your results are much cleaner. To compile it on Linux I had to make the attached changes in src/, otherwise it fails complaining about multiple definitions of half a dozen variables. That just leaves a slew of warnings about inline functions being declared but not defined, which can be fixed by sed -i 's/inline //' *.[ch] as modern compilers and linkers can probably decide better what to inline and what not to. Many thanks for publishing this M...

Output file with patch applied

Confirmed on Debian stable (bullseye) with git master HEAD 42b3557e1 also when the au is converted to wav, also when the "rate 3k" effect is omitted. The segfault occurs in print_at_() when it's trying to add the "Time (s)" label to the graph, and is probably caused by the file only being 100 samples long, about 0.002 seconds. "sox int8-linear-pcm.au -n spectrogram -r" does not segfault and produces a png file 13 pixels wide. Without -r, it's trying to print "Time (s)" at screen coordinate x = -2147483572...

Sox's default fade type is logarithmic, not linear. It is behaving correctly. You need "fade t 0.5 0 0.5" Not a bug.

This has several issues: - it doesn't apply cleanly to the git master HEAD - the formatting doesn't match the exiting code - the compiler reports that "i" may be used uninitialized at line 915 (tick=i/10) and the later "Tick label" line. Use of i here, left over from the end of a previous loop, is bizarre. - I've tried applying the failed patch (to spectrogram.c) manually to git master HEAD, but the x-axis ticks remain in seconds, and "Time (s)" is replaced by the file length in HMS format (that's...

This has several issues: - it doesn't apply cleanly to the git master HEAD - the formatting doesn't match the exiting code - the compiler reports that "i" may be used uninitialized at line 915 (tick=i/10) and the later "Tick labes" line. Use of i here, left over from the end of a previous loop, is bizarre. - I've tried applying the failed patch (to spectrogram.c) manually to git master HEAD, but the x-axis ticks remain in seconds, and "Time (s)" is replaced by the file length in HMS format (that's...