From: "Ilpo Järvinen" <ilpo.jarvinen-pxSi+dnQzZMxHbG02/KK1g@public.gmane.org>
To: Zack Weinberg <zweinberg-4eJtQOnFJqFBDgjK7y7TUQ@public.gmane.org>
Cc: Arnaldo Carvalho de Melo
<acme-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
dwarves-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: Formatting "drivers" was Re: Can't persuade pahole to see through forward declarations
Date: Wed, 24 Jun 2009 09:58:43 +0300 (EEST) [thread overview]
Message-ID: <Pine.LNX.4.64.0906240954550.26611@melkinkari.cs.Helsinki.FI> (raw)
In-Reply-To: <20090623234018.61f1ce02-4eJtQOnFJqFBDgjK7y7TUQ@public.gmane.org>
[-- Attachment #1: Type: TEXT/PLAIN, Size: 1451 bytes --]
On Tue, 23 Jun 2009, Zack Weinberg wrote:
> "Ilpo Järvinen" <ilpo.jarvinen-pxSi+dnQzZMxHbG02/KK1g@public.gmane.org> wrote:
> > On Thu, 18 Jun 2009, Arnaldo Carvalho de Melo wrote:
> > > Em Thu, Jun 18, 2009 at 01:28:20PM -0700, Zack Weinberg escreveu:
> > > > - You do not have to process C declaration syntax to find the
> > > > name of each field.
> >
> > Good point, this is one of the most complex tasks in scripts to get
> > general case right. Besides typedefs, especially those function
> > pointers which appear in return value and arguments make it a task
> > too hairy for any sane people, not that I'd say it is
> > impossible... :-)
>
> I used to hack gcc. I know *exactly* how hard it is to parse C
> declarations. :-)
>
> I had been doing okay, for the limited thing I am trying to do, with
> sed scripts to munge the pahole output into something that could be
> relatively easily parsed by a Python script, but then I ran into this
> construct:
>
> struct S {
> ...
> struct T {
> int a;
> } tee[2];
> };
>
> No hint that there are two copies of T embedded in S, here, until it's
> far too late to do anything about it (if you're a sed script).
Agreed, those kind of constructs are quite annoying. I'd probably do it so
that I'd tac the input and preprocess it into something I can do line by
line after another tac (but that of course has some performance penalty
if the input is very large).
--
i.
next prev parent reply other threads:[~2009-06-24 6:58 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20090615172409.6f0f322b@mozilla.com>
[not found] ` <20090617170217.GB21530@ghostprotocols.net>
[not found] ` <20090617170217.GB21530-f8uhVLnGfZaxAyOMLChx1axOck334EZe@public.gmane.org>
2009-06-17 17:25 ` Can't persuade pahole to see through forward declarations Zack Weinberg
[not found] ` <20090617102506.34aaf8e2-4eJtQOnFJqFBDgjK7y7TUQ@public.gmane.org>
2009-06-17 20:56 ` Arnaldo Carvalho de Melo
2009-06-18 18:36 ` Arnaldo Carvalho de Melo
[not found] ` <20090618183634.GE21530-f8uhVLnGfZaxAyOMLChx1axOck334EZe@public.gmane.org>
2009-06-18 20:28 ` Zack Weinberg
[not found] ` <20090618132820.2eb7371a-4eJtQOnFJqFBDgjK7y7TUQ@public.gmane.org>
2009-06-18 20:50 ` Formatting "drivers" was " Arnaldo Carvalho de Melo
[not found] ` <20090618205053.GA4258-f8uhVLnGfZaxAyOMLChx1axOck334EZe@public.gmane.org>
2009-06-24 6:29 ` Ilpo Järvinen
[not found] ` <Pine.LNX.4.64.0906240918170.26611-tOJ5Hk0ALFH+1EwMtL+0ZheBoQapMCRCVQQcQy+6Uvc@public.gmane.org>
2009-06-24 6:40 ` Zack Weinberg
[not found] ` <20090623234018.61f1ce02-4eJtQOnFJqFBDgjK7y7TUQ@public.gmane.org>
2009-06-24 6:58 ` Ilpo Järvinen [this message]
2009-06-24 6:42 ` Zack Weinberg
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Pine.LNX.4.64.0906240954550.26611@melkinkari.cs.Helsinki.FI \
--to=ilpo.jarvinen-pxsi+dnqzzmxhbg02/kk1g@public.gmane.org \
--cc=acme-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
--cc=dwarves-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=zweinberg-4eJtQOnFJqFBDgjK7y7TUQ@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).