Re: [pgAdmin][SonarQube] Shadowing variables - Mailing list pgadmin-hackers

From Akshay Joshi
Subject Re: [pgAdmin][SonarQube] Shadowing variables
Date
Msg-id CANxoLDeY2-8yrVyA6CAkQHKZBsk2-pNegthZmgu4GtsMtyksiA@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin][SonarQube] Shadowing variables  (Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>)
List pgadmin-hackers
Thanks, patch applied.

On Thu, Jul 16, 2020 at 3:11 PM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Hackers,

Attached is the patch to fix a few more SonarQube reported issues(around 28) related to "potential hiding of variables declared in an outer scope".
Please review.

On Tue, Jul 14, 2020 at 4:06 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Thanks, patch applied.

On Mon, Jul 13, 2020 at 10:15 AM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Hackers,

Attached is the patch to fix the SonarQube reported issues (40+) which potentially hides a variable declared in an outer scope.

Please review.

--
Thanks and Regards,
Aditya Toshniwal
pgAdmin Hacker | Sr. Software Engineer | EnterpriseDB India | Pune
"Don't Complain about Heat, Plant a TREE"


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


--
Regards,
Aditya Toshniwal
pgAdmin hacker | Sr. Software Engineer | edbpostgres.com
"Don't Complain about Heat, Plant a TREE"


--
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 code smell 'potential hiding of variables decla
Next
From: Dave Page
Date:
Subject: RM5554 - Fix runtime sizing