0
\$\begingroup\$

I've got a slightly big but also not big-as-Linux-kernel C project which is ~1200 lines of code , ~20 functions , 3 files, and so on.

I'm new to this site, and was curious to know if this site is for reviewing small, neatly pasted-to-the-question functions and M.R.Es or if you folks are also involved in reviewing semi-small full Github-linked projects on request.

Judging by the few questions I've come across, I only see <200 lines posted within question, say like :

bubble-sort()
{
    x();
    y();
    z(); 
}
| |
\$\endgroup\$
  • \$\begingroup\$ 1200 lines is very likely to be over our limit (65k-and-change characters, including description). \$\endgroup\$ – Mast Sep 5 at 10:05
  • \$\begingroup\$ Note that MRE are off-topic, since they're example code. \$\endgroup\$ – Mast Sep 5 at 10:07
  • 2
    \$\begingroup\$ How to get the best value out of Code Review - Asking Questions \$\endgroup\$ – Mast Sep 5 at 10:21
  • \$\begingroup\$ codereview.stackexchange.com/help/how-to-ask , codereview.stackexchange.com/help/on-topic , codereview.stackexchange.com/help/dont-ask : there are three help pages that are (AFAIK) in the Help section of every SE and you should read them before posting. \$\endgroup\$ – BCdotWEB Sep 5 at 12:47
  • \$\begingroup\$ Thanks a lot to everyone, So it is safe and acurate to conclude that we review specifc code that does one thing rather than entire program that consist of multiple blocks of code that do many a thing under a general umbrella term ? \$\endgroup\$ – A P Jo Sep 5 at 14:34
  • \$\begingroup\$ Well, we can, if it's small enough. We don't review entire libraries if they're too big, but specialized libraries can and have been reviewed. \$\endgroup\$ – Mast Sep 5 at 15:11
  • \$\begingroup\$ @Mast Alright, so its a fuzzy line.. \$\endgroup\$ – A P Jo Sep 5 at 15:20
  • 1
    \$\begingroup\$ No, there's a hard line at the size of a post. On most sites it's 30k characters, on Code Review it's 65536. \$\endgroup\$ – Mast Sep 5 at 15:27
  • 2
    \$\begingroup\$ I'll note that some browsers give out before then. I've written reviews that are so long that Edge locks up, and that would have been nowhere near 65k characters. \$\endgroup\$ – Carcigenicate Sep 6 at 0:23
  • \$\begingroup\$ @Carcigenicate Edge ? XD \$\endgroup\$ – A P Jo Sep 8 at 5:42

You must log in to answer this question.

Browse other questions tagged .