This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Patch: natMethod fix


>>>>> "Anthony" == Anthony Green <green@cygnus.com> writes:

Anthony> I found this omission (handling of FFI_TYPE_POINTER) while
Anthony> running rhino.  I've also moved the constructor test above
Anthony> the switch as a minor optimization.

Anthony> Ok for branch and trunk?

Yes, thanks.

Anthony> +   else
Anthony> +     switch (rtype->type)
Anthony> +       {

I think there should be braces around the switch:

    else
      {
        switch (...)

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]