diff options
author | Dylan Baker <baker.dylan.c@gmail.com> | 2013-04-30 08:16:55 -0700 |
---|---|---|
committer | Dylan Baker <baker.dylan.c@gmail.com> | 2013-05-31 14:11:20 -0700 |
commit | 90def8532e5f69c95debc7f21f55878f1678ea33 (patch) | |
tree | f09cfab8980182dc65c12e469b7b515b3a384976 /.gitignore | |
parent | 6b4f5fbdac11ff56fcdba06d56bf1ab1784b6f54 (diff) |
framework/summary.py: Adds new summary class and two support classes
This new summary class (currently NewSummary), is a complete redesign of
the summary generation system. Much of the code in summary was not
commented, and also not clear as to what it was doing, making it very
difficult to maintain and even more difficult to improve. The other
problem with the code was architecturally it was clobbered together,
doing some of the work in the summary.py file, and some of the work in
the piglit-summary-html.py file.
The new summary class does all of it's work in the summary.py file, and
from an API point of view, in one class. The summary class uses it's
constructor to build lists of passes, failures, skips, problems, etc.
and then uses methods to generate output. The reference implementation
of that kind of method is for generating HTML, but in theory it could be
used to generate anything: XML, csv lists, printed lists ala
piglit-summary, etc.
Also updates the .gitignore file to ignore some generated content
created by this class.
V3: - Corrects some spelling mistakes and some comments that had old
method/variable names
- renames several methods/functions
- moves sanitizePath to a toplevel function
- changes sanitizePath to use lambda per ken
- Moves buildDictionary to a helper function instead of private
method
V4: - Allows new summary generation to work with broken json.
Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
Reviewed-By: Aaron Watry <awatry@gmail.com>
Reviewed-By: Kenneth Graunke <kenneth@whitecape.org>
Diffstat (limited to '.gitignore')
-rw-r--r-- | .gitignore | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/.gitignore b/.gitignore index 65ece5da3..9180a6cc5 100644 --- a/.gitignore +++ b/.gitignore @@ -20,3 +20,5 @@ cmake_install.cmake CMakeCache.txt target_api tests/glslparsertest/glslcompiler + +.makotmp |