Wireshark-commits: [Wireshark-commits] master 19153cf: wsutil: Add config_file_exists_with_entries(
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Thu, 11 Oct 2018 12:33:11 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=19153cf9113c73f110f1f40d37cbf232f056fa51
Submitter: "Peter Wu <peter@xxxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

19153cf by Stig Bjørlykke (stig@xxxxxxxxxxxxx):

    wsutil: Add config_file_exists_with_entries()
    
    The purpose of this function is to check if a configuration file exists
    and has at least one entry which is not a comment.
    
    Use this when building the list of profiles where the user can copy
    configuration from, to avoid listing profiles with empty files or files
    with only comments.
    
    Change-Id: If45f52025959818fb1213ffac488cd59441e9fce
    Reviewed-on: https://code.wireshark.org/review/30113
    Petri-Dish: Stig Bjørlykke <stig@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    

Actions performed:

    from  b6d1828   DCERPC: remove unused variable and mark another global as static
     add  19153cf   wsutil: Add config_file_exists_with_entries()


Summary of changes:
 debian/libwsutil0.symbols                |  1 +
 ui/qt/widgets/copy_from_profile_menu.cpp |  2 +-
 wsutil/filesystem.c                      | 30 ++++++++++++++++++++++++++++++
 wsutil/filesystem.h                      |  6 ++++++
 4 files changed, 38 insertions(+), 1 deletion(-)