search

Biryukov239 / matrix-multiplication-kotlin

0 stars 0 forks source link

Лишняя переменная `firstColumnCount`? #16

Closed lounres closed 5 months ago

lounres commented 5 months ago

В классе MultiplierBasedOnCPU в конструкторе определяется параметр firstColumnCount, который нигде не используется. Стоит либо заменить secondMatrix.size / secondColumnCount и firstMatrix.size / firstRowCount на него, либо убрать этот параметр.

Biryukov239 commented 5 months ago

Исправил.

lounres commented 5 months ago

Забыли избавиться от деления в calculate (первая же строчка). Но ладно, это не страшно.