[jvm] Интеграция FILE* с Java-потоками

Maxim Karpov me at makkarpov.ru
Thu Mar 22 21:31:45 MSK 2018


Вмержил в master.


22.03.2018 13:46, Михаил Мутафян пишет:
> Спасибо за ревью, исправил недочёты, тесты тоже добавил.
>
>     21.03.2018, 21:31, Maxim Karpov <me at makkarpov.ru
>     <mailto:me at makkarpov.ru>>
>     В целом код хороший, хотя и с небольшими замечаниями.
>
>     1. (и основное) Если fopencookie() по каким-то неведомым причинам
>     таки
>     не сможет создать поток, то вызывающий код обрушит через exit()
>     всю JVM.
>     Хотелось бы видеть создание и выкидывание исключения, причем внутри
>     самой функции открытия потока.
>
>     2. Тестовый код лучше не смешивать с основным и вынести как в
>     отдельный
>     .cpp-файл, так и в отдельный класс. Можно добавить еще тесты на
>     fscanf и
>     ungetc, хотя, наверное, и не обязательно, ведь если поток работает, а
>     scanf - нет, то это уже проблемы libc.
>
>     3. В тестовом коде можно использовать NewStringUTF, пляски с буферами
>     будет меньше.
>



More information about the jvm mailing list