[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [avrdude-dev] [patch #7538] Add support for AT89S52/51 MCS-51 MPUs
From: |
René Liebscher |
Subject: |
Re: [avrdude-dev] [patch #7538] Add support for AT89S52/51 MCS-51 MPUs |
Date: |
Thu, 1 Nov 2012 09:40:36 +0100 |
Hi,
as there is no documentation of internals, may be we should think about at
least to document it from the sources using a tool like doxygen. This would
need some rework of the code comments, but I think this would evolve when
people try to understand the code and write patches they might also add some
doc comments to existing functions. (I guess I would also use some of my
spare time to document some functions and setup doxygen in the Makefiles.)
For most programmers it would make sense to add a file-comment describing
the handling of the programmer in general, and adding specific explanations
to functions itself.
Kind regards
René Liebscher
-----Ursprüngliche Nachricht-----
Von: address@hidden
[mailto:address@hidden Im Auftrag von
Joerg Wunsch
Gesendet: Donnerstag, 1. November 2012 08:07
An: Tobias Kaiser; Joerg Wunsch; address@hidden
Betreff: [avrdude-dev] [patch #7538] Add support for AT89S52/51 MCS-51 MPUs
Follow-up Comment #3, patch #7538 (project avrdude):
> Where can I find the preamble in the patch tracker?
Just try submitting a new patch, and you'll see it. (Don't
complete the procedure, just start it, pretending you were
submitting something new.)
> Should I submit a patch containing further program
> internal documentation?
There is no document for internal stuff, so your patch should
include respective comments.
> Will the patch afterwards be applied to the avrdude source
I generally don't make promises about what is going to be
included in the next release (because I have to find the
respective spare-time), but if everything fits and is
documented, chances are good.
_______________________________________________________
Reply to this item at:
<http://savannah.nongnu.org/patch/?7538>
_______________________________________________
Message sent via/by Savannah
http://savannah.nongnu.org/
_______________________________________________
avrdude-dev mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/avrdude-dev