Skip to content

Report RecordsAffected according to ADO.NET - #1289

Open
alaricd wants to merge 1 commit into
FirebirdSQL:masterfrom
pengdows:fix/records-affected-contract
Open

alaricd wants to merge 1 commit into
FirebirdSQL:masterfrom
pengdows:fix/records-affected-contract

Conversation

@alaricd

@alaricd alaricd commented Oct 20, 2026 •

Copy link
Copy Markdown

Fixes #1285.

RecordsAffected now starts at -1 and uses the statement type to count only INSERT, UPDATE, and DELETE statements; SELECT and other statements report -1. Adds regression coverage for SELECT.

Found by the pengdows ADO.NET conformance suite.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make FbDataReader.RecordsAffected follow the ADO.NET contract

1 participant