bugGlobulation 2 - Bugs: bug #27313, LibUsl TODO (num1's branch...

 
 

bug #27313: LibUsl TODO (num1's branch Boost::Test)

Submitter:  None
Submitted:  Tue 25 Aug 2009 05:01:12 AM UTC
   
 
Severity:  3 - Normal Status:  Need Info
Assigned to:  None Open/Closed:  Open
Release:  None Operating System:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 03 Nov 2011 04:57:50 AM UTC, comment #1: 

I don't understand a word. As far as I can see there is no file called UslHarness.cpp. I'm confused.

Leo Wandersleb <giszmo>
Group Member
Tue 25 Aug 2009 05:01:12 AM UTC, original submission:  

UslHarness.cpp
- Remove the output of executing code. e.g. "NativeValue<int>(0x807a5a0)" and replace it with a fleshed out version of std::string runResult(Value* Value);
- Format errors nicely, say where the error was and a rough idea of what's wrong
 - Probably extend Position or Debug to hold this information
 - Don't assert() and throw exceptions based on usl input, only based on the interpreters' handling of such.
- Make target binary "usl", as opposed to "Usl". Command names should be all lower-case.

General
- Add copyright notices to all new files.
- Make Usl obey order of operations (Multiplication comes before division.) Talk to main usl dev before doing this though, it might be advantageous to have a clean, forth-like stack-based system. (Depending on the decision, remove OrderOfOperations Unit Test)
- Add an assert() statement to usl. This would allow us to write a functional test handler that gives a helpful message should a assert(false) happen.
- Unit Test more classes than Position and Usl.

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by giszmo (Posted a comment)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2011-11-03 giszmo StatusNone Need Info

    Back to the top

    Powered by Savane 3.13-3230.
    Corresponding source code