Postgres Pro
Downloads
Home   >   mailing lists

Re: Reg-[Sonarqube]CodeSmell issues - Mailing list pgadmin-hackers

From Akshay Joshi
Subject Re: Reg-[Sonarqube]CodeSmell issues
Date
Msg-id CANxoLDfz44NvUq_b=TAx1vpHh-U=VAJHkLYs3CxSN6FXQrphpw@mail.gmail.com
Whole thread Raw
In response to Reg-[Sonarqube]CodeSmell issues  (Satish V <satish.v@enterprisedb.com>)
List pgadmin-hackers
Tree view
Thanks, patch applied.

On Tue, Jun 16, 2020 at 12:49 PM Satish V <satish.v@enterprisedb.com> wrote:
Hi Hackers,

Please find the patch in the attachment which resolves below issues
1.Replace this if-then-else statement by a single return statement.(clumsy)
2."switch" statements should have at least 3 "case" clauses. (bad practise)
Kindly review and share the changes, if  required.


Thanks
Sathish V


--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: pgAdmin 4 commit: Fixed following issues reported by SonarQube:
Next
From: Yogesh Mahajan
Date:
Subject: [pgAdmin4][Patch] - Issue fixes reported by SonarQube -2