Skip to content
This repository has been archived by the owner on Jan 26, 2024. It is now read-only.

Misuses of multiple SQL statements in an execSQL() #625

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

lordpyre
Copy link

Hello,
I've spotted some misuses of multiple SQL statements in an execSQL(). In fact, the second INSERTs do not get executed by SQLite like this. Please consider reviewing and fixing it.
Best,
Csaba

@itadventurer
Copy link
Member

Hi @lordpyre,
Thank you for your pull request. It looks fine. Do you consider to improve Mirakel further or is it just a one-time PR?
If it is the latter we are still very grateful for your PR but as we do not maintain Mirakel anymore we would not publish a new version.
If you want to maintain Mirakel further we would be very happy to help you to get up to speed and support you wherever we can.

Best,
@weiznich and Anatoly

@lordpyre
Copy link
Author

Hi @weiznich and Anatoly,
I was testing a tool to analyze embedded SQL code in Android apps, when I found this issue in Mirakel. So this was a one-time PR now.
Thanks for your answer!
Csaba

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants