Uploaded image for project: 'Apache Fineract'
  1. Apache Fineract
  2. FINERACT-1237

var: Bulk change all existing code to use Java 10+ local variable type inference

    XMLWordPrintableJSON

Details

    • New Feature
    • Status: Open
    • Major
    • Resolution: Unresolved
    • None
    • None
    • None

    Description

      Java 10+ introduced the var keyword for local variable type inference.

      I'm assuming it's OK to use it in new Fineract code - everyone agrees?

      The question is whether we should change bulk change all existing code to use it.

      If yes, the question is how to do this once time. In Eclipse there is Preferences > Save Actions > Additional Save Actions > Missing Code > Use local variable type inference (Java 10 or higher), which can do this for any file we edit - but that makes code reviews unnecessarily harder... need to check if there is a way to do this one time in bulk instead.

      An IntelliJ user instead of Eclipse may have another tool to do this more easily once time?

      ptuomola chime in here - this came out of your (absolutely justified!) feedback on my https://github.com/apache/fineract/pull/1456/files.

      Attachments

        Activity

          People

            Unassigned Unassigned
            vorburger Michael Vorburger
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated: