Please do not reply to this email: if you want to comment on the bug, go to
the URL shown below and enter yourcomments there.
https://bugs.freedesktop.org/show_bug.cgi?id=8283
------- Additional Comments From [EMAIL PROTECTED] 2006-09-15 00:33 -------
Firstly, the approach looks like a good one, but a couple of comments:
- Please use universal diffs (diff -u) as they're easier for others to
read...
- It looks like you don't seem to cope with the situation where
get_free_rreg() fails, ie. the return value isn't checked.
- There should already be a 'NumInstructions' field hanging around, so you
shouldn't need to scan the list twice.
- It would be good to move the calculation of temporaries used into core
mesa somewhere, perhaps taking the regsNeeded array as an argument. But I don't
think this is a big deal at this point.
Otherwise, it looks good - nice work.
--
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
--
_______________________________________________
Dri-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dri-devel
