(i line-break my urls using backslashes -- breaks links but keeps my code sane) 😛

is there a way to find out unused public vars?

@ikitommi https://github.com/borkdude/carve

Carve is pretty darn good 🙂

Maybe the #kaocha channel?

Too many Ks, sorry :)

clj-kondo warns here:

src\app\lib\vk\java_sdk\core.clj:10:47: warning: Unused import GetResponse
src\app\lib\vk\java_sdk\core.clj:12:35: warning: Unused import DomainResolved

seems like a false positive when using type hints in ->. welcome to post a repro with a minimal example in an issue

ok

@serioga I do always forget the right place to type hints, but if I remember correctly, you have your return type hints in the wrong place

(defn ^:private ^ServiceActor conn-service-actor
  "Service actor from connection."
  [{:keys [::app-id
           ::app-service-token]}]
  (ServiceActor. app-id app-service-token))

You want ^ServiceActor hint to be on the arg vector, not the var

Like so:

(defn ^:private conn-service-actor
  "Service actor from connection."
  ^ServiceActor [{:keys [::app-id
           ::app-service-token]}]
  (ServiceActor. app-id app-service-token))

Maybe But I've put type hints to get rid of warning from Cursive and reflection warning 🙂

Ya, but those probably do nothing

So whatever got rid of your warnings it's not that

I'll have a look with macroexpand

See the doc https://clojure.org/reference/java_interop

> For function return values, the type hint can be placed before the arguments vector

I actually think clj-kondo should warn against that 😝

When you type hint where you did, you're type hinting the Var, but the Var holds a Fn, so its correct type is Fn, not ServiceActor

In the code above the problem is with result from .execute which has different types

@didibus are we talking about this code fragment?

(try
    (-> vk-api-client
        (.friends)
        (.get (conn-service-actor conn))
        (.userId (Integer/parseInt user-id))
        ^GetResponse (.execute)
        (.getItems)
        (->> (mapv str)))
    (catch ApiAccessException _
      (private-profile-friends))
    (catch ApiPrivateProfileException _
      (private-profile-friends)))

I'm talking about:

(defn ^:private ^ServiceActor conn-service-actor
  "Service actor from connection."
  [{:keys [::app-id
           ::app-service-token]}]
  (ServiceActor. app-id app-service-token))

there is not warning about this https://clojurians.slack.com/archives/CHY97NXE2/p1583871935143100

the warnings are about type hints in -> (2 places)

I just mean, in your function signature, I think the correct place to type hint the return value is the args vector

“I think the correct place to type hint the return value is the args vector” Agree I also think those hints are useless But I did not know this when wrote that code 🙂

Maybe yet another case for clj-kondo to check and warn :-)

@borkdude what do you think? 😉 https://clojurians.slack.com/archives/CHY97NXE2/p1583872276147100

Shall we focus on the first issue first (type hints in threading macros do prevent from unused import warnings). I haven't seen a Github issue for that yet 😉

@borkdude https://github.com/borkdude/clj-kondo/issues/795 new (I think) false positive

@serioga I think this is a minimal repro for your issue:

$ clj-kondo --lint - <<< '(ns foo (:import [foo Bar])) (defn foo [x] x) (defn bar [x] (-> x ^Bar (foo)))'
<stdin>:1:23: warning: Unused import Bar
linting took 11ms, errors: 0, warnings: 1

@tanzoniteblack Thanks, duplicate of #787

I saw that one and wasn't clear they were duplicates of each other or not

seemed like the same error, but probably due to different reasons

@serioga Filed your issue here now: https://github.com/borkdude/clj-kondo/issues/796

@tanzoniteblack I might have closed it prematurely. I'll take another look.

re-opened it. I'll have a look at it before the next release

@serioga @tanzoniteblack issues are solved on master now