Re: SQLSetStmtAttr()+SQL_ATTR_ROWS_FETCHED_PTR

Nick Gorham wrote:

> George Neill wrote:
>
>> On Wed, Feb 4, 2009 at 3:32 AM, Nick Gorham 
>> <[email protected]> wrote:
>>  
>>
>>> George Neill wrote:
>>>
>>>    
>>>
>>>> Nick,
>>>>
>>>> On Tue, Feb 3, 2009 at 5:10 AM, Nick Gorham <[email protected]> wrote:
>>>>
>>>>      
>>>>
>>>>> George Neill wrote:
>>>>>
>>>>>
>>>>>        
>>>>>
>>>>>> On Tue, Feb 3, 2009 at 2:11 AM, Nick Gorham <[email protected]> 
>>>>>> wrote:
>>>>>>
>>>>>>
>>>>>>          
>>>>>>
>>>>>>> George Neill wrote:
>>>>>>>
>>>>>>>
>>>>>>>            
>>>>>>>
>>>>>>>> Hi All,
>>>>>>>>
>>>>>>>> I am having a little bit of confusion with SQLSetStmtAttr() and
>>>>>>>> SQL_ATTR_ROWS_FETCHED_PTR.  From this documentation,
>>>>>>>> http://www.unixodbc.org/doc/ODBC64.html, it states the 
>>>>>>>> Value/ValuePtr
>>>>>>>> should be of type SQLULEN ... but the actual code seems to use
>>>>>>>> SQLUINTEGER.  Is this a bug in the docs or the code?
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>               
>>>>>>>
>>>>>>> I would guess its a problem in the code, which bit of code in
>>>>>>> particular?
>>>>>>> so
>>>>>>> I can fix it.
>>>>>>>
>>>>>>>
>>>>>>>             
>>>>>>
>>>>>> SQLSetStmtAttr.c:753
>>>>>>
>>>>>> 2.2.15pre
>>>>>>
>>>>>> Thanks,
>>>>>> George
>>>>>>
>>>>>>
>>>>>>           
>>>>>
>>>>> Ok, try the 2.2.15pre now.
>>>>>
>>>>>         
>>>>
>>>> diff -r unixODBC-2.2.15pre.old/DriverManager/SQLSetStmtAttr.c
>>>> unixODBC-2.2.15pre.new/DriverManager/SQLSetStmtAttr.c
>>>> 756c756
>>>> <         statement -> row_ct_ptr = (SQLUINTEGER*) value;
>>>> ---
>>>>
>>>>      
>>>>
>>>>>       statement -> row_ct_ptr = (SQLULEN*) value;
>>>>>
>>>>>         
>>>>
>>>> All looks good, except I think you may have missed this one!   I will
>>>> give it a test drive here in a bit.
>>>>
>>>> Thanks
>>>> George.
>>>> _______________________________________________
>>>> unixODBC-dev mailing list
>>>> [email protected]
>>>> http://mailman.unixodbc.org/mailman/listinfo/unixodbc-dev
>>>>
>>>>
>>>>
>>>>       
>>>
>>> Ok, that bits done now.
>>>     
>>
>>
>> in SQLSetStmtAttr.c:694
>>         statement -> fetch_bm_ptr = (SQLUINTEGER*) value;
>>
>> Also should be changed to SQLULEN?
>>
>> Later,
>> George.
>>   
>
> Woops, not sure how I missed that
>
Checking, I don't think I did, my copy of SQLSetStmtAttr.c has been fixed.

-- 
Nick

-- 
Nick Gorham
Easysoft Limited
http://www.easysoft.com, http://www.unixODBC.org

_______________________________________________
unixODBC-dev mailing list
[email protected]
http://mailman.unixodbc.org/mailman/listinfo/unixodbc-dev