Roland Knall <rknall@...> writes: > I have provided some time ago a patch for submission into wireshark > (Bug #5753). Over the course of the next two weeks a new version of > this patch would be completed, which would enable the dissector to > talk Modbus/TCP as well. But this version would be very preliminary, > and not stable for a longer time.
"Not stable" or "not complete"? I think incomplete dissectors are OK as they can always be improved upon later as time permits, but unstable is not acceptable. > > Should I remove the current patch and resubmit it with the changes, or > should I wait until this version of the patch gets submitted (which is > pretty stable right now), and then submit the changes? The latest patch you provided seems to have been reviewed by Guy, Jeff and Jakub. I *think* it's probably close to the point of being accepted, so I'd say rather than introducing another patch that "might not be stable for a longer time", that you should hold off until after this patch has been accepted. Another patch at this point might prolong the time it takes to be accepted, but I guess it depends somewhat on the extent of your new changes and if you really meant "not complete" rather than "not stable". ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev@wireshark.org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe