Thank you!

On Wed May06'26 05:19:06PM, Barry Scott wrote:
> From: Barry Scott <[email protected]>
> Date: Wed, 6 May 2026 17:19:06 +0100
> To: Community support for Fedora users <[email protected]>
> CC: Ranjan Maitra <[email protected]>
> Reply-To: Community support for Fedora users <[email protected]>
> Subject: Re: volume icon no longer works after F42 -> F43 upgrade
> 
> 
> 
> > On 3 May 2026, at 23:29, Ranjan Maitra via users 
> > <[email protected]> wrote:
> > 
> > Here is the error I get:
> > 
> > pulse_glue.c: In function ‘pulse_glue_sync_active_profile’:
> > pulse_glue.c:348:12: error: variable ‘index’ set but not used 
> > [-Werror=unused-but-set-variable=]
> >   348 |   uint32_t index = 0;
> >       |            ^~~~~
> > cc1: all warnings being treated as errors
> > make[2]: *** [Makefile:511: pa_applet-pulse_glue.o] Error 1
> > 
> > So, if I go in and remove it, I should (and do get) error because the 
> > variable is used two lines down as an argument to a function. So, I do not 
> > understand where this error of unused but set is coming from.
> 
> You have to remove index in 2 places:
> 
>     uint32_t index = 0;
>     for (GSList *entry = as->profiles; entry;entry = g_slist_next(entry), 
> ++index) {
> 
> You missed the ", ++index" that is not needed.
> 

Indeed, I should have used my mind:-( Thank you!

Btw, I did not notice earlier that this code was that old on github. It is 
likely unsupported given the unaddressed issues and pull requests.

I forked it and updated the file on github.

https://github.com/maitra/pa-applet

I wonder if Bob Marčan will be open to submitting it to the official repo. He 
wrote the spec file, after all, and that is a big contribution here to make it 
into a Fedora RPM.

Many thanks again and best wishes,
Ranjan

-- 
_______________________________________________
users mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/[email protected]
Do not reply to spam, report it: 
https://forge.fedoraproject.org/infra/tickets/issues/new

Reply via email to