[jira] Created: (HARMONY-5926) [classlib] [linuxfont] LinuxNativeFont.c needs code review

["Mark Hindess (JIRA)" <ji...@apache.org>]

[classlib] [linuxfont] LinuxNativeFont.c needs code review
----------------------------------------------------------

                 Key: HARMONY-5926
                 URL: https://issues.apache.org/jira/browse/HARMONY-5926
             Project: Harmony
          Issue Type: Bug
          Components: Classlib
         Environment: linux
            Reporter: Mark Hindess


While looking at the issues in HARMONY-5914, I notice that there are a number of other issues with this code.  Specifically, there are a number of clauses like:

    if (!blah) {
        ...
        throwNPException(env, "Error during blah");
    }

where the exceptions are thrown but control flow continues regardless.  Obviously, the code should return after the exception is thrown to avoid causing more exceptions that would likely hide the root cause.

It is probably worth doing a review of this code since I suspect there are other improvements to be made.  I wonder however whether the test cases exercise this code sufficiently such that we can be sure any refactoring  doesn't break anything.


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.