[OpenBIOS] Update: Tokenizer Version 1.00.01 is on its way!

[David L Paktor]

Stefan Reinauer  wrote on Fri Aug 18 17:43:44 CEST
2006
> Stefan Reinauer  [060818 17:06]:
>
> . . . . .
>
> For a quick test I ran a coverage test with our new and shiny test suite
> and here's the results:
>
> Code covered:  95.9 %
>
> That is pretty damn good for a first run. Even if it was a safety
> critical applications.
>
> The whole coverage report can be checked here:
> http://openbios.org/~stepan/toke/coverage/fcode-utils/toke/index.html
>
> As you can clearly see most of the files only have their panic code
> uncovered. Which is pretty normal. Only whats wrong with strsubvocab.c
> ;-)
>
> David, thank you very much for this high quality contribution.
>
> Stefan

Stefan Reinauer  wrote on 09/05/2006 12:04:09 PM:
> Dear David,
>
> thank you for the conference call and the nice conversation.
>
> Please go ahead and send the next round of toke files with a
> bumped version number, so that people can see the progress being made.
>
> I will merge the changes into the new tree.
>
> Stefan

I have completed the changes we discussed and will be sending them to you
separately.

The coverage report identified several areas of "dead code" (most notably
in  strsubvocab.c ) as well as a few cases that weren't covered in the Test
Suite.

I changed the Test Suite to fill those gaps (there are now 167 tests, as
counted by the AutoExec script) and removed the dead routines (except for
one I believe should remain for possible future use -- read the top of the
Data Stack without popping it off).  I also fine-tuned some error detection
in "User-defined command-line compilation-control symbols" (when attempting
to access a defined value, distinguish between the case where no value is
assigned to the symbol versus where the symbol is altogether undefined; and
identify multiple definitions of the same symbol).

And, as we spoke, I bumped the version number to 1.0.01

I then ran my own coverage test; it showed a satisfactory improvement.  I
will not be sending you my results; your independent confirmation would be
preferable.

One other thing I'd like to request.  I think the Coverage Report is a Very
Good Thing:  it deserves to be displayed prominently, not just mentioned in
an item in the Mail Archives.  When you complete your coverage test, could
you open up an additional section --  like the sections for  "Unit-Test
Suite" and "Unit-Test Suite Logs" -- for it, at the end of the "OpenBIOS -
FCode suite" page?

Thanks.

-----

David L. Paktor                  System Firmware Developer
System and Technology Group      Global Firmware Division
[e-mail]              David L Paktor/Almaden/IBM@IBMUS

18880 Homestead Rd.              Building 9945
Cupertino CA 95014               Room 1026
408-342-6110                     T/L 560-6110

"The Bug Stops Here"