Re: New dissectors for BACnet-APDU (packet-bacapp.c)

[Jaap Keuter <jaap.keuter () xs4all nl>]

> On 29 Nov 2018, at 19:27, Jonathan Fleming <jonathan () optigo net> wrote:
>
> Hello,
>
> my name is Jonathan and I am currently working for Optigo Networks in Vancouver, BC. I recently added several new 
> dissectors to the BACnet-APDU protocol for my company, all contained within the packet-bacapp.c file. The list of 
> dissectors I added are as follows:
> bacapp.present_value
> bacapp.notify_type
> bacapp.object_name
> bacapp.to_state
> bacapp.from_state
> bacapp.error_class
> bacapp.error_code
> This is still a work in progress, and some more testing will be done before any kind of pull request is made. I'm 
> reaching out to everyone here to let you know I have been working on this and probably will create a pull request in 
> the near future if everything looks valid. Feel free to contact me if you have any questions or concerns!
>
> Sincerely,
> Jonathan Fleming

Hi Jonathan,

To get things going in the right direction you may want to start off with the following.
1. Create a bug in our bugzilla.
Set the Version to 'Git’, the Severity to ‘Enhancement’ and the Status to ‘IN_PROGRESS’.
The description should contain the intended work to be added, e.g., as listed above.
This bug allows for attaching sample captures as well, used in regression testing.
2. Review the Wiki about creating and submitting patches: https://wiki.wireshark.org/CreatingPatches 
<https://wiki.wireshark.org/CreatingPatches>
3. If not already, prepare an account on Gerrit, our code review site: https://code.wireshark.org/review 
<https://code.wireshark.org/review>

Good luck,
Jaap

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe