<feed xmlns='http://www.w3.org/2005/Atom'>
<title>tools/wireshark/ws_diag_control.h, branch master</title>
<subtitle>WIP Patches to add a samsung-ipc dissector to Wireshark 
</subtitle>
<link rel='alternate' type='text/html' href='https://git.replicant.us/contrib/GNUtoo/tools/wireshark/'/>
<entry>
<title>Add an include sub-directory to the tree</title>
<updated>2021-11-12T17:33:13+00:00</updated>
<author>
<name>João Valverde</name>
<email>j@v6e.pt</email>
</author>
<published>2021-11-11T18:09:22+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/contrib/GNUtoo/tools/wireshark/commit/?id=ca06d7e4fe59d997d0fa69eea5b025b48dcff6f8'/>
<id>ca06d7e4fe59d997d0fa69eea5b025b48dcff6f8</id>
<content type='text'>
Blind attempt to fix Debian package brokenness managing
headers separately from install rules.

Global public headers that don't fit any of the Wireshark
libraries should be placed in include/.

Eventually the C files in the root dir should be placed somewhere
else as well (like src/) but this is not a priority.

Fixes #17726.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Blind attempt to fix Debian package brokenness managing
headers separately from install rules.

Global public headers that don't fit any of the Wireshark
libraries should be placed in include/.

Eventually the C files in the root dir should be placed somewhere
else as well (like src/) but this is not a priority.

Fixes #17726.
</pre>
</div>
</content>
</entry>
<entry>
<title>text_import: just suppress the two-initializers warning.</title>
<updated>2021-03-27T21:00:44+00:00</updated>
<author>
<name>Guy Harris</name>
<email>gharris@sonic.net</email>
</author>
<published>2021-03-27T20:18:34+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/contrib/GNUtoo/tools/wireshark/commit/?id=2b9a6ee592a63b21093e1afa0a7eecb7319a204e'/>
<id>2b9a6ee592a63b21093e1afa0a7eecb7319a204e</id>
<content type='text'>
Stick with the simpler "init everything to invalid, and then override
that" mechanism, and just turn off the warning for thse structures.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Stick with the simpler "init everything to invalid, and then override
that" mechanism, and just turn off the warning for thse structures.
</pre>
</div>
</content>
</entry>
<entry>
<title>wiretap: have file handlers advertise blocks and options supported.</title>
<updated>2021-02-21T23:18:35+00:00</updated>
<author>
<name>Guy Harris</name>
<email>gharris@sonic.net</email>
</author>
<published>2021-02-21T22:18:04+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/contrib/GNUtoo/tools/wireshark/commit/?id=842a7cccf9ec96ab7ce8945d4de77537f69efc2f'/>
<id>842a7cccf9ec96ab7ce8945d4de77537f69efc2f</id>
<content type='text'>
Instead of a "supports name resolution" Boolean and bitflags for types of
comments supported, provide a list of block types that the file
type/subtype supports, with each block type having a list of options
supported.  Indicate whether "supported" means "one instance" or
"multiple instances".

"Supports" doesn't just mean "can be written", it also means "could be
read".

Rename WTAP_BLOCK_IF_DESCRIPTION to WTAP_BLOCK_IF_ID_AND_INFO, to
indicate that it provides, in addition to information about the
interface, an ID (implicitly, in pcapng files, by its ordinal number)
that is associated with every packet in the file.  Emphasize that in
comments - just because your capture file format can list the interfaces
on which a capture was done, that doesn't mean it supports this; it
doesn't do so if the file doesn't indicate, for every packet, on which
of those interfaces it was captured (I'm looking at *you*, Microsoft
Network Monitor...).

Use APIs to query that information to do what the "does this file
type/subtype support name resolution information", "does this file
type/subtype support all of these comment types", and "does this file
type/subtype support - and require - interface IDs" APIs did.

Provide backwards compatibility for Lua.

This allows us to eliminate the WTAP_FILE_TYPE_SUBTYPE_ values for IBM's
iptrace; do so.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Instead of a "supports name resolution" Boolean and bitflags for types of
comments supported, provide a list of block types that the file
type/subtype supports, with each block type having a list of options
supported.  Indicate whether "supported" means "one instance" or
"multiple instances".

"Supports" doesn't just mean "can be written", it also means "could be
read".

Rename WTAP_BLOCK_IF_DESCRIPTION to WTAP_BLOCK_IF_ID_AND_INFO, to
indicate that it provides, in addition to information about the
interface, an ID (implicitly, in pcapng files, by its ordinal number)
that is associated with every packet in the file.  Emphasize that in
comments - just because your capture file format can list the interfaces
on which a capture was done, that doesn't mean it supports this; it
doesn't do so if the file doesn't indicate, for every packet, on which
of those interfaces it was captured (I'm looking at *you*, Microsoft
Network Monitor...).

Use APIs to query that information to do what the "does this file
type/subtype support name resolution information", "does this file
type/subtype support all of these comment types", and "does this file
type/subtype support - and require - interface IDs" APIs did.

Provide backwards compatibility for Lua.

