[patch #6405] Make multiple error messages display in a single dialog

[Ben Pfaff]

Follow-up Comment #2, patch #6405 (project pspp):

Wow, I'm glad you're paying attention.  I'm going to apply all of your
suggestions, except that I have a few follow-up comments:

>The condition of this if statement is always false, since 
>get_widget_assert, asserts that its return value is non NULL 

Well, sort of.  I actually tested this path, and if there's no widget named
"message", it just to use_fallback.  get_widget_assert doesn't assert, really,
it calls g_critical(), which by default just prints a message.

>Why do we have to drop any of the messages? They're in a scrolled 
>window, so the can all be viewed easily. If there's a huge number,
>then we could have two GtkTextArea widgets in a GtkvPaned, one 
>defaulting to the top, the other to the bottom of the textbuffer. 

The problem is that the amount of memory that could be consumed that way is
unlimited.  We support reading data files of unbounded size.  If each input
line produces an error, and we store each of those errors in memory, then we
will eventually run out of memory.

If there's a good way to display the contents of a text file of unbounded
size, we could use a file.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?6405>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/