Removed bool return value from LoadSettings()

The original intent was to indicate success or failure of loading
settings. However, it's not clear what success or failure means
because it is legitimate to not have a user- or site-settings file, so
indicating failure if one of these cannot be loaded is not really
accurate. Moreover, the return value wasn't used by the caller.

In addition, there was a bug in the previous implementation. If the
user settings weren't loaded, the site settings wouldn't be loaded
either. Now this method no longer returns any boolean value.

Change-Id: I039fc39aec3866e0d5b349fa43fc8f5c0d648d2a
Added PV_SETTINGS_DEBUG environment variable

When set, the application will print information about where settings
file are being looked for and whether loading them succeeded.

Change-Id: Ie98c568c417cef4bdbfb545ea5fe87bb3663453f
Try to load site settings files and use return value in search logic

Now instead of checking whether a settings file exists before
attempting to load it, we just try to load it and use the return code
from vtkSMSettings::AddCollectionFromString() to determine whether to
search other locations. This required fixing some of the behavior of
vtkSMSetting::AddCollectionFromFile() when the given file couldn't be
loaded.

Change-Id: Ic4f1caea33454453a4c7049c10642148a91f9cad
Added some more debugging macro statements

Change-Id: Ied84439e6743102e698814d1ef12c6d10c115ab5
If no user settings file is found, create a user settings collection

If this isn't done, then the site settings collection, if it exists,
will be treated incorrectly as the user settings collection, setting
changes will be saved there, and that collection will be written to
the user settings file.

Change-Id: I3581efc6ecc2bb2ae43f41854c7f8580776632e9
Merge topic '13575_PV_PLUGIN_DEBUG' into next

8d4ed01b If no user settings file is found, create a user settings collection
4ccb9ea3 Added some more debugging macro statements
ebd7c0e3 Try to load site settings files and use return value in search logic
c7f463ea Added PV_SETTINGS_DEBUG environment variable
ba08af44 Removed bool return value from LoadSettings()
!= was being used instead of == in conditional

With this operator, this conditional was checking for whether the
global proxy ID existed in a std::set that used to ensure uniqueness
of the proxy in the collection. Instead, it should check whether that
ID wasn't in the std::set.

Added test for this fix

Change-Id: I0a2b5bb614acff6bdc557ba9da7c1ef85c501691
Merge topic 'fix_get_proxies' into next

645ac1ea != was being used instead of == in conditional
Updated baseline

Change-Id: I40ea7f18d7023dc04af77343a3f0564e640b2c44
Merge topic 'widen_properties_panel' into next

017472d8 Updated baseline
Ensured that Python interpreter is initialized

Prevents crash on Windows

Change-Id: I0f1da6922f284f3332c2a5fdcd6e981ee52873e8
Added removal of DOS line-endings in Python code

Cribbed from vtkPythonInterpreter.cxx.

Change-Id: I2b8de47ce3f2e8d93e5c7cded3af0ea969445aad
Merge topic '14885_python_view_crash' into next

85602edf Added removal of DOS line-endings in Python code
b0b48e84 Ensured that Python interpreter is initialized
Removed vtkMatplotlibUtilities class

It is no longer needed.

Change-Id: Ia60dfec4b394355e7eda848a19dd12c28a9d3a37
Merge topic 'remove_matplotlib_utilities' into next

777e72b5 Removed vtkMatplotlibUtilities class
Updating another baseline

Change-Id: Id737a02209462c18494022d301b33c0350deb7e5
Merge topic 'widen_properties_panel' into next

a9f7af8b Updating another baseline
Update VTK to bring in opacity fix.

Change-Id: Ie0437119b65891b9eb065b6e4519ec07b4204f1f
Merge topic 'remove_duplicate_tiff_readers' into next

f0999171 Update VTK to bring in opacity fix.
Fix vtkSMTrace segfault when char* is NULL.

Handled by passing a None when char* is NULL.

Change-Id: I256c759a8f4a4b120b6ac042441e76afcf407256