[jvm] Интеграция FILE* с Java-потоками
Михаил Мутафян
mishamut at rambler.ru
Thu Mar 22 13:46:48 MSK 2018
Спасибо за ревью, исправил недочёты, тесты тоже добавил.
21.03.2018, 21:31, Maxim Karpov <me at makkarpov.ru>В целом код хороший, хотя и
с небольшими замечаниями.
1. (и основное) Если fopencookie() по каким-то неведомым причинам таки
не сможет создать поток, то вызывающий код обрушит через exit() всю JVM.
Хотелось бы видеть создание и выкидывание исключения, причем внутри
самой функции открытия потока.
2. Тестовый код лучше не смешивать с основным и вынести как в отдельный
.cpp-файл, так и в отдельный класс. Можно добавить еще тесты на fscanf и
ungetc, хотя, наверное, и не обязательно, ведь если поток работает, а
scanf - нет, то это уже проблемы libc.
3. В тестовом коде можно использовать NewStringUTF, пляски с буферами
будет меньше.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://compilers.ispras.ru/pipermail/jvm/attachments/20180322/7cc4a92d/attachment.html>
More information about the jvm
mailing list