Skip to content

Fix/Issue on can not commit screenText data into database - #38

Merged
tinazhang128 merged 2 commits into
awareframework:betafrom
EcZww:fix/screentext-accessibility-issues
May 4, 2024
Merged

Fix/Issue on can not commit screenText data into database#38
tinazhang128 merged 2 commits into
awareframework:betafrom
EcZww:fix/screentext-accessibility-issues

Conversation

@EcZww

Copy link
Copy Markdown
Collaborator

Hotfix for #36

  • Fix error that reset the textbuffer on app changes

  • Replace ArrayList with HashSet for textBuffer to improve duplicate check performance

  • clear contentBuffer and textBuffer and handling logic to commit data into database specifically on app change

EcZww added 2 commits April 21, 2024 23:45
Resolved an issue where the text buffer was incorrectly reset upon every foreground app change, leading to potential loss of relevant data. Introduced a new buffer, textBuffer_app_change, dedicated to managing text hashes specifically during app transitions.
- Replace ArrayList with HashSet for textBuffer to improve duplicate check performance
- Streamline contentBuffer flushing and handling logic to commit changes specifically on app change
@EcZww
EcZww requested a review from tinazhang128April 22, 2024 02:11
@EcZwwEcZww self-assigned this Apr 22, 2024
@tinazhang128
tinazhang128 merged commit 879a304 into awareframework:betaMay 4, 2024
Sign up for freeto 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.

2 participants

@EcZww@tinazhang128