On 5/1/20 12:02 PM, Luke Mewburn wrote:
> On 20-05-01 07:34, Jaap Keuter wrote:
> |
> | > On 1 May 2020, at 04:13, Luke Mewburn <luke@xxxxxxxxxxx> wrote:
> | > However, looking at the code some more, it appears that generally
> | > wireshark_gen.py generates code in the order the operations are defined;
> | > the exception (hah!) is the user exceptions.
> | >
> | > If I instead add at the top
> | > import collections
> | > and change get_exceptionList() from
> | > ex_hash = {} # holds a hash of unique exceptions.
> | > to
> | > ex_hash = collections.OrderedDict() # holds a hash of unique exceptions.
> | >
> | > This results in consistent generated code with both python 2.7 (CentOS 7)
> | > and python 3.7 (Fedora 31).
> | >
> | > I've also fixed a whitespace issue in the generated code by indenting
> | > the break in template_helper_switch_msgtype_default_end, so that it
> | > matches the epan/dissectors code and other default statements.
> | >
> | >
> | > Here's a patch with my suggested fixes.
> | > Or would you prefer a commit/pull request (etc)?
> | >
> | >
> | > regards,
> | > Luke.
> | >
> |
> | Hi Luke,
> |
> | That’s great, I didn’t have the opportunity yet to dig into this.
> | Nice that you compared Python 2.7 and 3.7 already.
> | I’ll pick this up and put it in with the other fixes I've lined up,
> | so you won’t have to push a change. I’ll credit you in the commit :)
> |
> | Thanks,
> | Jaap
>
> No problem; happy to help!
>
> cheers,
> Luke.
>
>
He Luke,
Are you able to recreate the parlay dissector currently in the repository with
this. So far I haven't succeeded.
Thanks,
Jaap