Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HEAD does not compile with AACPLUS without LIBSAMPLERATE +FIX #41

Closed
GoogleCodeExporter opened this issue May 13, 2015 · 2 comments
Closed

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. compile with aacplus-2 without libresample
2.
3.

What is the expected output? What do you see instead?
- compile to success

What version of the product are you using? On what operating system?
- HEAD 1.0.1

Please provide any additional information below.

- typo in src/aacPlusEncoder.cpp inSamples/inputSamples

FIX IS HERE


diff -u ../darkice-0.20.1/src/aacPlusEncoder.cpp src/aacPlusEncoder.cpp
--- ../darkice-0.20.1/src/aacPlusEncoder.cpp    2010-11-18 17:56:12.000000000 +0100
+++ src/aacPlusEncoder.cpp  2010-12-27 15:01:55.266664112 +0100
@@ -114,7 +114,7 @@
 #else
         converter->initialize( resampleRatio, getInChannel());
         //needed 2x(converted input samples) to handle offsets
-    int         outCount = 2 * getInChannel() * (inSamples + 1);
+    int         outCount = 2 * getInChannel() * (inputSamples + 1);
         if (resampleRatio > 1)
         outCount = (int) (outCount * resampleRatio);
         resampledOffset = new short int[outCount];

Original issue reported on code.google.com by oetelaar.automatisering on 27 Dec 2010 at 2:12

@GoogleCodeExporter
Copy link
Author

Issue 46 has been merged into this issue.

Original comment by rafael2k...@gmail.com on 16 Jan 2012 at 9:03

@GoogleCodeExporter
Copy link
Author

Already applied.

Original comment by rafael2k...@gmail.com on 14 May 2013 at 3:58

  • Changed state: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant