Skip to content

Replace java.util.Stack with Kotlin's ArrayDeque implementation #5760

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

Merged
merged 1 commit into from
Feb 5, 2023

Conversation

3flex
Copy link
Member

@3flex 3flex commented Feb 5, 2023

From java.util.Stack Javadoc:

A more complete and consistent set of LIFO stack operations is provided by the Deque interface and its implementations, which should be used in preference to this class.

I've used Kotlin's implementation instead of a Java implementation.

@codecov
Copy link

codecov bot commented Feb 5, 2023

Codecov Report

Merging #5760 (52baad9) into main (488d0b8) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##               main    #5760   +/-   ##
=========================================
  Coverage     84.52%   84.52%           
  Complexity     3731     3731           
=========================================
  Files           544      544           
  Lines         12759    12759           
  Branches       2232     2232           
=========================================
  Hits          10785    10785           
  Misses          864      864           
  Partials       1110     1110           
Impacted Files Coverage Δ
...h/detekt/core/baseline/IndentingXMLStreamWriter.kt 56.33% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@schalkms schalkms added this to the 1.23.0 milestone Feb 5, 2023
@3flex 3flex merged commit d3c97f6 into detekt:main Feb 5, 2023
@3flex 3flex deleted the drop-stack branch February 5, 2023 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants