Bug 2494 – refactor Slim::Control::Command::execute() to lookup table

Bug 2494 - refactor Slim::Control::Command::execute() to lookup table

Summary:

refactor Slim::Control::Command::execute() to lookup table

Status:	RESOLVED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Misc
Version:	6.5b1
Platform:	All All

Importance:	P2 enhancement (vote)
Target Milestone:	---
Assigned To:	Chris Doherty

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2005-11-04 13:14 UTC by Chris Doherty
Modified:	2008-09-15 14:39 UTC (History)
CC List:	0 users

See Also:
Category:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Chris Doherty 2005-11-04 13:14:06 UTC

tracking bug.

the execute() function is big and complicated; pull out the individual command
code segments into their own functions and leave execute() to do a lookup based
on the inputs, plus maybe some housekeeping.

one open question is whether anything elsewhere in the server actually uses the
return array from execute(); if not, that would simplify this task.

Comment 1 Fred 2006-01-07 17:11:12 UTC

We can close this, no?

Comment 2 KDF 2006-01-08 12:59:08 UTC

Fred feels this task is complete as of svn 5569, so marking as closed.  Remaining issues can be opened as new bugs so that specific issues can be targetted.