diff options
author | Gerhard Sittig <gerhard.sittig@gmx.net> | 2022-08-21 13:52:19 +0200 |
---|---|---|
committer | Gerhard Sittig <gerhard.sittig@gmx.net> | 2022-08-21 13:52:19 +0200 |
commit | ec5fc441e38e59a57cd508abf420e30fc3c28e0d (patch) | |
tree | a8b15bf917dc8bb6734a3af7321ade3bfc6b0cdf /srd.c | |
parent | e556e1168af7027df08622ecfe11309811249e81 (diff) | |
download | libsigrokdecode-ec5fc441e38e59a57cd508abf420e30fc3c28e0d.tar.gz libsigrokdecode-ec5fc441e38e59a57cd508abf420e30fc3c28e0d.zip |
srd: add TODO comment on the SIGROKDECODE_DIR env var's motivation
A comment in the srd.c implementation suggests that the environment
variable SIGROKDECODE_DIR would serve debugging purposes and would
override other incarnations of available decoders. I disagree with
either of these interpretations. Add a TODO comment, the phrase may
need an update or rework.
Diffstat (limited to 'srd.c')
-rw-r--r-- | srd.c | 9 |
1 files changed, 9 insertions, 0 deletions
@@ -270,6 +270,15 @@ SRD_API int srd_init(const char *path) } /* Environment variable overrides everything, for debugging. */ + /* + * TODO + * Is the comment still applicable and correct or up to date? + * This implementation adds paths which were specified by the + * env var. Which can shadow files in other locations, or can + * extend the set of available decoders. Which need not only + * serve for development, it is as beneficial to regular users. + * Without shadowing all other files still are found. + */ if ((env_path = g_getenv("SIGROKDECODE_DIR"))) { if ((ret = srd_decoder_searchpath_add(env_path)) != SRD_OK) { Py_Finalize(); |