ref: Move Harvesters to kotlin.

[bgrozev]

No description provided.

[codecov]

Codecov Report

Attention: 27 lines in your changes are missing coverage. Please review.

Comparison is base (b2d4229) 0.00% compared to head (f9c4ad3) 43.99%.

Additional details and impacted files

@@              Coverage Diff              @@
##             master    #2081       +/-   ##
=============================================
+ Coverage          0   43.99%   +43.99%     
- Complexity        0     1919     +1919     
=============================================
  Files             0      340      +340     
  Lines             0    18881    +18881     
  Branches          0     2590     +2590     
=============================================
+ Hits              0     8307     +8307     
- Misses            0     9725     +9725     
- Partials          0      849      +849     

Files	Coverage Δ
...rg/jitsi/videobridge/transport/ice/IceTransport.kt	25.92% <50.00%> (ø)
jvb/src/main/kotlin/org/jitsi/videobridge/Main.kt	0.00% <0.00%> (ø)
...n/org/jitsi/videobridge/health/JvbHealthChecker.kt	34.78% <0.00%> (ø)
...ain/kotlin/org/jitsi/videobridge/ice/Harvesters.kt	26.66% <26.66%> (ø)

... and 336 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b2d4229...f9c4ad3. Read the comment docs.

[JonathanLennox]

I understand this constructs the object and thus does the static initialization, but it still feels weird idiomatically to just reference an object and not use it. Is there any way to make this look like a function call without making other things less idiomatic?