[P&AM Lab] Библиотека больших чисел

[Grigoriy A. Sitkarev]

Всё хорошо, и перестановка во втором условии в bnum_cmp() действительно 
не нужна т.к. там однозначно порядок одинаковый (мы это проверили выше).

Но в init.c пожалуй изменять не стоило т.к. пользователь который 
вызывает bnum_init() обязан чётко дать количество разрядов которые 
следует поместить в значение. Причём, ДАЖЕ если это нулевые разряды 
никакой проверки там быть не должно. Если они туда попадут это проблема 
пользователя. В bnum_init() не должно быть политики проверки что в этих 
разрядах лежит, если пользователю хочется их положить значит так нужно.

 > Григорий Александрович, а можно ли как то оставлять комментарии к 
изменениям в
 > ревизии?

По-моему, у тебя как раз получается их оставлять при коммите. Только они 
должны быть на английском языке. Как и всё что в этом проекте.

Боря, обрати пожалуйста внимание не стиль. НЕЛЬЗЯ скобки ставить так:

if ( a > b)

должно быть:

if (a > b)

Может быть тебе стоит почитать LKCS 
http://www.kernel.org/doc/Documentation/CodingStyle.

--
Г.А.

27.02.2011 23:44, LonelyRuyK пишет:
> Внёс небольшие изменения в файлики cmp.c и init.c. Надеюсь ничего не
> испортил:) Изменения вызваны тем что в init.c в функции bnum_set не совсем
> правильно рассчитывалась переменная bn->top ( в случае когда передавался
> массив с нулями вначале, top и их считал)
> А в файле cmp.c было несколько не нужных строчек.