[Patch] Silence some compiler warnings.

David Daney ddaney@avtrex.com
Thu May 17 00:26:00 GMT 2007


Keith Seitz wrote:
> David Daney wrote:
> 
>> OK to commit?
>>
>>    * gnu/classpath/jdwp/natVMVirtualMachine.cc (jdwpSingleStepCB):
>>    Don't declare unused parameter jni_env.
> 
> You shouldn't need this one anymore. That should have been silenced by 
> my patch from earlier today:
> 
> 2007-05-16  Keith Seitz  <keiths@redhat.com>
> 
>     [snip]
>         (jdwpBreakpointCB): If the event list is not empty, send
>         whatever events are in it and the breakpoint event in a single
>         notification.
> -->     Mark parameter jni_env as MAYBE_UNUSED.
>     [snip]
> 
> Although I see that I marked it as belonging to jdwpBreakpointCB instead 
> of jdwpSingleStepCB...
> 

If the patch is approved, I will of course have to merge it with the 
current state of the trunk before it can be committed.  I will post a 
final diff if it differs from the original.

David Daney



More information about the Java-patches mailing list