Empty try-catch block

[marjanjavid]

@Karvan If we have a block like below picture, we should warn about redundant code. Now we just warn about empty finally block.

@Karvan Do you agree to add this rule?

[Karvan]

@marjanjavid What is redundant here?

[marjanjavid]

@Karvan the empty try-catch is redundant. It doesn't do anything. just redundant lines of codes

[Karvan]

@Foroughi Let's do this.

[Foroughi]

@marjanjavid @Karvan Isn`t it better instead of we raise a warning for empty try-catch block , we raise a warning for every empty block not just try-catch block ?

[marjanjavid]

@Foroughi I am agree with you. The programmer can put comment in empty blocks to avoid get warning. Please note that we have rules for empty classes, empty methods and empty constructor and empty else clauses in GCop.

[azadimehdi]

@Karvan Do we need to add any fix providers for this case or only a warning would be enough?

[Karvan]

@azadimehdi

• let's update this rule to check any empty block, not just try-catch.
• for redundant try-catch we can have another role, as a new feature.