I'd make it raise a exception. It's bad that it sometimes goes
undetected and other times completely segfaults python.
on the other hand i've talked with pro-rsoft, he said pnmimage is
implemented to be as fast as possible. While adding a checking mechanism
in c/c++ would slow down the implementation where it needs to be fast,
it definately should be checked when accessing from python (if that's
possible).
Treeform wrote:
> *** This bug is a duplicate of bug 372253 ***
> https://bugs.launchpad.net/bugs/372253
>
> I wonder what it should return instead? Should it wrap the image or
> raise exception?
>
> On Tue, Jun 2, 2009 at 12:06 AM, pro-rsoft <email address hidden> wrote:
>
>> *** This bug is a duplicate of bug 372253 ***
>> https://bugs.launchpad.net/bugs/372253
>>
>> ** This bug has been marked a duplicate of bug 372253
>> pnmimage crashes if x/y coodinate outside image
>>
>> --
>> pnmimage crashes if x/y coodinate outside image
>> https://bugs.launchpad.net/bugs/372254
>> You received this bug notification because you are subscribed to
>> Panda3D.
>>
>>
>
>
I'd make it raise a exception. It's bad that it sometimes goes
undetected and other times completely segfaults python.
on the other hand i've talked with pro-rsoft, he said pnmimage is
implemented to be as fast as possible. While adding a checking mechanism
in c/c++ would slow down the implementation where it needs to be fast,
it definately should be checked when accessing from python (if that's
possible).
Treeform wrote: /bugs.launchpad .net/bugs/ 372253 /bugs.launchpad .net/bugs/ 372253 /bugs.launchpad .net/bugs/ 372254
> *** This bug is a duplicate of bug 372253 ***
> https:/
>
> I wonder what it should return instead? Should it wrap the image or
> raise exception?
>
> On Tue, Jun 2, 2009 at 12:06 AM, pro-rsoft <email address hidden> wrote:
>
>> *** This bug is a duplicate of bug 372253 ***
>> https:/
>>
>> ** This bug has been marked a duplicate of bug 372253
>> pnmimage crashes if x/y coodinate outside image
>>
>> --
>> pnmimage crashes if x/y coodinate outside image
>> https:/
>> You received this bug notification because you are subscribed to
>> Panda3D.
>>
>>
>
>