[PATCH] mklib: cosmetic: use case instead of expr match

2009-04-30 Thread Tormod Volden
From: Tormod Volden debian.tor...@gmail.com Saves forking an expr for every object. Signed-off-by: Tormod Volden debian.tor...@gmail.com --- On Thu, Apr 30, 2009 at 8:37 AM, Tormod Volden wrote: Dan Nicholson wrote: Is there any reason not to just use a case statement instead of forking expr?

Re: [PATCH] mklib: cosmetic: use case instead of expr match

2009-04-30 Thread Brian Paul
Thanks. I'll commit this and your prev patch after a bit more testing... -Brian Tormod Volden wrote: From: Tormod Volden debian.tor...@gmail.com Saves forking an expr for every object. Signed-off-by: Tormod Volden debian.tor...@gmail.com --- On Thu, Apr 30, 2009 at 8:37 AM, Tormod

Re: [PATCH] mklib: cosmetic: use case instead of expr match

2009-04-30 Thread Dan Nicholson
On Thu, Apr 30, 2009 at 3:57 PM, Brian Paul bri...@vmware.com wrote: Thanks.  I'll commit this and your prev patch after a bit more testing... I have the other one queued up, I just didn't have time to give it a spin yet. If you can wait until tomorrow (later tonight actually), I'll push them.

Re: [PATCH] mklib: cosmetic: use case instead of expr match

2009-04-30 Thread Brian Paul
Dan Nicholson wrote: On Thu, Apr 30, 2009 at 3:57 PM, Brian Paul bri...@vmware.com wrote: Thanks. I'll commit this and your prev patch after a bit more testing... I have the other one queued up, I just didn't have time to give it a spin yet. If you can wait until tomorrow (later tonight