patchSimulavr: an AVR simulator - Patches: patch #7034, display frequency in fool-proof way

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #7034: display frequency in fool-proof way

Submitter:  Petr Hluzin <petrh>
Submitted:  Sun 20 Dec 2009 08:52:05 PM UTC
   
 
Category:  feature add Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  None Open/Closed:  Closed
Component Version:  simulavrxx

Tue 23 Mar 2010 09:35:16 PM UTC, comment #5: 

Correct. It is because the patch has been committed:
http://git.savannah.gnu.org/cgit/simulavr.git/commit/?id=9adbcbcfd2decab2c46248667a324cf0fec8f46d

(Used to be somewhere at http://github.com/onnokort/simulavrxx/tree/master/src/, now 404 for some reason.)

Petr Hluzin <petrh>
Group Member
Tue 23 Mar 2010 09:23:34 PM UTC, comment #4: 

I consider this closed as simulavr displays enough now in HEAD

Onno Kortmann <onnokort>
Group Member
Mon 21 Dec 2009 10:15:00 PM UTC, comment #3: 


> %1.4f might leave off a couple significant digits.
> I'd use %1.6f .


Are there crystals with that much digits?
Are they used?
Are they worth the clutter?
(I suppose no crystals with 0.1 Hz resolution exist.)
I do not mind the proposed change, though.

> Also, do we want to go back and forth between cout and printf?


I suppose we do not but I am not familiar with C++ streams and the printf style is more universally known anyway.

Petr Hluzin <petrh>
Group Member
Mon 21 Dec 2009 04:42:49 PM UTC, comment #2: 

A couple quibbles.
%1.4f might leave off a couple significant digits.
I'd use %1.6f .
Also, do we want to go back and forth between cout and printf?

Michael Hennebry <skeeve>
Group Member
Sun 20 Dec 2009 08:53:48 PM UTC, comment #1: 

(Rev 839, 858, 859 from my private repository.)

Petr Hluzin <petrh>
Group Member
Sun 20 Dec 2009 08:52:05 PM UTC, original submission:  

What is easier to tell right?
Is frequency 8000000 correct? Or: Is frequency 8.0000 MHz correct?
No zero-counting please!

Petr Hluzin <petrh>
Group Member

 

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

Attached Files
file #19315:  839-main-freq-print.patch added by petrh (474B - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by onnokort (Posted a comment)
  • -email is unavailable- added by skeeve (Posted a comment)
  • -email is unavailable- added by petrh (Submitted the item)
  •  

    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.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2010-03-23 onnokort StatusNone Done
        Open/ClosedOpen Closed
    2009-12-20 petrh Attached File- Added 839-main-freq-print.patch, #19315

    Back to the top

    Powered by Savane 3.13-bb6a.
    Corresponding source code