diff options
| author | Mikael Kanstrup <mikael.kanstrup@gmail.com> | 2017-08-28 10:38:31 +0200 |
|---|---|---|
| committer | Peter Wu <peter@lekensteyn.nl> | 2017-08-28 16:53:32 +0000 |
| commit | 879920a2ebaacf33e554ab47e340c34fe2e878d8 (patch) | |
| tree | e7be37cd3912c5bf3146c52f0fe114170b1b3bdf /capture_opts.h | |
| parent | 8646596829494dbe054a18958eac68682ba42721 (diff) | |
iface_lists: Remove locked field
It seems the locked field of interface_t was used to avoid
simultaneous updates of interface entries from either multiple
threads or most likely the recursive UI update callbacks
case later identified.
Since 802362e ("Avoid recursive scan_local_interfaces operation")
the recursive callback behavior is no longer happening. And as
code does not have consistent checks the locked field can anyway
hardly protect a multi-threaded case if such a case exists.
Remove the unnecessary locked field.
Ping-Bug: 13864
Change-Id: Idc393f702b82aa6014dd636572d00f0d67120bf3
Reviewed-on: https://code.wireshark.org/review/23262
Petri-Dish: Peter Wu <peter@lekensteyn.nl>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
Diffstat (limited to 'capture_opts.h')
| -rw-r--r-- | capture_opts.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/capture_opts.h b/capture_opts.h index 311cc4ac97..8d38cac607 100644 --- a/capture_opts.h +++ b/capture_opts.h @@ -200,7 +200,6 @@ typedef struct interface_tag { if_info_t if_info; gboolean selected; gboolean hidden; - gboolean locked; #ifdef HAVE_EXTCAP /* External capture cached data */ GHashTable *external_cap_args_settings; |