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

Is it duplicated with forgelin? #1

Closed
forestbat opened this issue Dec 14, 2017 · 1 comment
Closed

Is it duplicated with forgelin? #1

forestbat opened this issue Dec 14, 2017 · 1 comment
Assignees
Labels

Comments

@forestbat
Copy link

forestbat commented Dec 14, 2017

https://minecraft.curseforge.com/projects/shadowfacts-forgelin

@ocpu ocpu added the question label Dec 14, 2017
@ocpu ocpu self-assigned this Dec 14, 2017
@ocpu
Copy link
Owner

ocpu commented Dec 14, 2017

Now that I have looked at Forgelin source code I can say we do really similar things. But that is quite inevitable since we both are language adapters for Kotlin. Though there are some things to note.

  1. Making it a core mod is not needed since Kotlin is great for Java interop. The only thing I see Forgelin do with it being a core mod is load itself.
    As of the time of writing you get a warning using Forgelin and it was created 6 of August this year. (Forgelin issue)
  2. Does not support you using instance factory methods. Not giving you all the flexibility you could use if you wanted.

Otherwise there arn't any big drawbacks using either mod when coding with Kotlin for forge. It is possible to not use any of our mods when building forge mods. You just have to use the syntax similar to how you code a java mod now.

@ocpu ocpu closed this as completed Dec 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants