ruiu added inline comments.

================
Comment at: clang/utils/bash-autocomplete.sh:3
+
+_clang_filedir()
+{
----------------
yamaguchi wrote:
> ruiu wrote:
> > Is the output of `compgen -f` the same as `_filedir`? If so, can you always 
> > use `compgen -f`?
> _filedir is better than `compgen -f`, because it honors spaces in filenames.
You want to describe it in a comment: _filedir function provided by recent 
versions of bash-completion package is better than "compgen -f" because the 
former honors spaces in pathnames while the latter doesn't. So we use compgen 
only when _filedir is not provided.


================
Comment at: clang/utils/bash-autocomplete.sh:17-18
+  if [[ "$?" != 0 ]]; then
+    COMPREPLY=()
+    _get_comp_words_by_ref cur prev words cword
+  fi
----------------
It's not clear what this code does. Can you explain?


https://reviews.llvm.org/D34924



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to