M0OPK / SDJSharp

Schedules Direct C# Library and Demo UI
GNU General Public License v3.0
2 stars 2 forks source link

Code readability #2

Open M0OPK opened 7 years ago

M0OPK commented 7 years ago

Generally in the classes for specific functions this is fine, but for the main builder process it could do with breaking down some more to clarify what's happening and where.

M0OPK commented 6 years ago

Some update on this, I think a rewrite of the core classes is in order. Mainly to provide some proper inheritance (XML->XMLTV->XMLTV_SD for example). Including the XMLTV indexing as an extension to XMLDocument perhaps. Some investigation needed here.