[Home]

Summary:ASTERISK-09598: Doxygen Updates res folder
Reporter:snuffy (snuffy)Labels:
Date Opened:2007-06-06 00:32:59Date Closed:2007-08-08 17:15:08
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Documentation
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) features_v2.diff
( 1) res_features.diff
Description:Decided to make just one bug report for all doxygen i'm doing for the res/ folder of asterisk.

I'm trying to get through one file a day time/file size dependant.
These patches provide the basics only not a truly descriptive documentation.
Comments:By: Russell Bryant (russell) 2007-06-06 09:36:23

I have a request regarding adding comments for structures.

Personally, I prefer to have the comments not on the same line.  However, if you must use inline comments, please use spaces to space them away from the code instead of tabs.  Otherwise, they don't align correctly unless people use the same tab size in their editor that you do.

Thanks.  :)

By: snuffy (snuffy) 2007-06-06 16:16:33

Ok russell,
I will attempt a different way and see if that works, as for tabs vs spaces i will make sure all my new ones use only spaces. The reason i used on the same line commets is because that's how i'd seen other structs in * documented.

I will run over my stuff the 'indent' command listed in CODING-GUIDELINES for the subsequent patches

By: Russell Bryant (russell) 2007-06-07 16:26:20

Check your res_crypto patch.  I can't get it to apply.  Patch claims it is malformed.

By: snuffy (snuffy) 2007-07-26 20:47:21

files can be removed will upload new ones

By: Russell Bryant (russell) 2007-07-30 12:59:23

the old files have been removed

By: snuffy (snuffy) 2007-08-05 17:23:59

This update was against SVN trunk 78158
Found issue with patch, delete res_features.diff
new patch changes formatting in the .h and adds more doc to actual features



By: Digium Subversion (svnbot) 2007-08-07 17:46:27

Repository: asterisk
Revision: 78541

------------------------------------------------------------------------
r78541 | russell | 2007-08-07 17:46:26 -0500 (Tue, 07 Aug 2007) | 4 lines

Add another big set of doxygen documentation improvements from snuffy.
(closes issue ASTERISK-9598)
(closes issue ASTERISK-10034)

------------------------------------------------------------------------

By: Digium Subversion (svnbot) 2007-08-07 19:52:02

Repository: asterisk
Revision: 78563

------------------------------------------------------------------------
r78563 | murf | 2007-08-07 19:52:00 -0500 (Tue, 07 Aug 2007) | 12 lines

Merged revisions 78541 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
r78541 | russell | 2007-08-07 17:04:01 -0600 (Tue, 07 Aug 2007) | 4 lines

Add another big set of doxygen documentation improvements from snuffy.
(closes issue ASTERISK-9598)
(closes issue ASTERISK-10034)

........

------------------------------------------------------------------------

By: Digium Subversion (svnbot) 2007-08-07 20:35:41

Repository: asterisk
Revision: 78565

------------------------------------------------------------------------
r78565 | juggie | 2007-08-07 20:35:37 -0500 (Tue, 07 Aug 2007) | 36 lines

Merged revisions 78527,78544,78561,78563-78564 via svnmerge from
https://origsvn.digium.com/svn/asterisk/team/group/CDRfix5

................
r78527 | automerge | 2007-08-07 18:29:43 -0400 (Tue, 07 Aug 2007) | 1 line

automerge commit
................
r78544 | automerge | 2007-08-07 19:26:08 -0400 (Tue, 07 Aug 2007) | 1 line

automerge cancel
................
r78561 | murf | 2007-08-07 20:19:50 -0400 (Tue, 07 Aug 2007) | 1 line

Set the cdr app fields when entering dial
................
r78563 | murf | 2007-08-07 21:09:35 -0400 (Tue, 07 Aug 2007) | 12 lines

Merged revisions 78541 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
r78541 | russell | 2007-08-07 17:04:01 -0600 (Tue, 07 Aug 2007) | 4 lines

Add another big set of doxygen documentation improvements from snuffy.
(closes issue ASTERISK-9598)
(closes issue ASTERISK-10034)

........

................
r78564 | murf | 2007-08-07 21:10:54 -0400 (Tue, 07 Aug 2007) | 1 line

oops. set the automerge again
................

------------------------------------------------------------------------

By: Digium Subversion (svnbot) 2007-08-08 17:15:08

Repository: asterisk
Revision: 78690

------------------------------------------------------------------------
r78690 | murf | 2007-08-08 17:15:07 -0500 (Wed, 08 Aug 2007) | 12 lines

Merged revisions 78541 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
r78541 | russell | 2007-08-07 17:04:01 -0600 (Tue, 07 Aug 2007) | 4 lines

Add another big set of doxygen documentation improvements from snuffy.
(closes issue ASTERISK-9598)
(closes issue ASTERISK-10034)

........

------------------------------------------------------------------------