Проверять на выход за границу типа в функции Factorial(int v)

[GoogleCodeExporter]

What steps will reproduce the problem?
1.Подать значение больше v > 29
2.
3.

What is the expected output? What do you see instead?
30! = 2.65252859812191E32 instead ?

Please use labels and text to provide additional information.
./sandbox/.svn/text-base/tdd_example_test.cpp.svn-base:16:  
// TODO(timurrrr): CHECK(v < MAX_FACTORIAL);

Original issue reported on code.google.com by DanichBl...@gmail.com on 10 Oct 2011 at 12:25

[GoogleCodeExporter]

Я правильно понял, что ты хочешь за это 
взяться?

Замечание:
> ./sandbox/.svn/text-base/tdd_example_test.cpp.svn-base:16: 
Я правильно понимаю, что ты воспользовался 
grep -r ?
Советую использовать --exclude-dir=".svn"
Файлы в папке .svn - "скрытые", простому 
смертному туда лучше не лезть.

Original comment by timurrrr on 10 Oct 2011 at 8:42

• Changed state: Assigned

[GoogleCodeExporter]

Да хочу взяться.

Да использовал grep, спасибо за совет. Только 
он говорит что нет
такого флага, зато --exclude="*\.svn*" сработало.

Original comment by DanichBl...@gmail.com on 12 Oct 2011 at 9:57

[GoogleCodeExporter]

> Только он говорит что нет такого флага
Возможно, старая версия.
У меня grep 2.5.4

Original comment by timurrrr on 13 Oct 2011 at 8:11

[GoogleCodeExporter]

(делаю то, что по идее должен был сделать 
Данила)
Проверка была добавлена в r144, но позже 
оказалось ( http://codereview.appspot.com/5288042/#msg4 )
что проверка неправильная:
  const int kMaxFactorial = 28;

  12!    =   479 001 600
  2^32-1 = 4 294 967 295
  13!    = 6 227 020 800

Original comment by timurrrr on 21 Oct 2011 at 9:35

• Changed state: Started

[GoogleCodeExporter]

Так какой статус?

Original comment by timurrrr on 2 Dec 2011 at 3:57

[GoogleCodeExporter]

Fixed. Reviewed at http://codereview.appspot.com/5330047/

Original comment by DanichBl...@gmail.com on 2 Dec 2011 at 4:42

• Changed state: Fixed

[GoogleCodeExporter]

А ты это закоммитил?

Original comment by timurrrr on 2 Dec 2011 at 4:44

[GoogleCodeExporter]

хм... Я сейчас слделал svn update и svn status, и он мне 
говорит что tdd_example_test.cpp не модифицирован. 
Это значит закомитил?

Original comment by DanichBl...@gmail.com on 2 Dec 2011 at 5:00

[GoogleCodeExporter]

И при этом у меня в локальном файле все так 
же как здесь http://codereview.appspot.com/5330047/

Original comment by DanichBl...@gmail.com on 2 Dec 2011 at 5:02

[GoogleCodeExporter]

А, понятно - r167
На будущее: закрывай issue как можно раньше; 
указывая номер ревизии/ревизий с фиксами.

Original comment by timurrrr on 2 Dec 2011 at 5:24

[GoogleCodeExporter]

Ок, виноват.

Original comment by DanichBl...@gmail.com on 2 Dec 2011 at 5:52