[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