This allows us to eliminate the WTAP_FILE_TYPE_SUBTYPE_ values for IBM's
iptrace; do so.
</pre>
</div>
</content>
</entry>
<entry>
<title>wiretap: Convert ascend.y to Lemon.</title>
<updated>2020-11-30T08:15:43+00:00</updated>
<author>
<name>Gerald Combs</name>
<email>gerald@wireshark.org</email>
</author>
<published>2020-11-19T01:39:19+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/contrib/GNUtoo/tools/wireshark/commit/?id=f21cd2e23f372bd42e4792ef0b4294d2db881a92'/>
<id>f21cd2e23f372bd42e4792ef0b4294d2db881a92</id>
<content type='text'>
Convert wiretap/ascend.y.in from Bison/YACC to Lemon and rename it to
wiretap/ascend_parser.lemon. Tighten up some of our scanning and
parsing. Make the indentation in it and related files consistent. Aside
from the recent IPv4 fragment offset changes, this produces identical
output to the 3.4 branch for the Ascend trace files I have here.

Remove the comment about supporting other commands. Another timeline
might have an Ascend that successfully pivoted to DSL or 15625B+1D
gigabit ISDN, but this one has neither.

This was our last/only Bison/YACC file, so remove Bison/YACC as a
development and packaging dependency and remove references to it from
the documentation.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Convert wiretap/ascend.y.in from Bison/YACC to Lemon and rename it to
wiretap/ascend_parser.lemon. Tighten up some of our scanning and
parsing. Make the indentation in it and related files consistent. Aside
from the recent IPv4 fragment offset changes, this produces identical
output to the 3.4 branch for the Ascend trace files I have here.

Remove the comment about supporting other commands. Another timeline
might have an Ascend that successfully pivoted to DSL or 15625B+1D
gigabit ISDN, but this one has neither.

This was our last/only Bison/YACC file, so remove Bison/YACC as a
development and packaging dependency and remove references to it from
the documentation.
</pre>
</div>
</content>
</entry>
<entry>
<title>Revert "Make versions of DIAG_OFF() and DIAG_ON() for GCC 4.2 through 4.5."</title>
<updated>2018-09-11T18:54:02+00:00</updated>
<author>
<name>Guy Harris</name>
<email>guy@alum.mit.edu</email>
</author>
<published>2018-09-11T18:53:56+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/contrib/GNUtoo/tools/wireshark/commit/?id=420766099630d0fdcf403e93f5cfffc52889f7ce'/>
<id>420766099630d0fdcf403e93f5cfffc52889f7ce</id>
<content type='text'>
This reverts commit f6b8bc39c66c878ff25a4f48fff382f8775a2711.

Reason for revert: prior to 4.6 or so, the pragmas didn't work as well,
e.g. they couldn't be used inside a function; we depend on them being
usable within a function.

Change-Id: I7a7a0c08a2aca553233c237b8b6c43115095804e
Reviewed-on: https://code.wireshark.org/review/29610
Reviewed-by: Guy Harris &lt;guy@alum.mit.edu&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This reverts commit f6b8bc39c66c878ff25a4f48fff382f8775a2711.

Reason for revert: prior to 4.6 or so, the pragmas didn't work as well,
e.g. they couldn't be used inside a function; we depend on them being
usable within a function.

Change-Id: I7a7a0c08a2aca553233c237b8b6c43115095804e
Reviewed-on: https://code.wireshark.org/review/29610
Reviewed-by: Guy Harris &lt;guy@alum.mit.edu&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Make versions of DIAG_OFF() and DIAG_ON() for GCC 4.2 through 4.5.</title>
<updated>2018-09-11T07:31:31+00:00</updated>
<author>
<name>Guy Harris</name>
<email>guy@alum.mit.edu</email>
</author>
<published>2018-09-11T07:30:58+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/contrib/GNUtoo/tools/wireshark/commit/?id=f6b8bc39c66c878ff25a4f48fff382f8775a2711'/>
<id>f6b8bc39c66c878ff25a4f48fff382f8775a2711</id>
<content type='text'>
The macOS buildbot is currently using llvm-gcc, which is based on GCC
4.2.

Change-Id: I8497243333410380e1d0bc7fa35bccbf841cada1
Reviewed-on: https://code.wireshark.org/review/29596
Reviewed-by: Guy Harris &lt;guy@alum.mit.edu&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The macOS buildbot is currently using llvm-gcc, which is based on GCC
4.2.

