Add a New Comment (Rich Markup)
Comment Type & Canned Response: None None > Multiple Canned Responses Fixed in development Crash with no stack trace Already fixed in newer version Fixed in stable Bad description Bad description and crash Bad stack trace Obsolete version Duplicate and not fixed Duplicate and needs more info Duplicate and fixed Need info and old
I just asked on discourse for the exact code that conflicts before reading jwe's new comment in this bug report. I am going to merge stable to default.
@Torsten: After this change I see a conflict when trying to merge stable to default and it's not clear to me what the proper resolution is. It seems that libgui/src/m-editor/file-editor-tab.cc is significantly different on stable and default now. Could you please merge stable to default and resolve the conflict?
Thanks for the quick fix. Works for me now.
Closing report.
This should be fixed with changeset http://hg.savannah.gnu.org/hgweb/octave/rev/7ea420f2c722
While changing the open mode from WriteOnly to ReadWrite in order to avoid cleared files in case of some write failures I missed the case where a shorter contents is written to a longer file.
When removing characters in a file using the built-in editor, the file isn't shrunk to the new size. Instead, the tail of the file is duplicated.
Steps to reproduce: 1. Create some file with the following content and save it as `test_editor_tail.m`:
This a file with arbitrary content to reproduce the error.
>> system('cat test_editor_tail.m') This a file with arbitrary content to reproduce the error. ans = 0
2. Delete some part of the file and save it again:
This a file to reproduce the error.
>> system('cat test_editor_tail.m') This a file to reproduce the error. o reproduce the error. ans = 0
For some reason, the file appears as expected in the built-in editor. But actually, the file size didn't change and the trailing characters remain at the end of the stored file.
"Force newline at end when saving file" and "Remove trailing spaces when saving file" is activated in case that should matter.
This is a regression from Octave 7.1.0 on the stable branch. Tested on Linux. But it's probably an issue on any platform.
CC'ing Torsten because he might hopefully have an idea of what might be going on.
(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)
Attach Files: Comment:
No files currently attached
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Only project members can vote.
Please enter the title of George Orwell's famous dystopian book (it's a date):
Follow 4 latest changes.
Copyright © 2022 Free Software Foundation, Inc. Verbatim copying and distribution of this entire article is permitted in any medium, provided this notice is preserved. The Levitating, Meditating, Flute-playing Gnu logo is a GNU GPL'ed image provided by the Nevrax Design Team. Source Code
Powered by Savane 3.9