Re: PATCH: FB#15614: Use CALL for callable statements

Le 27/05/2010 12:49, Dave Page a écrit :
> On Thu, May 27, 2010 at 11:20 AM, Guillaume Lelarge
> <guillaume(at)lelarge(dot)info> wrote:
>> Le 27/05/2010 11:32, Dave Page a écrit :
>>> Thanks - patch applied.
>>>
>>> On Thu, May 27, 2010 at 10:22 AM, Ashesh Vashi
>>> <ashesh(dot)vashi(at)enterprisedb(dot)com> wrote:
>>>> Hi Dave,
>>>>
>>>> Please find the patch for the same.
>>>> This is valid for both trunk and REL-1_10 patch.
>>>>
>>
>> Are we sure that the dlgDirectDbg::invokeTargetCallable() method is only
>> called when connected to an EDB server? I didn't work a lot with the
>> debugger frontend source code, but it seems to me that this function can
>> be called even with vanilla PostgreSQL servers, especially on Windows
>> (see dlgDirectDbg::invokeTarget() method). Am I wrong?
>
> Hopefully :-)
>
> m_targetInfo->getIsFunction() should return false for anything other
> than an EDB-SPL stored procedure:
>
> #ifdef __WXMSW__
> if (!m_targetInfo->getIsFunction() &&
> PQiGetOutResult &&
> PQiPrepareOut &&
> PQiSendQueryPreparedOut)
> invokeTargetCallable();
> else
> #else
> #ifdef EDB_LIBPQ
> if (!m_targetInfo->getIsFunction())
> invokeTargetCallable();
> else
> #endif
> #endif
>

Oh OK, I didn't notice the getIsFunction() call.

--
Guillaume
http://www.postgresql.fr
http://dalibo.com