Re: [Mesa-dev] [PATCH] util: require debug options to be separated by commas

2011-01-27 Thread Dan Nicholson
On Wed, Jan 26, 2011 at 3:17 AM, José Fonseca wrote: > On Mon, 2011-01-24 at 20:52 -0800, Marek Olšák wrote: >> Let's assume there are two options with names such that one is a substring >> of another. Previously, if we only specified the longer one as a debug >> option, >> the shorter one would

Re: [Mesa-dev] [PATCH] util: require debug options to be separated by commas

2011-01-27 Thread Marek Olšák
On Wed, Jan 26, 2011 at 12:17 PM, José Fonseca wrote: > On Mon, 2011-01-24 at 20:52 -0800, Marek Olšák wrote: > > Let's assume there are two options with names such that one is a > substring > > of another. Previously, if we only specified the longer one as a debug > option, > > the shorter one w

Re: [Mesa-dev] [PATCH] util: require debug options to be separated by commas

2011-01-26 Thread José Fonseca
On Mon, 2011-01-24 at 20:52 -0800, Marek Olšák wrote: > Let's assume there are two options with names such that one is a substring > of another. Previously, if we only specified the longer one as a debug option, > the shorter one would be considered specified as well (because of strstr). > This com

Re: [Mesa-dev] [PATCH] util: require debug options to be separated by commas

2011-01-25 Thread Brian Paul
On Mon, Jan 24, 2011 at 9:52 PM, Marek Olšák wrote: > Let's assume there are two options with names such that one is a substring > of another. Previously, if we only specified the longer one as a debug option, > the shorter one would be considered specified as well (because of strstr). > This comm

[Mesa-dev] [PATCH] util: require debug options to be separated by commas

2011-01-24 Thread Marek Olšák
Let's assume there are two options with names such that one is a substring of another. Previously, if we only specified the longer one as a debug option, the shorter one would be considered specified as well (because of strstr). This commit fixes it by checking that each option is surrounded by com