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 jfons...@vmware.com 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

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 commit

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 mar...@gmail.com 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

[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