DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH] doc: remove reference to pcapng init function
@ 2022-05-27 13:21 Cian Ferriter
  2022-05-27 15:38 ` Stephen Hemminger
  0 siblings, 1 reply; 3+ messages in thread
From: Cian Ferriter @ 2022-05-27 13:21 UTC (permalink / raw)
  To: Reshma Pattan, Stephen Hemminger; +Cc: dev, Cian Ferriter

The rte_pcapng_init function doesn't exist, so remove it from the docs.

Also fix minor mistakes in the file.

Signed-off-by: Cian Ferriter <cian.ferriter@intel.com>

---

The rte_pcapng_init function existed in earlier versions of patches for
this file on the mailing list, but was dropped in favour of
RTE_INIT(pcapng_init) which doesn't need to be called explicitly by the
programmer, if I understand the RTE_INIT macro correctly.
---
 doc/guides/prog_guide/pcapng_lib.rst | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/doc/guides/prog_guide/pcapng_lib.rst b/doc/guides/prog_guide/pcapng_lib.rst
index cc3eccc757..d18ebdbd1f 100644
--- a/doc/guides/prog_guide/pcapng_lib.rst
+++ b/doc/guides/prog_guide/pcapng_lib.rst
@@ -5,14 +5,14 @@ Packet Capture Next Generation Library
 ======================================
 
 Exchanging packet traces becomes more and more critical every day.
-The de facto standard for this is the format define by libpcap;
+The de facto standard for this is the format defined by libpcap;
 but that format is rather old and is lacking in functionality
 for more modern applications.
 The `Pcapng file format`_ is the default capture file format
 for modern network capture processing tools
 such as `wireshark`_ (can also be read by `tcpdump`_).
 
-The Pcapng library is a an API for formatting packet data
+The Pcapng library is an API for formatting packet data
 into a Pcapng file.
 The format conforms to the current `Pcapng RFC`_ standard.
 It is designed to be integrated with the packet capture library.
@@ -20,9 +20,6 @@ It is designed to be integrated with the packet capture library.
 Usage
 -----
 
-Before the library can be used, the function ``rte_pcapng_init``
-should be called once to initialize timestamp computation.
-
 The output stream is created with ``rte_pcapng_fdopen``,
 and should be closed with ``rte_pcapng_close``.
 
-- 
2.25.1


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

* Re: [PATCH] doc: remove reference to pcapng init function
  2022-05-27 13:21 [PATCH] doc: remove reference to pcapng init function Cian Ferriter
@ 2022-05-27 15:38 ` Stephen Hemminger
  2022-06-01 14:41   ` Thomas Monjalon
  0 siblings, 1 reply; 3+ messages in thread
From: Stephen Hemminger @ 2022-05-27 15:38 UTC (permalink / raw)
  To: Cian Ferriter; +Cc: Reshma Pattan, dev

On Fri, 27 May 2022 13:21:42 +0000
Cian Ferriter <cian.ferriter@intel.com> wrote:

> The rte_pcapng_init function doesn't exist, so remove it from the docs.
> 
> Also fix minor mistakes in the file.
> 
> Signed-off-by: Cian Ferriter <cian.ferriter@intel.com>
> 
> ---
> 
> The rte_pcapng_init function existed in earlier versions of patches for
> this file on the mailing list, but was dropped in favour of
> RTE_INIT(pcapng_init) which doesn't need to be called explicitly by the
> programmer, if I understand the RTE_INIT macro correctly.
> ---
>  doc/guides/prog_guide/pcapng_lib.rst | 7 ++-----
>  1 file changed, 2 insertions(+), 5 deletions(-)
> 
> diff --git a/doc/guides/prog_guide/pcapng_lib.rst b/doc/guides/prog_guide/pcapng_lib.rst
> index cc3eccc757..d18ebdbd1f 100644
> --- a/doc/guides/prog_guide/pcapng_lib.rst
> +++ b/doc/guides/prog_guide/pcapng_lib.rst
> @@ -5,14 +5,14 @@ Packet Capture Next Generation Library
>  ======================================
>  
>  Exchanging packet traces becomes more and more critical every day.
> -The de facto standard for this is the format define by libpcap;
> +The de facto standard for this is the format defined by libpcap;
>  but that format is rather old and is lacking in functionality
>  for more modern applications.
>  The `Pcapng file format`_ is the default capture file format
>  for modern network capture processing tools
>  such as `wireshark`_ (can also be read by `tcpdump`_).
>  
> -The Pcapng library is a an API for formatting packet data
> +The Pcapng library is an API for formatting packet data
>  into a Pcapng file.
>  The format conforms to the current `Pcapng RFC`_ standard.
>  It is designed to be integrated with the packet capture library.
> @@ -20,9 +20,6 @@ It is designed to be integrated with the packet capture library.
>  Usage
>  -----
>  
> -Before the library can be used, the function ``rte_pcapng_init``
> -should be called once to initialize timestamp computation.
> -
>  The output stream is created with ``rte_pcapng_fdopen``,
>  and should be closed with ``rte_pcapng_close``.
>  

Yes, this is leftover from earlier code.

Acked-by: Stephen Hemminger <stephen@networkplumber.org>

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

* Re: [PATCH] doc: remove reference to pcapng init function
  2022-05-27 15:38 ` Stephen Hemminger
@ 2022-06-01 14:41   ` Thomas Monjalon
  0 siblings, 0 replies; 3+ messages in thread
From: Thomas Monjalon @ 2022-06-01 14:41 UTC (permalink / raw)
  To: Cian Ferriter; +Cc: dev, Reshma Pattan, Stephen Hemminger

27/05/2022 17:38, Stephen Hemminger:
> On Fri, 27 May 2022 13:21:42 +0000
> Cian Ferriter <cian.ferriter@intel.com> wrote:
> 
> > The rte_pcapng_init function doesn't exist, so remove it from the docs.
> > 
> > Also fix minor mistakes in the file.
> > 
> > Signed-off-by: Cian Ferriter <cian.ferriter@intel.com>
> 
> Yes, this is leftover from earlier code.
> 
> Acked-by: Stephen Hemminger <stephen@networkplumber.org>

Applied, thanks.



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

end of thread, other threads:[~2022-06-01 14:41 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-27 13:21 [PATCH] doc: remove reference to pcapng init function Cian Ferriter
2022-05-27 15:38 ` Stephen Hemminger
2022-06-01 14:41   ` Thomas Monjalon

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).