Coccinelle archive on lore.kernel.org
 help / Atom feed
* [Cocci] "llu" integer literal suffix confuses coccinelle
@ 2018-12-06  0:03 Timur Tabi
  2018-12-06  6:03 ` Julia Lawall
  2018-12-06  6:30 ` Julia Lawall
  0 siblings, 2 replies; 3+ messages in thread
From: Timur Tabi @ 2018-12-06  0:03 UTC (permalink / raw)
  To: cocci

This line is rejected by coccinelle:

            NvU64      barSize   = pBus->PciBarSizes[i] * 0x100000llu;

If I change the "llu" to "ull", it works.  However, "llu" appears to
be valid C, so it should be accepted.  Or at least, gcc accepts it.  A
google search says that it's new in C++14.
_______________________________________________
Cocci mailing list
Cocci@systeme.lip6.fr
https://systeme.lip6.fr/mailman/listinfo/cocci

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Cocci] "llu" integer literal suffix confuses coccinelle
  2018-12-06  0:03 [Cocci] "llu" integer literal suffix confuses coccinelle Timur Tabi
@ 2018-12-06  6:03 ` Julia Lawall
  2018-12-06  6:30 ` Julia Lawall
  1 sibling, 0 replies; 3+ messages in thread
From: Julia Lawall @ 2018-12-06  6:03 UTC (permalink / raw)
  To: Timur Tabi; +Cc: cocci



On Wed, 5 Dec 2018, Timur Tabi wrote:

> This line is rejected by coccinelle:
>
>             NvU64      barSize   = pBus->PciBarSizes[i] * 0x100000llu;
>
> If I change the "llu" to "ull", it works.  However, "llu" appears to
> be valid C, so it should be accepted.  Or at least, gcc accepts it.  A
> google search says that it's new in C++14.

OK, so problem.  I will try to add it.

julia

> _______________________________________________
> Cocci mailing list
> Cocci@systeme.lip6.fr
> https://systeme.lip6.fr/mailman/listinfo/cocci
>
_______________________________________________
Cocci mailing list
Cocci@systeme.lip6.fr
https://systeme.lip6.fr/mailman/listinfo/cocci

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Cocci] "llu" integer literal suffix confuses coccinelle
  2018-12-06  0:03 [Cocci] "llu" integer literal suffix confuses coccinelle Timur Tabi
  2018-12-06  6:03 ` Julia Lawall
@ 2018-12-06  6:30 ` Julia Lawall
  1 sibling, 0 replies; 3+ messages in thread
From: Julia Lawall @ 2018-12-06  6:30 UTC (permalink / raw)
  To: Timur Tabi; +Cc: cocci



On Wed, 5 Dec 2018, Timur Tabi wrote:

> This line is rejected by coccinelle:
>
>             NvU64      barSize   = pBus->PciBarSizes[i] * 0x100000llu;
>
> If I change the "llu" to "ull", it works.  However, "llu" appears to
> be valid C, so it should be accepted.  Or at least, gcc accepts it.  A
> google search says that it's new in C++14.

This is fixed.  Thanks for the report.

julia


> _______________________________________________
> Cocci mailing list
> Cocci@systeme.lip6.fr
> https://systeme.lip6.fr/mailman/listinfo/cocci
>
_______________________________________________
Cocci mailing list
Cocci@systeme.lip6.fr
https://systeme.lip6.fr/mailman/listinfo/cocci

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, back to index

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-06  0:03 [Cocci] "llu" integer literal suffix confuses coccinelle Timur Tabi
2018-12-06  6:03 ` Julia Lawall
2018-12-06  6:30 ` Julia Lawall

Coccinelle archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/cocci/0 cocci/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 cocci cocci/ https://lore.kernel.org/cocci \
		cocci@systeme.lip6.fr cocci@archiver.kernel.org
	public-inbox-index cocci


Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/fr.lip6.systeme.cocci


AGPL code for this site: git clone https://public-inbox.org/ public-inbox