Merge pull request #694 in SNORT/snort3 from appid_no_so_wild to master

Merge pull request #694 in SNORT/snort3 from appid_no_so_wild to master

Squashed commit of the following:

commit e5ae9e59f829351580d1b880a0077090c7b6cb5f
Author: davis mcpherson <davmcphe.cisco.com>
Date:   Fri Nov 4 11:34:51 2016 -0400

    updates from code review comments and a few generall coding style fixups

commit 1c6e1c075a61c8237090c42b18da97f8bb0dec62
Author: davis mcpherson <davmcphe.cisco.com>
Date:   Wed Nov 2 15:28:32 2016 -0400

    fix memory leak in service_rpc, list of rpc program nodes were dynamically allocated but never freed

    enable appid in default snort.lua configuration

    report parse errors with ParseWarning, functionality may be reduced but appid will still work

commit a0a9c28afccfc93f550369d9f60d3e47f8c1c916
Author: davis mcpherson <davmcphe.cisco.com>
Date:   Wed Nov 2 10:26:40 2016 -0400

    refactor NetworkSet utility functions into a proper class

commit cabb0a02458a3f3ff3c8eec9de7b36b5f992515e
Author: davis mcpherson <davmcphe.cisco.com>
Date:   Wed Nov 2 08:54:12 2016 -0400

    consolidate utility functions to single utilities file or file where the function is used, eliminate duplicated types for custom config elements, delete deprecated files

commit 10e5cac96c626f8dbc6cd5322b612cb0969545c4
Author: davis mcpherson <davmcphe.cisco.com>
Date:   Wed Nov 2 07:48:07 2016 -0400

    reduce appid chatiness - changed LogMessage to DebugMessage where appropriate, move more functions to appropriate source file and delete files no longer useful, ensure all Debug* logging for AppId is controlled by the DEBUG_APPID flag

commit 8913c81160ae3ed00361b6946a369eb48856fa9e
Author: davis mcpherson <davmcphe.cisco.com>
Date:   Wed Nov 2 06:48:16 2016 -0400

    refactoring to move functions and data defs to appropriate source file and delete source files no longer useful