All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tomasz Bursztyka <tomasz.bursztyka@linux.intel.com>
To: ell@lists.01.org
Subject: Re: [RFC] genl: Add a message builder API to help creating complex nl messages
Date: Wed, 25 Mar 2015 13:04:34 +0200	[thread overview]
Message-ID: <55129642.9020105@linux.intel.com> (raw)
In-Reply-To: <55115AB9.9070603@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 927 bytes --]

Hi Denis,

> I sent my own take on this.  I was playing around with it yesterday.

Which is the exact same thing as I first proposed in "[PATCH] genl: Add 
utility function no append nested attribute into a message"
plus the nested of nested stuff I added in the other one via the fixed 
table I advised also (on which I am still waiting feedback). That's fine.
But such API was requested to be moved into the "builder" type of thing 
like DBus does.

> The only thing I'm not sure about is whether the nested message length 
> is aligned or not.  Looking at libnl code it seems to be always 
> aligned, but if someone can whip up a quick unit test, that would be 
> really helpful. 

Seriously?


So guys what's the decision then: which API is the right one here?
Can we clear this up once and for all? There are dependent code that 
cannot proceed until this is over.
This is a show-stopper.

Tomasz

      reply	other threads:[~2015-03-25 11:04 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-24 11:21 [RFC] genl: Add a message builder API to help creating complex nl messages Tomasz Bursztyka
2015-03-24 12:38 ` Denis Kenzior
2015-03-25 11:04   ` Tomasz Bursztyka [this message]

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=55129642.9020105@linux.intel.com \
    --to=tomasz.bursztyka@linux.intel.com \
    --cc=ell@lists.01.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.