Re: [PATCH] Documentation: Explain a second alternative for multi-line macros.

  #define setcc(cc) ({ \
    partial_status &= ~(SW_C0|SW_C1|SW_C2|SW_C3); \
    partial_status |= (cc) & (SW_C0|SW_C1|SW_C2|SW_C3); })

This _does_ return a value though, bad example.

Where does it return a value?

partial_status |=

I don't see any uses of it

Ah, that's a separate thing -- it returns a value, it's just
never used.

And with a small change to put it inside a do-while block
instead of ({ ... }), it at least builds cleanly.

Well please replace it then, statement expressions should be
avoided where possible (to start with, they don't have well-
defined semantics).


Segher

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Follow-Ups:
  • Re: [PATCH] Documentation: Explain a second alternative for multi-line macros.
    • From: Randy Dunlap <[email protected]>

• References:
  • [PATCH] Documentation: Explain a second alternative for multi-line macros.
    • From: "Robert P. J. Day" <[email protected]>
  • Re: [PATCH] Documentation: Explain a second alternative for multi-line macros.
    • From: Muli Ben-Yehuda <[email protected]>
  • Re: [PATCH] Documentation: Explain a second alternative for multi-line macros.
    • From: "Robert P. J. Day" <[email protected]>
  • Re: [PATCH] Documentation: Explain a second alternative for multi-line macros.
    • From: Segher Boessenkool <[email protected]>
  • Re: [PATCH] Documentation: Explain a second alternative for multi-line macros.
    • From: Randy Dunlap <[email protected]>

• Prev by Date: Re: [PATCH] Open Firmware device tree virtual filesystem
• Next by Date: Re: [PATCH] Make JFFS depend on CONFIG_BROKEN
• Previous by thread: Re: [PATCH] Documentation: Explain a second alternative for multi-line macros.
• Next by thread: Re: [PATCH] Documentation: Explain a second alternative for multi-line macros.
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]