amarao (amarao_san) wrote,
amarao
amarao_san

code review request

Я знаю, это жопа, и так жить нельзя, но а) жопа дана нам в ощущениях б) жить надо.

Просьба посмотреть на этот код и рассказать мне почему он говно (кроме экзестенциальной части):

(начиная с функции continue_in_fork() и далее)

https://github.com/amarao/thread_timeout/blob/double_fork/thread_timeout/__init__.py#L148

Пример использования:
https://github.com/amarao/thread_timeout/blob/double_fork/fork_test.py

(По нажатию Ctrl-C сигнал ловится родителем, родитель перепосылает сигнал ребёнку, ребёнок отрабатывает atexit() и перепосылает сигнал родителю, который по SIGTERM завершается).

Как для такого писать адекватные тесты - не знаю.
Tags: code review, python, thread_timeout
Subscribe

  • mindmaps

    Я нашёл новый метод читать скучную документацию (скучная - это которая на 50% похожа на man с ключиками) - это заполнение mindmap'а по…

  • первые недостатки mindmap'ов

    Вот они и начинают появляться. Раньше я точно знал, что если я что-то сильно отложу, я к этому смогу вернуться только с большими усилиями. А теперь…

  • mind map

    1) Выматывают жутко. Исчезает масса "работы клерка" по заполнению всякого рода неудобных файлов, журналов и т.д. - приходится работать ВСЁ время.…

  • Post a new comment

    Error

    default userpic

    Your IP address will be recorded 

    When you submit the form an invisible reCAPTCHA check will be performed.
    You must follow the Privacy Policy and Google Terms of use.
  • 13 comments

  • mindmaps

    Я нашёл новый метод читать скучную документацию (скучная - это которая на 50% похожа на man с ключиками) - это заполнение mindmap'а по…

  • первые недостатки mindmap'ов

    Вот они и начинают появляться. Раньше я точно знал, что если я что-то сильно отложу, я к этому смогу вернуться только с большими усилиями. А теперь…

  • mind map

    1) Выматывают жутко. Исчезает масса "работы клерка" по заполнению всякого рода неудобных файлов, журналов и т.д. - приходится работать ВСЁ время.…