| Re: [AD] P3D merge cleanup patch |
[ Thread Index |
Date Index
| More lists.liballeg.org/allegro-developers Archives
]
On 10 Jun 2001, Bertrand Coconnier <bcoconni@xxxxxxxxxx> wrote:
> I have not posted any patch to the list for a while...
> Attached to this mail is a patch which cleans up the P3D merge :
...
> @@int @scene_polygon3d(int type, BITMAP *texture, int vc, V3D *vtx[]);
> @@int @scene_polygon3d_f(int type, BITMAP *texture, int vc, V3D_f *vtx[]);
...
> - Returns zero on success, a negative number if there is no space in the
> - edge or polygon tables, or a positive number if it won't be rendered for
> + Returns zero on success, or a positive number if it won't be rendered for
> lack of a rendering routine.
I think the positive number is an artifact of the old API. Would it
be better to change it to a negative or "non-zero" number? Other
than that, I'll apply this soon.