- Status Closed
- Percent Complete
- Task Type Bugs
- Category Plugins
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Release 3.4
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#11169 - Unstable text viewer
The recent updates after revision 24897 to viewer.c (currently revision 25400) causes the following:
- Latin script files open normally, but on exit of viewer and returning to file menu, rockboxui terminates with a message: stack smashing detected
- When trying to open a file containing diacritics (Arabic), the file doesn’t open successfully and rockboxui terminates with the message:
Can’t open font: /.rockbox/fonts/6-GNU-Unifont.fnt
Floating point exception
Closed by tomers
2010-04-07 18:08
Reason for closing: Fixed
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
2010-04-07 18:08
Reason for closing: Fixed
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
Should be fixed in r25516. Reopen if
needed.
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
Can you please locate when these bugs were introduced with respect to the following commits?
Revision 25233-25235, by uchida: text viewer plugin applies patches
FS#8445,FS#9546,FS#9853,FS#9855,FS#9892,FS#9893,FS#9898,FS#9902, andFS#9990.Revision 25262, by torne: Prevent accidental skipping to beginning/end of file in text viewer.
Revision 25400, by me: Viewer: Implement alignment option, to provide alignment of RTL text (Hebrew, Arabic) to the right
P.S. Amr, please upload the two files that causes the symptoms you describe (the one with arabic diacritics and that with the latin script)
Thanks!
I've traced the updates revision by revision and here are my observations:
R 24897 is the last stable one that produces the expected results
R 25233 introduces unwanted new lines occasionally (the viewport could afford showing 2 words on one line in the former revision, here in some cases each word appears on a separate line)
R 25234 the same as the previous
R 25235 the same too
R 25262 the same too
R 25400 some files (attached README.txt, 1.txt) open normally but on viewer exit, rockboxui aborts with error:
While rockboxui aborts on opening other files (attached 2.txt, كلام مُشَكَّلٌ.txt) with error:
Notes:
- I updated svn to the latest revision and reverted my local changes in other files to isolate the problem
- I created a new simulator build and I simulate on target Sansa e200
- I've stated before that the first problem (stack smashing) relates to Latin characters and the second (floating point exception) relates to diacritics, but this isn't true as both problems happened with both types of scripts
Hopefully, all of the above mentioned issues have now been resolved. Please test r25516 (and up) and report your your findings.
Thanks