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

Comment to Chapter 13 #60

Open
mr-Serg opened this issue Aug 4, 2022 · 0 comments
Open

Comment to Chapter 13 #60

mr-Serg opened this issue Aug 4, 2022 · 0 comments

Comments

@mr-Serg
Copy link

mr-Serg commented Aug 4, 2022

p. 169 A wrong statment: "RECALL that a package may add new functionality to a class even if it does not define this class." "Recall" is wrong: there is not any word about this neither in the previous chapters nor in the next ones.

p. 170 A reference to the Listing 13-1 is missing

p. 171 "The message storeOn: writes to its argument..." - message or method?

pp. 172-173 Should be references to The Listings

p. 174 typo: "13.6 About isKindOf: and respondTo:" should be "respondsTo:"

p. 175 Question: "Since instance variables are accessed by reference, a shallow copy of an object shares its references to instance variables with the original object." Really "are variables accessed by reference"? Or variables hold references to their values? Really "shares references to instance variables"? Or shares references to values? Each instance has its own set of instance variables, so "to share references to instance variables" isn't correct on my mind.

p. 177 "Stack>>pop could easily have been implemented as follows (note that this definition is anhypothetical example and not in the Pharo 8.0 system):" but

  • the Listing 13-6 is above the text, so how it can be "as follows"
  • Pharo 8.0 or 9.0?

p. 180

  • Error in Figure 13-10: LargeInteger missing. LargePositiveInteger and LargeNegativeInteger both are subclasses of LargeInteger.
  • "The remaining operators are generically defined. For example:" - Ups, the example jumped to the Listing to the top of the page.

p.181 ">>> 100@10 "Point creation" " Should be ">>> (100@10) "Point creation" "

p. 182 Error: "Fraction>>numerator:denominator:" Should be "Fraction class >> numerator:denominator:"

p. 183 "The protocol accessing untypeable characters contains a number of convenience constructor methods such as backspace, cr, escape, euro, space, tab, and so on." - the Character class >> euro do not belongs to that protocol, but it is an extension method.

p. 185 Error: "'hello' at: 2 put: $u; yourself" - the string literal is immutable

p. 186 "The implemen�tations in its concrete subclasses are both trivial:" again, the code jumped to the top

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

No branches or pull requests

1 participant