From: SourceForge.net <no...@so...> - 2005-10-28 14:24:54
|
Patches item #1340260, was opened at 2005-10-28 05:22 Message generated for change (Settings changed) made by dkf You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=310894&aid=1340260&group_id=10894 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. >Category: 44. Parsing and Eval Group: None Status: Open Resolution: None Priority: 5 Submitted By: Mo DeJong (mdejong) Assigned to: Kevin B KENNY (kennykb) Summary: Simplify round() implementation in expr cmd Initial Comment: I was just reviewing the round() implementation in Tcl 8.4 vs Tcl 8.5. The round() implementation seems to be more complex than it needs to be. The attached patch simplifies the function so that it works more like the 8.4 version. I also added tests cases around the min int and max int cases. As far and I can tell, the results of no tests are changed by this patch. Kevin, could you review this patch before I apply it? ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=310894&aid=1340260&group_id=10894 |