Please consider registering

sp_LogInOut Log In sp_Registration Register

Register | Lost password?
Advanced Search

— Forum Scope —

— Match —

— Forum Options —

Minimum search word length is 3 characters - maximum search word length is 84 characters

sp_Feed Topic RSS sp_TopicIcon
Typos in print edition
February 7, 2015 - 5:45 pm
Member Since: January 10, 2015
Forum Posts: 8
sp_UserOfflineSmall Offline

As I am just working through the print book, I suggest that I post typos I find along the way here to help improve the 2nd edition. If this is not ok, please just delete this topic.

Page 231: on the very last line, "synx" should be "sync"

Page 240: "dOut = dTrivialSaw + do_BlepN(&dPLEPTable 4096, " should be "dOut = dTrivialSaw + doBLEP_N(&dBLEPTable[0], 4096,"

Page 241: doTriangle() should be declared inline

Page 242: The line checkWrapIndex(dCalcModulo); has an intendation, but should have none.

Page 242: the switch case for SINE could be revised, as it is not 100% identical to the LFO implementation, as it uses the local variable dCalcModulo. I would insert the complete code here, as it is just like 4 lines.

Page 247: in the Destructor signature is an unnecessary space

Page 248: the signature of the update function is updateOscillator() instead of update().

February 7, 2015 - 7:39 pm
Member Since: January 10, 2015
Forum Posts: 8
sp_UserOfflineSmall Offline

on page 249: pSawTable[i] += ... there is one closing brace missing at the end of the statement.

February 7, 2015 - 8:33 pm
Member Since: January 28, 2017
Forum Posts: 659
sp_UserOfflineSmall Offline

Thanks! I will add to the Errata document.


February 7, 2015 - 9:17 pm
Member Since: January 10, 2015
Forum Posts: 8
sp_UserOfflineSmall Offline

on page 231: the variable name of LFO1 Amplitude should be m_dLFO1Amplitude

on page 259, suggestion: the conditional for input channels could be removed, it's about synths here, and the inputs is not relevant in the current context. resulting in "if (uNumOutputChannels == 2) pOutputBuffer[1] = dOut ;

February 13, 2015 - 2:45 am
Member Since: January 10, 2015
Forum Posts: 8
sp_UserOfflineSmall Offline

On page 346: CFilter Member Functions (virtual): Update function should not start with a capital letter.

On page 348: in the source code right at the top of the page, there is a html entity instead of a > in the line: if (m_dFc > FILTER_FC_MAX)

On page 369: in the code block on the line "m_DCA.doDCA(dFilterOut, dFilterOut, dOutL, dOutR);" both instances of dFilterOut should be printed in bold, as they also do change from the previous code.

On page 371: This one has cost me a good hour of debugging. In the code the line "m_Filter1.setQControl(m_dQControl);" is wrong, it's a bug, and it needs to be as in the example source "m_Filter1.m_dQControl = m_dQControl;", because if this is not changed, the member variable of the filter class never gets set to the actual value, and in the update() method of the filter base class, always the initial value is applied again after every update from the GUI, causing the Q to not work at all. This should make it to the official errata (speaking of which, as of today I could only find one for the Audio FX book).

P399 + P400: I think the QBLimitedOscillator.h and WTOscillator.h don't need the code update, there are no corresponding source definitions in synthfunctions.h. Cant find it in the downloadable source files as well.

Forum Timezone: America/New_York

Most Users Ever Online: 152

Currently Online:
5 Guest(s)

Currently Browsing this Page:
1 Guest(s)

Top Posters:

Chaes: 56

Skyler: 48

Derek: 46

Frodson: 45

Peter: 43

TheSmile: 43

clau_ste: 39

Nickolai: 39

JimmyM: 33

Gwen: 32

Member Stats:

Guest Posters: 1

Members: 742

Moderators: 1

Admins: 6

Forum Stats:

Groups: 13

Forums: 42

Topics: 820

Posts: 3172

Moderators: W Pirkle: 659