Hi Justin,
Many thanks!
> * copyright ownership. The ASF licenses this file to you under the
Apache License, Version 2.0
It is a obvious inconsistent format. I will fix it.
For others, they look different because they are from different file
formats, e.g., java file with /***/, bat file with @REM, antlr g file with
//, script file with #.
Best,
-----------------------------------
Xiangdong Huang
School of Software, Tsinghua University
黄向东
清华大学 软件学院
Justin Mclean <jus...@classsoftware.com> 于2019年12月1日周日 下午3:17写道：
> Hi,
>
> >> I did notice a mix of unix and windows new lines, you might want to fix
> > that. This is also a very minor slight inconsistency with the ASF headers
> > (extra spaces in some).
>
> Compare these lines:
> regarding copyright ownership. The ASF licenses this file
> * regarding copyright ownership. The ASF licenses this file
> * copyright ownership. The ASF licenses this file to you under the
> Apache License, Version 2.0
> * regarding copyright ownership. The ASF licenses this file
> * regarding copyright ownership. The ASF licenses this file
> # regarding copyright ownership. The ASF licenses this file
> // regarding copyright ownership. The ASF licenses this file
> @REM regarding copyright ownership. The ASF licenses this file
> regarding copyright ownership. The ASF licenses this file
>
> This was put through a sort -u so each line is a variation on the other.
> There may be other differences, I’ve not checked.
>
> I found this by accident via:
> search “copyright” | sort -u
>
> Where search is:
> function search() {
> find . -type f -exec grep -i "$1" {} \; -print
> }
>
> Just a little tool I use to help find things in releases. My incubator
> talks usually mention this.
>
> Thanks,
> Justin
>
>
>
>
