Opened 12 years ago

Closed 12 years ago

Last modified 12 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@… 12 years ago.
My first patch, a bunch of doxygen comments to add to MythThemedMenu?

Download all attachments as: .zip

Change History (4)

Changed 12 years ago by hendrixski@…

Attachment: firstPlugin.diff added

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

comment:1 Changed 12 years ago by danielk

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

comment:2 Changed 12 years ago by danielk

Resolution: fixed
Status: newclosed

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

comment:3 Changed 12 years ago by danielk

(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.