Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 364217 - Exception x.signum is not a function running mathLib.modf
Summary: Exception x.signum is not a function running mathLib.modf
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: EDT (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Kathy Carroll CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-19 08:50 EST by broy2 CLA
Modified: 2017-02-23 14:14 EST (History)
4 users (show)

See Also:

Attachments
fix for problem 1 (946 bytes, patch)
2011-12-29 00:55 EST, Huang Ji Yong CLA 		lasher: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description broy2 CLA 2011-11-19 08:50:28 EST 
This is an example in the EDT Help for mathLib.modf.
Use EUnit to run:

library modfRBD 
    y int; 
    x, result smallFloat;  
    function modfTest1(){@Test}
        x = 23.5678;
        try
            result = mathLib.modf(x, y);
        onException(oops AnyException)
            LogResult.failed("Exception");
            exit;
        end
        LogResult.assertFloatEqual("modf", 0.5678005218505859, result);
    end
    function modfTest2(){@Test}
        LogResult.assertFloatEqual("modf", 23, y);
    end
end 

eglx.javascript.JavaScriptObjectException x.signum is not a function

modfRBD::modfTest1: FAILED - Exception
modfRBD::modfTest2: FAILED - modf - Failed: Expected value = '23' Actual value = '0'
Comment 1 Jing Qian CLA 2011-11-21 20:15:29 EST 
deferred
Comment 2 Jing Qian CLA 2011-11-21 20:16:22 EST 
Brenda, if this is not in eunit.test project in cvs, please add to it and commit the changes, thanks, so it will be regression tested everytime we run eunit.test
Comment 3 broy2 CLA 2011-11-22 15:45:40 EST 
Done.
Comment 4 Huang Ji Yong CLA 2011-12-29 00:55:29 EST 
Created attachment 208838 [details]
fix for problem 1
Comment 5 Huang Ji Yong CLA 2011-12-29 00:56:34 EST 
The exception problem is fixed.
The second problem is the same as bug 364216. Waiting for the discussion.
Comment 6 Matt Heitz CLA 2012-01-13 17:17:12 EST 
Ji Yong is right.  The remaining problem is the same as 364216.  We ought to take care of that problem using 364216, and close this one since the exception has been fixed.
Comment 7 Huang Ji Yong CLA 2012-01-15 20:34:31 EST 
Route to Kathy to change the testcase.
If any problems, reopen it for me, thanks.
Comment 8 Kathy Carroll CLA 2012-02-24 10:53:38 EST 
The testcases using MODF have been modified and released to CVS.  The modified variations are in project org.eclipse.edt.eunit.test: api.SystemTypes.mathLibFunctions and fvt.librarySupport.TestMathLib2Lib.

The exception issue has been resolved.
Comment 9 Kathy Carroll CLA 2012-02-24 10:54:22 EST 
Verified and closed with 02/22 build