|
From: Bob R. <bo...@br...> - 2004-04-16 21:48:04
|
Hi, If I added another choice to the 8778== ---- Print suppression ? --- [Return/N/n/Y/y/C/c] ---- question, that was basically A/a, which would automatically have valgrind append the suppression into the suppression file, would it be accepted by the valgrind community? Or how about a command line option to always append suppressions to the file? Bob Rossi |
|
From: Nicholas N. <nj...@ca...> - 2004-04-16 21:58:22
|
On Fri, 16 Apr 2004, Bob Rossi wrote: > If I added another choice to the > 8778== ---- Print suppression ? --- [Return/N/n/Y/y/C/c] ---- > question, that was basically A/a, which would automatically have > valgrind append the suppression into the suppression file, would it be > accepted by the valgrind community? > > Or how about a command line option to always append suppressions to the > file? Madhu Kurup has a patch that does this, AIUI. I think this seems like the best option. Anyone else have comments, alternative suggestions? N |
|
From: Bob R. <bo...@br...> - 2004-04-16 21:59:26
|
On Fri, Apr 16, 2004 at 10:58:18PM +0100, Nicholas Nethercote wrote: > On Fri, 16 Apr 2004, Bob Rossi wrote: > > > If I added another choice to the > > 8778== ---- Print suppression ? --- [Return/N/n/Y/y/C/c] ---- > > question, that was basically A/a, which would automatically have > > valgrind append the suppression into the suppression file, would it be > > accepted by the valgrind community? > > > > Or how about a command line option to always append suppressions to the > > file? > > Madhu Kurup has a patch that does this, AIUI. I think this seems like the > best option. Anyone else have comments, alternative suggestions? Will it be in the next release? or should I bother him for the patch? Bob Rossi |
|
From: Madhu M K. <mm...@ya...> - 2004-04-16 22:46:19
|
Hi, Bob Rossi <bo...@br...> said on April 16,2004: > > > > Madhu Kurup has a patch that does this, AIUI. I think this seems > > like the best option. Anyone else have comments, alternative > > suggestions? > > Will it be in the next release? or should I bother him for the patch? Apologies for the slow turn around. I've added this into the bugzilla, viz: http://bugs.kde.org/show_bug.cgi?id=79787 The patch is attached to the bug, so you can use it with CVS head right away. Cheerio, M Madhu M Kurup /* Nemo Me Impune Lacessit */ mmk at yahoo-inc dt com |