Skip to content

Conversation

vigneshsarma
Copy link
Contributor

I have fixed the easy ones. There is some more left.

the remaining ones,

Reflection warning, cider/nrepl/middleware/out.clj:53:22 - call to method write can't be resolved (target class is unknown).
Reflection warning, cider/nrepl/middleware/out.clj:55:24 - call to method write on java.lang.Object can't be resolved (no such method).
Reflection warning, cider/nrepl/middleware/out.clj:55:24 - call to method write on java.lang.Object can't be resolved (no such method).
Reflection warning, cider/nrepl/middleware/out.clj:57:22 - call to method write can't be resolved (target class is unknown).
Reflection warning, cider/nrepl/middleware/out.clj:59:24 - call to method write on java.lang.Object can't be resolved (no such method).
Boxed math warning, byte_streams/graph.clj:130:77 - call: public static double clojure.lang.Numbers.unchecked_add(java.lang.Object,double).
Reflection warning, cider/nrepl/middleware/out.clj:59:24 - call to method write on java.lang.Object can't be resolved (no such method).
Reflection warning, cider/nrepl/middleware/out.clj:61:21 - reference to field flush can't be resolved.
Reflection warning, cider/nrepl/middleware/out.clj:63:23 - reference to field flush on java.lang.Object can't be resolved.
Reflection warning, cider/nrepl/middleware/out.clj:63:23 - reference to field flush on java.lang.Object can't be resolved.

Reflection warning, cider/nrepl/middleware/inspect.clj:47:25 - call to java.lang.Exception ctor can't be resolved.

Reflection warning, cider/nrepl/middleware/debug.clj:159:29 - reference to field sym on java.lang.Object can't be resolved.
Reflection warning, cider/nrepl/middleware/debug.clj:159:29 - reference to field sym on java.lang.Object can't be resolved.

Not completly sure how to fix the remaining ones.

Copy link
Member

@cichli cichli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution! Could you change the commit message to be in the imperative? i.e. Fix reflection warnings.

[url-str]
(if-let [url (try (URL. url-str)
(if-let [^URL url (try (URL. url-str)
(catch MalformedURLException e nil))]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you fix the indentation here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated.

@cichli
Copy link
Member

cichli commented Jan 19, 2019

I have pushed a commit that will fix the Eastwood warnings, if you would like to rebase.

@vigneshsarma vigneshsarma force-pushed the fix-reflection-warnings branch 2 times, most recently from dba97d2 to 309ab4a Compare January 19, 2019 21:01
@vigneshsarma
Copy link
Contributor Author

I have rebased it with master and also fixed the commit message.

I am not sure how to fix the remaining reflection warning. If you have ideas, I can fix them as well.

@codecov
Copy link

codecov bot commented Jan 19, 2019

Codecov Report

Merging #589 into master will increase coverage by 0.04%.
The diff coverage is 40%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #589      +/-   ##
==========================================
+ Coverage   76.68%   76.72%   +0.04%     
==========================================
  Files          38       38              
  Lines        2393     2389       -4     
  Branches      140      138       -2     
==========================================
- Hits         1835     1833       -2     
  Misses        418      418              
+ Partials      140      138       -2
Impacted Files Coverage Δ
src/cider/nrepl/middleware/content_type.clj 33.33% <0%> (ø) ⬆️
src/cider/nrepl/print_method.clj 76.92% <100%> (ø) ⬆️
src/cider/nrepl/middleware/slurp.clj 61.11% <25%> (+0.58%) ⬆️
src/cider/nrepl/middleware/debug.clj 55.21% <66.66%> (ø) ⬆️

Continue to review full report at Codecov.

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

@vigneshsarma vigneshsarma force-pushed the fix-reflection-warnings branch from 309ab4a to 72adfd4 Compare January 23, 2019 21:23
I have fixed the easy ones. There is some more left.
@cichli
Copy link
Member

cichli commented Feb 11, 2019

Fixed in d5f1f3a. Thanks so much for looking at this – sorry it's taken me a while to get it merged.

I am not sure how to fix the remaining reflection warning. If you have ideas, I can fix them as well.

If I get around to taking a look at them I'll ping you.

@cichli cichli closed this Feb 11, 2019
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.

2 participants