search

svtk / AtomicKotlinCourse

87 stars 21 forks source link

fix(Summary 2 Quadratic.kt Example): corrected a formula #49

Closed sofiiako closed 3 years ago

sofiiako commented 3 years ago

based on the operator precedence rules for / and the formula should be (-b + squareRoot) / (2 a) and not (-b + squareRoot) / 2 * a (same for the other one)

Closes https://youtrack.jetbrains.com/issue/EDC-487

BruceEckel commented 3 years ago

This was corrected recently in the book from a comment by a reader (Joel Neely, who's been meticulously going through the book and reporting issues directly to me).

Bruce Eckel SummerTechForum.com MindViewLLC.com Blog: BruceEckel.com EvolveWork.co WinterTechForum.com http://www.WinterTechForum.com OnJava8.com http://www.OnJava8.com www.AtomicKotlin.com Reinventing-Business.com http://www.Reinventing-Business.com

On Thu, Aug 5, 2021 at 6:43 AM Svetlana Isakova @.***> wrote:

Merged #49 https://github.com/svtk/AtomicKotlinCourse/pull/49 into master.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/svtk/AtomicKotlinCourse/pull/49#event-5117245293, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHUTLAPB2Y3FCOSADK3SFLT3KBNPANCNFSM5A4H2NKQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .

sofiiako commented 3 years ago

@BruceEckel right, it's the same user who reported the issue referenced in this pull request :) (https://youtrack.jetbrains.com/issue/EDC-487)