------- Comment #2 from nickc at redhat dot com 2007-03-29 11:11 ------- Hi Brooks,
> One difference that I see between our patches -- you have: > + if (* descrip_extra == '\0') > + descrip_extra = lang_names [i]; > whereas mine just unconditionally assigns descrip_extra to > lang_names[i]. Is there are reason you made this conditional? Not really, just a hunch. I felt that the first language name set would probably be the most "mainstream" of the possible language names that might be displayed. Of course I did not actually go so far as to check this, and your version of the patch is simpler and therefore better. Cheers Nick -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31356
