check syntax does not group errors by file

Bug #936705 reported by Curtis Hovey
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Gedit Developer Plugins
Fix Released
High
Curtis Hovey

Bug Description

The recent change to append syntax errors in a background process add the file name/path for each error instead once. When checking one file, The file should only be listed once. When checking multiple files, each file should be listed once, and expanding the file will reveal the issues.

There are two issues with the code. The checker must set reporter.piter to None before each file to indicate that a new file is being appended. _message_file_lines() must append the file in real time. IT cannot queue the append because the piter must be known at that very moment to ensure all subsequent calls know their file.

Tags: gdpformat
Curtis Hovey (sinzui)
Changed in gdp:
status: In Progress → Fix Committed
Curtis Hovey (sinzui)
Changed in gdp:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.