Skip to content

ReplaceBinaryOperator Mutation of static numbrer generation is unnecessary #597

Description

@liquidpele

Given a lines such as this:

# mutated by things like core/ReplaceBinaryOperator_Mul_Add
URL_EXPIRATION_SECONDS = 60 * 60 * 3

Changing initial values for constants holds very little value. I think the system needs to be a little smarter about this, only only mutate if the numbers are being multiplied with or compared to a non-number.

Example:

URL_EXPIRATION_SECONDS = 60 * 60 * some_other_logic()

mutate between the 60 and some_other_logic call, but not the 60 * 60. While still not ideal, this would at least limit mutations to things that might have more value without wasting time and effort on what amounts to just changing a single number.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions