Follow-up from "Dorofeev Nickolay P32101"
The following discussions from !134 (closed) should be addressed:
-
@iwishyoujoy started a discussion:
можно константы, пожалуйста, капсом?
-
@iwishyoujoy started a discussion:
не соблюдается соглашение вызова, регистры r12-r15 - callee-saved, их необходимо сначала пушить, а потом попить, если хочешь хранить в них значения. я бы порекомендовала использовать для этого caller-saved регистры
-
@iwishyoujoy started a discussion:
лучше пушить значение, которое хочешь сохранить перед вызовом функции. не соблюдается соглашение вызова.
-
@iwishyoujoy started a discussion:
магические числа. лучше вынести в константу с понимаемым названием
-
@iwishyoujoy started a discussion:
call + ret = jmp
-
@iwishyoujoy started a discussion:
32?... магические числа
-
@iwishyoujoy started a discussion:
20?.. в этой функции куча магических чисел, лучше замени/поясни
-
@iwishyoujoy started a discussion:
лучше написать просто '0'
-
@iwishyoujoy started a discussion:
call + ret = jmp тут и на строках 95-96
-
@iwishyoujoy started a discussion:
зачем?
-
@iwishyoujoy started a discussion:
вынести в константы пробельные символы
-
@iwishyoujoy started a discussion:
соглашение вызова? push, pop
-
@iwishyoujoy started a discussion:
зачем вызываешь, если не используешь результат функции?
-
@iwishyoujoy started a discussion:
можно реализовать функцию без лишнего прохода по строке
-
@iwishyoujoy started a discussion:
'0'
-
@iwishyoujoy started a discussion:
45? магические числа, лучше написать тогда '-', если это оно
-
@iwishyoujoy started a discussion:
call + ret = jmp
-
@iwishyoujoy started a discussion:
можно реализовать функцию без лишнего прохода по строке, длину строки тут знать не обязательно. подсказка: проверять нуль-терминатор
-
@iwishyoujoy started a discussion:
jne .loop ret так лучше, тогда можно удалить строки 264-265
-
@iwishyoujoy started a discussion:
your* здесь и на строке 16
-
@iwishyoujoy started a discussion:
test rax, rax работает быстрее
-
@iwishyoujoy started a discussion:
это, конечно, работает, но лучше писать jmp exit
-
@iwishyoujoy started a discussion:
ты молодец! очень постарался! исправить ошибочки и вообще будет идеально!
Все исправил, спасибо за ревью