Subject: Re: [PATCH] ares_parse_txt_reply: add `record_start` field

Re: [PATCH] ares_parse_txt_reply: add `record_start` field

From: Fedor Indutny <fedor_at_indutny.com>
Date: Tue, 27 May 2014 14:01:12 +0400

Sure, here are attached changes.

Please let me know if I should improve them even further.

On Tue, May 27, 2014 at 12:39 PM, Jakub Hrozek <jhrozek_at_redhat.com> wrote:

> On Tue, May 27, 2014 at 12:53:41AM +0400, Fedor Indutny wrote:
> > Agreed, will look into implementing it.
>
> Would you consider also adding TTL when creating this "ares_parse_txt2"
> ?
>
> >
> >
> > On Tue, May 27, 2014 at 12:50 AM, Daniel Stenberg <daniel_at_haxx.se>
> wrote:
> >
> > > On Thu, 22 May 2014, Fedor Indutny wrote:
> > >
> > > Unfortunately, there is no certain description of how TXT records
> should
> > >> be used after parsing. Clearly some people may want to get each chunk
> of
> > >> each record separately and use them as they want to. Concatenation
> will
> > >> definitely make c-ares unusable for them.
> > >>
> > >
> > > An option is to introduce another function that can return this info
> and
> > > yet we maintain API and ABI backwards compatibility.
> > >
> > > --
> > >
> > > / daniel.haxx.se
> > >
>

Received on 2014-05-27