[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