[cairo-bugs] [Bug 4722] Multistop gradients only work if the stops are equally spaced and end in offsets 0.0 and 1.0.

[bugzilla-daemon at freedesktop.org]

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=4722          
     




------- Additional Comments From jlt_fdo at shamrock.dyndns.org  2005-10-11 04:26 -------
(In reply to comment #3)
> (In reply to comment #2)
> > Created an attachment (id=3521)
 --> (https://bugs.freedesktop.org/attachment.cgi?id=3521&action=view) [edit] [edit]
> > Also Fix the comment.
> 
> Looks good.  Could you read through the CODING_STYLE document and update your
> patch to match those guidelines and to apply to CVS head?

Sure.

> Also, I was thinking that we should just always allocate an array of size
> n_steps + 2 cairo_pdf_color_stop_t's so we avoid the reallocation, and I you can
> add the pdf object id to the cairo_pdf_color_stop_t struct we avoid allocating
> the linear_ids array.

I had considerred allocating an oversized array as well and decided against it
since I considered a gradient wih stops at 0.0 and 1.0 the by far most common
case.  Same goes for the linear_ids.  If you prefer it the other way, I will be
happy to change it though.

Best Regards
Jens          
     
     
--           
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email         
     
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.