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

AISS L7 G02 - University work #1073

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

gonmarfer2
Copy link

This is a very small group contribution made to this project based on minor bugs found by using SonarCloud.
The changes in the pom.xml file were exclusive for the project to work on the forementioned website.

We do not intend to get accepted unless the bugs fixed (listed in all commits) are actually valuable.

gonmarfer2 and others added 7 commits March 31, 2021 18:50
…lg/linkprediction/PreferentialAttachmentLinkPrediction.java
org.jgrapht.alg.linkprediction SaltonIndexLinkPrediction: L86 double cast
org.jgrapht.alg.matching GreedyWeightedMatching: L117,119 double cast
org.jgrapht.alg.cycle BergeGraphInspector: L648 double cast
org.jgrapht.alg.matching PathGrowingWeightedMatching: L171,243 get() -> orElse(null)
org.jgrapht.alg.cycle Cycles: L86 get() -> orElse(null)
org.jgrapht.demo HelloJGraphT: L79 get() -> orElse(null)
org.jgrapht.graph.guava ImmutableValueGraphAdapter: L220 get() -> orElse(null)
org.jgrapht.graph.guava MutableValueGraphAdapter: L334 get() -> orElse(null)
org.jgrapht.nio.matrix MatrixExporter: L249 double cast
org.jgrapht.opt.graph.sparse.specifics IncomingNoReindexSparseDirectedSpecifics: L73 long cast
jgrapht-core/src/main/java/overview.html L4 Subsituted <b> by <strong>
jgrapht-unimi-dsi/src/main/java/org/jgrapht/sux4j/SuccinctDirectedGraph.java L125,134 long cast
jgrapht-core/src/main/java/org/jgrapht/alg/tour/PalmerHamiltonianCycle.java L131,133 commented assignments from self to self
jgrapht-core/src/main/java/org/jgrapht/generate/netgen/NetworkGeneratorConfig.java L292 long cast
jgrapht-core/src/main/java/org/jgrapht/traverse/ClosestFirstIterator.java L179 deleted hasNext() line that wasn't being used for anything
jgrapht-unimi-dsi/src/main/java/org/jgrapht/sux4j/SuccinctIntDirectedGraph.java L121,130 long cast
jgrapht-unimi-dsi/src/main/java/org/jgrapht/sux4j/SuccinctIntUndirectedDirectedGraph.java L121,130 long cast
jgrapht-unimi-dsi/src/main/java/org/jgrapht/sux4j/SuccinctUndirectedGraph.java L119,121 long cast
jgrapht-core/src/main/java/org/jgrapht/alg/shortestpath/IntVertexDijkstraShortestPath.java L273
jgrapht\jgrapht-core\src\main\java\org\jgrapht\traverse\ClosestFirstIterator.java -> L179
jgrapht-unimi-dsi/src/main/java/org/jgrapht/webgraph/ImmutableDirectedBigGraphAdapter.java -> L136
=> jgrapht-core/src/main/java/org/jgrapht/traverse/ClosestFirstIterator.java L179 deleted hasNext() line that wasn't being used for anything
New lines to enable SonarCloud analysis. As a pull request, it may not be useful
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.

None yet

2 participants