Change-Id: I8497243333410380e1d0bc7fa35bccbf841cada1
Reviewed-on: https://code.wireshark.org/review/29596
Reviewed-by: Guy Harris &lt;guy@alum.mit.edu&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix what seems like a copy-paste error.</title>
<updated>2018-09-10T03:49:13+00:00</updated>
<author>
<name>João Valverde</name>
<email>joao.valverde@tecnico.ulisboa.pt</email>
</author>
<published>2018-09-09T23:33:15+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/contrib/GNUtoo/tools/wireshark/commit/?id=ce3d7840c1a141dc61cf5db7267b8ed4d5142923'/>
<id>ce3d7840c1a141dc61cf5db7267b8ed4d5142923</id>
<content type='text'>
Change-Id: I084824c57f1eb61189b337ae3e4786d1d8a88707
Reviewed-on: https://code.wireshark.org/review/29525
Petri-Dish: Guy Harris &lt;guy@alum.mit.edu&gt;
Tested-by: Petri Dish Buildbot
Reviewed-by: Guy Harris &lt;guy@alum.mit.edu&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Change-Id: I084824c57f1eb61189b337ae3e4786d1d8a88707
Reviewed-on: https://code.wireshark.org/review/29525
Petri-Dish: Guy Harris &lt;guy@alum.mit.edu&gt;
Tested-by: Petri Dish Buildbot
Reviewed-by: Guy Harris &lt;guy@alum.mit.edu&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Revert "Try unconditionally using the pragmas in Clang."</title>
<updated>2018-09-09T21:58:21+00:00</updated>
<author>
<name>Guy Harris</name>
<email>guy@alum.mit.edu</email>
</author>
<published>2018-09-09T21:58:10+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/contrib/GNUtoo/tools/wireshark/commit/?id=ed2f332aa36479156cacfd848361ee42ee36569c'/>
<id>ed2f332aa36479156cacfd848361ee42ee36569c</id>
<content type='text'>
This reverts commit 051efd105a8404093139dd4a54620c98c4ff170e.

Reason for revert: The Clang on the macOS buildbot is 3.1, so it should
have been using the _Pragmas; perhaps it "supports" them but they don't
actually *work*, or perhaps something else is wrong.

Change-Id: Id5c3365875c694e6bd4f1a4d8d92e7581cd4585f
Reviewed-on: https://code.wireshark.org/review/29522
Reviewed-by: Guy Harris &lt;guy@alum.mit.edu&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This reverts commit 051efd105a8404093139dd4a54620c98c4ff170e.

Reason for revert: The Clang on the macOS buildbot is 3.1, so it should
have been using the _Pragmas; perhaps it "supports" them but they don't
actually *work*, or perhaps something else is wrong.

Change-Id: Id5c3365875c694e6bd4f1a4d8d92e7581cd4585f
Reviewed-on: https://code.wireshark.org/review/29522
Reviewed-by: Guy Harris &lt;guy@alum.mit.edu&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Try unconditionally using the pragmas in Clang.</title>
<updated>2018-09-09T21:14:40+00:00</updated>
<author>
<name>Guy Harris</name>
<email>guy@alum.mit.edu</email>
</author>
<published>2018-09-09T21:13:27+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/contrib/GNUtoo/tools/wireshark/commit/?id=051efd105a8404093139dd4a54620c98c4ff170e'/>
<id>051efd105a8404093139dd4a54620c98c4ff170e</id>
<content type='text'>
This should test whether the macOS buildbot Clang really *does* support
them, in which case we should use them, or not.

Change-Id: I52b7b9c450b5f7ba09df72a56ddb91b5750196dc
Reviewed-on: https://code.wireshark.org/review/29521
Reviewed-by: Guy Harris &lt;guy@alum.mit.edu&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This should test whether the macOS buildbot Clang really *does* support
them, in which case we should use them, or not.

Change-Id: I52b7b9c450b5f7ba09df72a56ddb91b5750196dc
Reviewed-on: https://code.wireshark.org/review/29521
Reviewed-by: Guy Harris &lt;guy@alum.mit.edu&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Older versions of Clang don't understand -Wpedantic.</title>
<updated>2018-05-18T22:31:05+00:00</updated>
<author>
<name>Guy Harris</name>
<email>guy@alum.mit.edu</email>
</author>
<published>2018-05-18T21:41:26+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/contrib/GNUtoo/tools/wireshark/commit/?id=b1de8c87a12ea431e45e86dea85546948937e21e'/>
<id>b1de8c87a12ea431e45e86dea85546948937e21e</id>
<content type='text'>
Define DIAG_OFF_PEDANTIC and DIAG_ON_PEDANTIC, and have it do nothing on
Clang prior to 4.0.

Change-Id: Ic6b2e607659db66f3210401024bf3f2239665506
Reviewed-on: https://code.wireshark.org/review/27649
Petri-Dish: Guy Harris &lt;guy@alum.mit.edu&gt;
Reviewed-by: Peter Wu &lt;peter@lekensteyn.nl&gt;
Tested-by: Petri Dish Buildbot
Reviewed-by: Guy Harris &lt;guy@alum.mit.edu&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Define DIAG_OFF_PEDANTIC and DIAG_ON_PEDANTIC, and have it do nothing on
Clang prior to 4.0.

Change-Id: Ic6b2e607659db66f3210401024bf3f2239665506
Reviewed-on: https://code.wireshark.org/review/27649
Petri-Dish: Guy Harris &lt;guy@alum.mit.edu&gt;
Reviewed-by: Peter Wu &lt;peter@lekensteyn.nl&gt;
Tested-by: Petri Dish Buildbot
Reviewed-by: Guy Harris &lt;guy@alum.mit.edu&gt;
</pre>
</div>
</content>
</entry>
</feed>
