Changing behavior of include directive (DSL)? about freefem-sources HOT 3 CLOSED

from freefem-sources.

I think there is a bug in mylex::xxxx::open in src/fflib/lex.cpp.

If FF_INCLUDEPATH is empty (i.e. lex->includedir), then the code tries to open ff (i.e. filename). It is ok.

However, if FF_INCLUDEPATH is not empty, then it tries to prepend ff with each path in FF_INCLUDEPATH without trying to open ff alone. But the error message in case of failure says that opening ff failed while it is actually false (even though it is actually the expected behavior). So, I think I will need to modify the original algorithm.

Furthermore, as I would like to modify the behavior of include directive, I need to know in which order files should be searched for. For example, let say that $FF_ROOT/foo.edp includes bar.edp and that it is run from $SOMEWHERE. What is the order of trials?

1. $FF_ROOT/bar.edp, then $FF_INCLUDEPATH/bar.edp, then $SOMEWHERE/bar.edp
2. $FF_INCLUDEPATH/bar.edp, then $FF_ROOT/bar.edp, then $SOMEWHERE/bar.edp
3. $SOMEWHERE/bar.edp, then $FF_ROOT/bar.edp, then $FF_INCLUDEPATH/bar.edp
4. $SOMEWHERE/bar.edp, then $FF_INCLUDEPATH/bar.edp, then $FF_ROOT/bar.edp

Something else?

from freefem-sources.

After checking, there is no bug in the code. I make a PR with the following behavior:

1. the file is searched with no path
2. then the file is searched with the path to the directory of the file which includes it
3. then the file is searched with paths contained in includedir (FF_INCLUDEPATH and more)

Therefore, only step 2 is new.

from freefem-sources.