On Thu, 23 Jan 2025 20:31:03 GMT, Phil Race <p...@openjdk.org> wrote:

>> Joachim Kern has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   remove old solution
>
> If absolutely nothing else needs fp_pipewire.h I guess that ought to be OK 
> ... but this is very new code.
> I'd like @azvegint to look at this as he may have had some reason that isn't 
> obvious.
> 
> I don't follow why you found it necessary to remove the include statement 
> from the .c file in the image case ?
> And I agree that there was actually nothing wrong with the existing code. The 
> compiler must have been confused.
> 
> And other than  *building* what testing was done with these changes ?

@prrace Is this OK for you now after Aleksandr has tested and approved the PR?

-------------

PR Comment: https://git.openjdk.org/jdk/pull/23236#issuecomment-2612112103

Reply via email to