Opened 19 years ago

Closed 19 years ago

Last modified 19 years ago

#3987 closed patch (fixed)

Doxygen for MythThemedMenu (including private classes)

Reported by: anonymous Owned by: danielk
Priority: trivial Milestone: 0.21
Component: mythtv Version: head
Severity: low Keywords: doxygen
Cc: Ticket locked: no

Description

Submitting Doxygen comments based on the excellent coding I saw in MythGesture. First patch so please bear with me now.

Attachments (1)

firstPlugin.diff (11.4 KB ) - added by hendrixski@… 19 years ago.
My first patch, a bunch of doxygen comments to add to MythThemedMenu

Download all attachments as: .zip

Change History (4)

by hendrixski@…, 19 years ago

Attachment: firstPlugin.diff added

My first patch, a bunch of doxygen comments to add to MythThemedMenu

comment:1 by danielk, 19 years ago

Milestone: unknown0.21
Owner: changed from Isaac Richards to danielk

comment:2 by danielk, 19 years ago

Resolution: fixed
Status: newclosed

(In [14596]) Fixes #3987. Applies hendrixski's documentation patch for myththemedmenu after some edits for style and brevity.

comment:3 by danielk, 19 years ago

(In [14597]) Refs #3987. Initializes a previously uninitialized variable.

I noticed this when checking hendrixski's documentation patch. We were actually using another variable to tell us if size had been initialized, but gcc doesn't know this and throws out warnings. There is no harm in initializing the size...

Note: See TracTickets for help on using tickets.