Skip to content

In apps_std_imp file build list of paths once without need of additional parsing #270

@quic-parihar

Description

@quic-parihar

recommendation came up is PR: #234

apps_std_imp.c file -> get_dirlist_from_env function.

First we concatenate the strings here, then you parse that during file open. Why? Can't you perform it exactly in the opposite way? Build the list of paths here only once, without additional parsing of the "resulting" string?
going through the string, looking for ; symbols, etc. That needs to happen only once, while you handle the environment variable. After that it should be a list of strings. The path that you got from the config file should be added to the list rather than being concatenated to the string.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions