Quote:
Originally Posted by sherman
Hi geek1011
A couple of thoughts on your new patch format.
Why not include the patch description as part of each patch, instead of outside in comments? It would make parsing the description easier.
Also, have you thought about more defined versioning information (with regards to firmware version etc), rather than specifying the version in a comment somewhere?
Cheers,
Sherman
|
That's actually a great idea about the description. I'll implement it this weekend.
As for the versioning, I was considering it, but decided against it. Firstly, the patches will most likely fail if applied to the wrong version. Secondly, it is very difficult to extract the version info from the binary, and thirdly, it is hard to represent things like ranges in an concise way.