Re: Bug report: OIDs, roles and Windows (signedness issue?)

Поиск
Список
Период
Сортировка
От Ashesh Vashi
Тема Re: Bug report: OIDs, roles and Windows (signedness issue?)
Дата
Msg-id CAG7mmoxXv23GtsuM7034XgHHiqf07G1iUi4=dk18pYeWgFO=eQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Bug report: OIDs, roles and Windows (signedness issue?)  (Ian Lawrence Barwick <barwick@gmail.com>)
Ответы Re: Bug report: OIDs, roles and Windows (signedness issue?)  (Ian Lawrence Barwick <barwick@gmail.com>)
Список pgadmin-hackers
Dave,

The patch, sent by Ian Lawrence, looks good me.
I also found some other places, where the similar changes were required (specially in debugger code).
I've attached the combined patch (debugger changes and Ian's patch).

If nobody has any objection, I will check-in the patch on both REL-1_18_0_PATCHES and master branch.


On Fri, Jan 10, 2014 at 2:11 PM, Ian Lawrence Barwick <barwick@gmail.com> wrote:
2014/1/10 Ashesh Vashi <ashesh.vashi@enterprisedb.com>
Sorry - it was missed due to busy schedule.
I will take a look at it.

Thanks for the reminder.

No worries, thanks for replying so quickly.

Regards

Ian Barwick
 


On Fri, Jan 10, 2014 at 1:36 PM, Ian Lawrence Barwick <barwick@gmail.com> wrote:
2013/11/22 Ian Lawrence Barwick <barwick@gmail.com>:
> 2013/11/22 Ian Lawrence Barwick <barwick@gmail.com>:
>
>>>
>>> BTW other objects (tables, schemas etc.) with OIDs greater than 256^4/2 - 1
>>> don't seem to be affected by this.
>>
>> It's been a while since I've looked at the pgAdmin source, but some wild
>> grepping shows that in:
>>
>>   pgadmin/schema/pgRole.cpp
>>
>> the role OID is set with:
>>
>>  role->iSetOid(roles->GetLong(wxT("oid")));
>>
>> whereas almost everywhere else it's set with GetOid(), e.g.
>>
>>  table->iSetOid(tables->GetOid(wxT("oid")));
>>
>> which may or may not have something to do with the issue.
>
> If that is the problem, then here's a patch. Untested as I don't have a Windows
> build environment available, though I might be able to cobble one together
> next week.

Just wondering - has anyone been able to look into this? Unfortunately I still
don't have a Windows build environment available to test.


Regards

Ian Barwick


--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers



--
--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA:
Enterprise PostgreSQL Company

 

http://www.linkedin.com/in/asheshvashi





--
--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA:
Enterprise PostgreSQL Company

 

http://www.linkedin.com/in/asheshvashi

Вложения

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Akshay Joshi
Дата:
Сообщение: Re: Fixed some compiler warnings
Следующее
От: Ian Lawrence Barwick
Дата:
Сообщение: Re: Bug report: OIDs, roles and Windows (signedness issue?)