2010-05-04, 17:09
Over the last few days, in an effort to learn XBMC code, I have been stepping through and reading the scraper code. Its an impressive framework, but also very complex if you're not familiar with.
Would the dev's accept a patch with comments only for the scraper? I think this would benefit anyone looking at the code in future.
I would propose adding doxygen comments to:
CIMDB
ScraperParser
ScraperUrl
It would be function signature commenting only, I wouldn't touch the code or add any inline commenting (to make it easier to accept the patch - I could add this later if needed).
I don't want to waste my time, so thought I would check first.
Would the dev's accept a patch with comments only for the scraper? I think this would benefit anyone looking at the code in future.
I would propose adding doxygen comments to:
CIMDB
ScraperParser
ScraperUrl
It would be function signature commenting only, I wouldn't touch the code or add any inline commenting (to make it easier to accept the patch - I could add this later if needed).
I don't want to waste my time, so thought I would check first.