Skip to content

False positive for Style/RedundantParentheses cop #14228

Closed
@ydakuka

Description

@ydakuka

Describe

class XyzService
  def account_today
    service.estimate_value(
      (yield(dates[yesterday.to_s]) || 0 rescue 0),
      (yield(dates[today.to_s])     || 0 rescue 0),
    ).to_i
  end
end

After running rubocop -A, the syntax of my code was broken:

class XyzService
  def account_today
    service.estimate_value(
      yield(dates[yesterday.to_s]) || 0 rescue 0,
      yield(dates[today.to_s])     || 0 rescue 0,
    ).to_i
  end
end

Rubocop

1.75.8 (using Parser 3.3.8.0, rubocop-ast 1.44.1, analyzing as Ruby 2.7, running on ruby 2.7.8) [x86_64-linux]
  - rubocop-capybara 2.22.1
  - rubocop-factory_bot 2.27.1
  - rubocop-migration 0.7.1
  - rubocop-performance 1.25.0
  - rubocop-rails 2.32.0
  - rubocop-rake 0.7.1
  - rubocop-rspec 3.6.0
  - rubocop-rspec_rails 2.31.0
  - rubocop-thread_safety 0.7.2

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions