Thanks - applied with minor changes to the comments and mutex name.
On Fri, Sep 27, 2013 at 8:51 AM, Dinesh Kumar <[email protected] > wrote: > Hi Dave, > > Sorry, it's my bad. The patch is build on another branch. > > Please find the attached patch which has been built on the pgAdmin master > branch. > > Thanks in advance. > > > Dinesh > > -- > *Dinesh Kumar* > Software Engineer > > Ph: +918087463317 > Skype ID: dinesh.kumar432 > www.enterprisedb.co > <http://www.enterprisedb.com/>m<http://www.enterprisedb.com/> > * > Follow us on Twitter* > @EnterpriseDB > > Visit EnterpriseDB for tutorials, webinars, > whitepapers<http://www.enterprisedb.com/resources-community> and > more <http://www.enterprisedb.com/resources-community> > > > On Thu, Sep 26, 2013 at 7:54 PM, Dave Page <[email protected]> wrote: > >> >> >> >> On Tue, Sep 24, 2013 at 12:37 PM, Dinesh Kumar < >> [email protected]> wrote: >> >>> Hi Dave/Team, >>> >>> While working on one of the assigned task, i have been noticed the race >>> condition on object "currentObject", which is concurrently updating >>> by "onSelRightClick" and "execSelChange" methods. Due to this, sometimes, i >>> am getting "n" Node context menu on "m" node. >>> >>> Ex:- When i right click on "server node", it is showing the "database >>> node" popup menu options. >>> >>> I have been attaching the patch along with this request, and will be >>> awaiting for your inputs. >>> >> >> This doesn't apply - can you check the patch is correct please? >> >> -- >> Dave Page >> Blog: http://pgsnake.blogspot.com >> Twitter: @pgsnake >> >> EnterpriseDB UK: http://www.enterprisedb.com >> The Enterprise PostgreSQL Company >> > > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company
