Re: [Cocci] [v7] coccicheck: Support search for SmPL scripts within selecteddirectory hierarchy

2019-11-11 Thread zhong.shiqi
> > Changes in v7: > > 1:adjust coccinelle.rst documentation > > Thanks for your contribution. > > Now I am curious if other contributors will add corresponding review comments. > > > > 2:fix a repo of "default" Sorry...it s

Re: [Cocci] [v4] coccicheck: Support search for SmPL scripts within selecteddirectory hierarchy

2019-10-31 Thread zhong.shiqi
> I suggest to improve the case distinction for the make (and environment) > variable “COCCI” also by adjusting the document outline. > * Empty string > * Directory selection > * File name specification > > See also: > https://docutils.readthedocs.io/en/sphinx-docs/user/rst/quickstart.html#section

Re: [Cocci] [PATCH v5] coccicheck: Support search for SmPL scripts withinselected directory hierarchy

2019-10-31 Thread zhong.shiqi
> I got the impression that you are struggling with difficulties (for unknown > reasons) > around adding space characters at some places. > How would you like to improve this situation? Thanks for your concern,but i haven't struggled with anything.I think i'm used to adding space characters. >

Re: [Cocci] [PATCH v4] coccicheck: Support search for SmPL scripts withinselected directory hierarchy

2019-10-29 Thread zhong.shiqi
> > Would you like to update the provided software documentation together with > > the small extension of this bash script? > > > > Update candidates: > > Please reconsider also the section “Using Coccinelle with a single semantic > patch”: > https://git.kernel.org/pub/scm/linux/kernel/git/torva

Re: [Cocci] [PATCH v4] coccicheck: Support search for SmPL scripts withinselected directory hierarchy

2019-10-29 Thread zhong.shiqi
Sorry for the late reply! I would say yes. > Would you like to update the provided software documentation together with > the small extension of this bash script? I'd like to but i don't have rights to update. Thansk, zhong.shiqi > Update candidates: > * >

Re: [Cocci] [PATCH v3] coccicheck: Support search for SmPL scripts withinselected directory hierarchy

2019-10-25 Thread zhong.shiqi
Allow defining the environment variable “COCCI” as a directory > to search SmPL scripts. > > * Start a corresponding file determination if it contains an acceptable > path. Thanks,i think it would be better! Best Regards, zhong.shiqi > Regards, > Markus_

Re: [Cocci] [PATCH v3] coccicheck: Support search for SmPL scripts withinselected directory hierarchy

2019-10-25 Thread zhong.shiqi
Allow defining the environment variable “COCCI” as a directory > to search SmPL scripts. > > * Start a corresponding file determination if it contains an acceptable > path. Thanks,i think it would be better! Best Regards, zhong.shiqi > Regards, > Markus_

Re: [Cocci] [v2] coccicheck: support $COCCI being defined as adirectory

2019-10-24 Thread zhong.shiqi
> >> How do you think about to use the subject “[PATCH] coccicheck: > >> Support search for SmPL scripts within selected directory hierarchy”? > > > > I would like to use subject as you said. > > Thanks for your positive feedback. > > > > But it seems a little bit long, > > I hope that this suggest

Re: [Cocci] [PATCH v2] coccicheck: support $COCCI being defined as adirectory

2019-10-24 Thread zhong.shiqi
> I find that the commit message should still be considerably improved. > How do you think about to use the subject “[PATCH] coccicheck: > Support search for SmPL scripts within selected directory hierarchy”? I would like to use subject as you said. But it seems a little bit long, does it matter?

Re: [Cocci] [PATCH v2] coccicheck:support $COCCI being defined as adirectory

2019-10-24 Thread zhong.shiqi
a directory that contains .cocci files. > > In general, at least in simple cases, it is not necessary to mention the > name of the file you are modifying in the comit log, because one can see > that just below from looking at the diffstat and the patch. > > thanks, > julia >

Re: [Cocci] [PATCH] Configuring COCCI parameter as a directory issupportted

2019-10-23 Thread zhong.shiqi
directory(based on "-d" option). If it is a directory, the next statement is executed. If it is not a directory or if it is just a file with a a suffix of cocci, the statement after the final else branch is executed. Thanks for you reply, zhong.shiqi E: zhong.sh...@