4 ms·

Is this only for c code? I ran it on a Java codebase and got 0LOC 0B on all dirs

• js2 4y ago
  Yes: https://github.com/skeeto/scratch/blob/5129fd45153999924334c5a989216e98cb5e1098/misc/watc.c#L111 https://github.com/skeeto/scratch/blob/5129fd45153999924334c...
  • unwind 4y ago
    OMG that is so ... I don't even know. Is it over-engineering to use a static 16-entry perfect hash of file name extensions to filter out the five supported languages? It's at least somewhat obscure, in the sense that adding support for another language becomes quite involved. It would be interesting to micro-bench this code against something more naïve that just strcmp()s the extension against a list of known strings, or something.
    • js2 4y ago
      > Is it over-engineering to use a static 16-entry perfect hash of file name extensions to filter out the five supported languages? It's patently absurd. I also like the ATOMIC_RADD for populating the line count: https://github.com/skeeto/scratch/blob/5129fd45153999924334c5a989216e98cb5e1098/misc/watc.c#L210 https://github.com/skeeto/scratch/blob/5129fd45153999924334c... This entire program is too clever by half, while still not doing a "correct" source line count (it just counts new lines).