Hey Hermet,
> if ((p[i] == '-') || (m == 256)) // 256 ?
> {
> dest[n][m] = 0; // If the m is 256 then it will be out
> of boundary.
> // Or if the m will never be more than
> 256, then we don't need to compare "m == 256"
Looks like either a typo, or just someone overlooking something, no idea
really.
> ----------------------------------------------------------------------------
> ------------------------------------------
> Second is here. It's not critical but looks strange.
> for ( ; src < end ; src ++, dst ++) // what does dst++ purpose for ?
> *dst = CONVERT_ARGB_8888_TO_A_8(*src); //Ok, This can be done.
This is actually pretty common in the C world. It advances both the src
and the dst by one each time and thus copying all the cells of the array
one by one.
--
Tom.
------------------------------------------------------------------------------
The ultimate all-in-one performance toolkit: Intel(R) Parallel Studio XE:
Pinpoint memory and threading errors before they happen.
Find and fix more than 250 security defects in the development cycle.
Locate bottlenecks in serial and parallel code that limit performance.
http://p.sf.net/sfu/intel-dev2devfeb
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
