On Tue, Dec 10, 2013 at 06:31:55PM -0800, Matt Turner wrote:
On Tue, Dec 10, 2013 at 5:55 PM, Francisco Jerez curroje...@riseup.net
wrote:
It just seems annoying and unnecessary to me to have to repeat the same
pattern before each use of unreachable(). If by definition the
unreachable
On 12/10/2013 03:27 PM, Francisco Jerez wrote:
Brian Paul bri...@vmware.com writes:
On 12/10/2013 11:06 AM, Francisco Jerez wrote:
Paul Berry stereotype...@gmail.com writes:
On 10 December 2013 08:42, Francisco Jerez curroje...@riseup.net wrote:
Brian Paul bri...@vmware.com writes:
On
Brian Paul bri...@vmware.com writes:
On 12/10/2013 03:27 PM, Francisco Jerez wrote:
Brian Paul bri...@vmware.com writes:
[...]
I'm more convinced now that we should simply have an assert there rather
than unreachable().
Finally, in release builds we generally don't want to crash in these
On 11 December 2013 08:56, Brian Paul bri...@vmware.com wrote:
On 12/10/2013 03:27 PM, Francisco Jerez wrote:
Brian Paul bri...@vmware.com writes:
On 12/10/2013 11:06 AM, Francisco Jerez wrote:
Paul Berry stereotype...@gmail.com writes:
On 10 December 2013 08:42, Francisco Jerez
Paul Berry stereotype...@gmail.com writes:
[...]
I'm glad you're dragging this out, Brian. IMHO we have to think carefully
about these sorts of issues to keep the quality of Mesa high.
Now that I've spent a day ruminating about this, I've changed my mind
somewhat. The question at hand is
On 11 December 2013 12:37, Francisco Jerez curroje...@riseup.net wrote:
Paul Berry stereotype...@gmail.com writes:
[...]
I'm glad you're dragging this out, Brian. IMHO we have to think
carefully
about these sorts of issues to keep the quality of Mesa high.
Now that I've spent a day
Brian Paul bri...@vmware.com writes:
On 12/07/2013 06:17 PM, Francisco Jerez wrote:
[...]
+ default:
+ unreachable();
I think I'd like to see an assertion or _mesa_problem() call to catch
unhandled cases if new texture targets are added in the future.
How about having the
On 12/10/2013 09:42 AM, Francisco Jerez wrote:
Brian Paul bri...@vmware.com writes:
On 12/07/2013 06:17 PM, Francisco Jerez wrote:
[...]
+ default:
+ unreachable();
I think I'd like to see an assertion or _mesa_problem() call to catch
unhandled cases if new texture targets are added
On 10 December 2013 08:42, Francisco Jerez curroje...@riseup.net wrote:
Brian Paul bri...@vmware.com writes:
On 12/07/2013 06:17 PM, Francisco Jerez wrote:
[...]
+ default:
+ unreachable();
I think I'd like to see an assertion or _mesa_problem() call to catch
unhandled cases
On 12/10/2013 11:06 AM, Francisco Jerez wrote:
Paul Berry stereotype...@gmail.com writes:
On 10 December 2013 08:42, Francisco Jerez curroje...@riseup.net wrote:
Brian Paul bri...@vmware.com writes:
On 12/07/2013 06:17 PM, Francisco Jerez wrote:
[...]
+ default:
+ unreachable();
On Tue, Dec 10, 2013 at 1:31 PM, Brian Paul bri...@vmware.com wrote:
On 12/10/2013 11:06 AM, Francisco Jerez wrote:
Paul Berry stereotype...@gmail.com writes:
On 10 December 2013 08:42, Francisco Jerez curroje...@riseup.net wrote:
Brian Paul bri...@vmware.com writes:
On 12/07/2013 06:17
Brian Paul bri...@vmware.com writes:
On 12/10/2013 11:06 AM, Francisco Jerez wrote:
Paul Berry stereotype...@gmail.com writes:
On 10 December 2013 08:42, Francisco Jerez curroje...@riseup.net wrote:
Brian Paul bri...@vmware.com writes:
On 12/07/2013 06:17 PM, Francisco Jerez wrote:
[...]
Matt Turner matts...@gmail.com writes:
On Tue, Dec 10, 2013 at 1:31 PM, Brian Paul bri...@vmware.com wrote:
On 12/10/2013 11:06 AM, Francisco Jerez wrote:
Paul Berry stereotype...@gmail.com writes:
On 10 December 2013 08:42, Francisco Jerez curroje...@riseup.net wrote:
Brian Paul
On Tue, Dec 10, 2013 at 5:55 PM, Francisco Jerez curroje...@riseup.net wrote:
It just seems annoying and unnecessary to me to have to repeat the same
pattern before each use of unreachable(). If by definition the
unreachable macro wants to be accompanied with an assert, why don't we
save
On 12/07/2013 06:17 PM, Francisco Jerez wrote:
And to check if it can have layers at all. This will be used by the
implementation of ARB_shader_image_load_store.
---
src/mesa/main/teximage.c | 85
src/mesa/main/teximage.h | 6
2 files
On 7 December 2013 17:17, Francisco Jerez curroje...@riseup.net wrote:
And to check if it can have layers at all. This will be used by the
implementation of ARB_shader_image_load_store.
---
src/mesa/main/teximage.c | 85
And to check if it can have layers at all. This will be used by the
implementation of ARB_shader_image_load_store.
---
src/mesa/main/teximage.c | 85
src/mesa/main/teximage.h | 6
2 files changed, 91 insertions(+)
diff --git
17 matches
Mail list logo