Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Port module 'demo' to Kotlin #27

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

ITroxxCH
Copy link

@ITroxxCH ITroxxCH commented Sep 2, 2017

Since I don't really know the project very well I've kept the refactoring to a minimum. That said: I did implement some changes:

  • This is unrelated to the port but I had to fix the NetworkHandlerTest in order to build the project and run all tests.
  • The numbered options in the commandline application start now at 0 and not 1 as before since I used the forEachIndexed function.
  • I extended the CommandLine#nextLineTrimmed function so that it actually trims the next line.
  • Nullability was kept down as much as possible, except where the existing Java code dictated otherwise. However there are two places where I had to add non-null asserts (!!) to make everything compile. I've added TODOs to these places so that they can be easily tracked.
  • There is one test in SystemTest which seemed a bit strange to me. It works perfectly fine and was ported almost 1:1 from Java but I've still added a TODO since I think i might be worth to check whether the test is still valid or not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant