-
Notifications
You must be signed in to change notification settings - Fork 57
Update code tree #80
Comments
I see two possible data structures: Either one tree by extension feature:
Or a common data structure containing a folder by feature:
|
I personally find the first structure to better fit the project, as it makes the various components more clear. That way if anything ever needs to be split out into a separate project in the future, the separation should already be more evident than in the second structure. Seeing that there are two XML parsers, perhaps both of them can be replaced by something built into Thunderbird these days? I would guess that these parsers were added a long time ago. |
That makes sense, indeed. I think too that first structure will help new developers to understand where to work to fix a bug. For the XML parsers, I think too we can use Mozilla tools to parse XML to DOM and then create JSON from DOM serialized as string. Although, I think that work will be a long term goal we shouldn't care right now, because it seems right now that code works and because other goals like directory structure update, automatic indentation and localization are more critical. |
Fully agree. The parsers are not urgent. |
Major changes are in place. I believe this can be closed now. |
Hello,
According to comments on #70, we will certainly do some initial big pull requests updating all the code (to automatically beautify the code files).
I want to suggest to update also the code tree structure and to document it.
Currently, what I've seen is this one if I understand it correctly:
Edit (2017.11.11):
According to the comments below, we will switch to such a structure:
I've made a task by main sub-directory as I think I'll make one pull request by task.
The text was updated successfully, but these errors were encountered: