Re: Weirdness in parsing cpp macros

2024-03-21 Thread Jani Nikula
On Thu, 21 Mar 2024, Julia Lawall wrote: > On Wed, 20 Mar 2024, Jani Nikula wrote: > >> On Wed, 20 Mar 2024, Julia Lawall wrote: >> > On Wed, 20 Mar 2024, Jani Nikula wrote: >> >> Okay, I have another one wrt macros. :) >> >> >> >> I'm trying to add a completely new variadic macro, but it fails

✗ Fi.CI.BUILD: failure for Weirdness in parsing cpp macros

2024-03-21 Thread Patchwork
== Series Details == Series: Weirdness in parsing cpp macros URL : https://patchwork.freedesktop.org/series/131409/ State : failure == Summary == Error: patch https://patchwork.freedesktop.org/api/1.0/series/131409/revisions/1/mbox/ not applied Applying: Weirdness in parsing cpp macros

Re: Weirdness in parsing cpp macros

2024-03-21 Thread Julia Lawall
On Wed, 20 Mar 2024, Jani Nikula wrote: > On Wed, 20 Mar 2024, Julia Lawall wrote: > > On Wed, 20 Mar 2024, Jani Nikula wrote: > >> Okay, I have another one wrt macros. :) > >> > >> I'm trying to add a completely new variadic macro, but it fails at > >> "...". I've tried all sorts of things,

Re: Weirdness in parsing cpp macros

2024-03-20 Thread Julia Lawall
On Wed, 20 Mar 2024, Jani Nikula wrote: > On Wed, 20 Mar 2024, Julia Lawall wrote: > > On Wed, 20 Mar 2024, Jani Nikula wrote: > >> Okay, I have another one wrt macros. :) > >> > >> I'm trying to add a completely new variadic macro, but it fails at > >> "...". I've tried all sorts of things,

Re: Weirdness in parsing cpp macros

2024-03-20 Thread Jani Nikula
On Wed, 20 Mar 2024, Julia Lawall wrote: > On Wed, 20 Mar 2024, Jani Nikula wrote: >> Okay, I have another one wrt macros. :) >> >> I'm trying to add a completely new variadic macro, but it fails at >> "...". I've tried all sorts of things, but can't seem to be able to add >> a literal "...". >>

Re: Weirdness in parsing cpp macros

2024-03-20 Thread Julia Lawall
On Wed, 20 Mar 2024, Jani Nikula wrote: > On Wed, 20 Mar 2024, Ville Syrjälä wrote: > > On Wed, Mar 20, 2024 at 02:24:08PM +0100, Julia Lawall wrote: > >> > >> > >> On Wed, 20 Mar 2024, Ville Syrjälä wrote: > >> > >> > Hi Julia et al, > >> > > >> > In Linux drm/i915 driver

Re: Weirdness in parsing cpp macros

2024-03-20 Thread Jani Nikula
On Wed, 20 Mar 2024, Ville Syrjälä wrote: > On Wed, Mar 20, 2024 at 02:24:08PM +0100, Julia Lawall wrote: >> >> >> On Wed, 20 Mar 2024, Ville Syrjälä wrote: >> >> > Hi Julia et al, >> > >> > In Linux drm/i915 driver (drivers/gpu/drm/i915/display/intel_pps.[ch]) >> > we have a magic macro like

Re: Weirdness in parsing cpp macros

2024-03-20 Thread Ville Syrjälä
On Wed, Mar 20, 2024 at 02:24:08PM +0100, Julia Lawall wrote: > > > On Wed, 20 Mar 2024, Ville Syrjälä wrote: > > > Hi Julia et al, > > > > In Linux drm/i915 driver (drivers/gpu/drm/i915/display/intel_pps.[ch]) > > we have a magic macro like this: > > > > #define with_intel_pps_lock(dp, wf) \ >

Re: Weirdness in parsing cpp macros

2024-03-20 Thread Julia Lawall
On Wed, 20 Mar 2024, Ville Syrjälä wrote: > Hi Julia et al, > > In Linux drm/i915 driver (drivers/gpu/drm/i915/display/intel_pps.[ch]) > we have a magic macro like this: > > #define with_intel_pps_lock(dp, wf) \ Did you try declaring: iterator name with_intel_pps_lock; up with the

Weirdness in parsing cpp macros

2024-03-20 Thread Ville Syrjälä
Hi Julia et al, In Linux drm/i915 driver (drivers/gpu/drm/i915/display/intel_pps.[ch]) we have a magic macro like this: #define with_intel_pps_lock(dp, wf) \ for ((wf) = intel_pps_lock(dp); (wf); (wf) = intel_pps_unlock((dp), (wf))) which we can then use like so: ...