diff options
| author | Denys Vlasenko <dvlasenk@redhat.com> | 2011-02-10 15:35:06 +0100 |
|---|---|---|
| committer | Denys Vlasenko <dvlasenk@redhat.com> | 2011-02-10 15:35:06 +0100 |
| commit | f2cc43b93920a7e32c5c31f4ca5171691226ee71 (patch) | |
| tree | 157415e6b265e0283068e7e9084a41724ce0bbf2 | |
| parent | 2ff1d396e701a56cd796828f83a51059b7bad9ba (diff) | |
| download | abrt-f2cc43b93920a7e32c5c31f4ca5171691226ee71.tar.gz abrt-f2cc43b93920a7e32c5c31f4ca5171691226ee71.tar.xz abrt-f2cc43b93920a7e32c5c31f4ca5171691226ee71.zip | |
updated doc/TODO_crash_dump_fields
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
| -rw-r--r-- | doc/TODO_crash_dump_fields | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/doc/TODO_crash_dump_fields b/doc/TODO_crash_dump_fields index 7a8407fb..a35b3b04 100644 --- a/doc/TODO_crash_dump_fields +++ b/doc/TODO_crash_dump_fields @@ -47,3 +47,21 @@ printable_form_of_crash_item(...) dd_load_text_ext(dd, "time", flags) may benefit from a new flag DD_LOAD_TEXT_PRINTABLE, which makes it generate printable representation of fields. + + + Reduce amount of special-casing in the code which works with crash_data + +Examples: + +Reporters use something like this to exclude "non-interesting" fields: + +for (each crash_data element) +{ + if (strcmp(short_name, FILENAME_COUNT) == 0) goto next; + if (strcmp(short_name, CD_DUMPDIR) == 0) goto next; + if (strcmp(short_name, FILENAME_INFORMALL) == 0) goto next; + if (strcmp(short_name, FILENAME_MESSAGE) == 0) goto next; // plugin's status message (if we already reported it yesterday) + ... +} + +Can it be done with a flag? |