Wireshark · Wireshark-commits: [Wireshark-commits] master b44172b: TFTP: Minor tweaks to tftp.destination_file field.

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=b44172b4ee1dcdc6b1d7b41f944cec17b117776f
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

b44172b by Darius Davis (darius@xxxxxxxxxx):

    TFTP: Minor tweaks to tftp.destination_file field.
    
    Two trivial cleanups of the definition of the tftp.destination_file field:
    There is probably no need to shout DESTINATION in capital letters, and change
    "source" to "destination" in the field's blurb.
    
    Testing Done: Built on macOS 10.12.6.  Examined the capture attached to
       bug 10305 (tftpConversationError.pcapng, which includes a TFTP WRQ), and saw
       that the capitalization of the "Destination File" field is as expected in
       the packet dissection, and that the status bar now describes the field as
       the "TFTP destination file name".
    
    Change-Id: I9f5bded321c16d4e200bf1caf80ad5733ecc8287
    Reviewed-on: https://code.wireshark.org/review/30857
    Petri-Dish: Anders Broman <a.broman58@xxxxxxxxx>
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  71268f8b  Apply port preferences during dissector handoff registration
     add  b44172b   TFTP: Minor tweaks to tftp.destination_file field.


Summary of changes:
 epan/dissectors/packet-tftp.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)