Skip to content

linecount counts more dirs than just src/ #1050

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Sep 8, 2014
Merged

linecount counts more dirs than just src/ #1050

merged 1 commit into from
Sep 8, 2014

Conversation

jeffdonahue
Copy link
Contributor

we were selling ourselves short with make linecount as it only counted the src/ directory.

Before:

$ make linecount
cloc --read-lang-def=caffe.cloc src/caffe/
     140 text files.
     140 unique files.
       5 files ignored.

http://cloc.sourceforge.net v 1.62  T=0.38 s (353.3 files/s, 62414.0 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
C++                            101           1664            832          18259
CUDA                            30            338            178           2300
CMake                            3             47             39            152
Python                           1              7              5             30
-------------------------------------------------------------------------------
SUM:                           135           2056           1054          20741
-------------------------------------------------------------------------------

After:

$ make linecount
cloc --read-lang-def=caffe.cloc \
                src/caffe include/caffe tools examples \
                python matlab
     267 text files.
     265 unique files.
      51 files ignored.

http://cloc.sourceforge.net v 1.62  T=0.71 s (308.1 files/s, 51849.3 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
C++                            117           1876           1036          19800
Python                          17            443            442           4451
C/C++ Header                    28            636           1577           2800
CUDA                            30            338            178           2300
MATLAB                           6             32             95            206
CMake                            7             64             47            183
Bourne Shell                    13             48             42            128
HTML                             1             13              4            121
-------------------------------------------------------------------------------
SUM:                           219           3450           3421          29989
-------------------------------------------------------------------------------

jeffdonahue added a commit that referenced this pull request Sep 8, 2014
@jeffdonahue
Merge pull request #1050 from jeffdonahue/linecount-more
8cfd587 
linecount counts more dirs than just src/
@jeffdonahue jeffdonahue merged commit 8cfd587 into BVLC:dev Sep 8, 2014
@jeffdonahue jeffdonahue deleted the linecount-more branch September 8, 2014 09:07
mitmul pushed a commit to mitmul/caffe that referenced this pull request Sep 30, 2014
@jeffdonahue
Merge pull request BVLC#1050 from jeffdonahue/linecount-more
3d7781d 
linecount counts more dirs than just src/
RazvanRanca pushed a commit to RazvanRanca/caffe that referenced this pull request Nov 4, 2014
@jeffdonahue
Merge pull request BVLC#1050 from jeffdonahue/linecount-more
9635b41 
linecount counts more dirs than just src/
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@jeffdonahue