Hi Seb, Respectfully, may I ask you to explain your argument for modifying a working / tested routine that is in use in the field by multiple boards?
When possible, please discuss your proposed changes prior to check- in. Consider how you'd feel if I started modifying the minix routines without making you aware of it first? I'm not sure I can even track down all of the installations that are using this routine in the field, much less take time to re-test. I'm not worried about samples, but real applications that utilize JALLIB. Some of these are in locations that are thousands of miles away from my location. Thank you, William On May 5, 6:42 am, Sebastien Lelong <[email protected]> wrote: > Le 5 mai 2011 à 13:09, William <[email protected]> a écrit : > > > Hi Matt, > > > On May 5, 12:39 am, mattschinkel <[email protected]> wrote: > > >> can_mcp2515 does not use the new procedures. The init procedure could > >> be left as it was if that is what you prefer. > > > Yes, that would be my preference -- please restore the init routine to > > the original > > Why? Please give arguments ! > > Seb > > > > > > > > > > >> Please leave the new procedures as they are, if you add pragma inline > >> to the new set mode procedure we will add code/data space to compiled > >> samples. > > > Here's an idea for you -- add 4 mode routines, and then inline each > > one -- will save code space and run faster. > > > William > > > -- > > You received this message because you are subscribed to the Google > > Groups "jallib" group. > > To post to this group, send email to [email protected]. > > To unsubscribe from this group, send email to > > [email protected] > > . > > For more options, visit this group > > athttp://groups.google.com/group/jallib?hl=en > > . -- You received this message because you are subscribed to the Google Groups "jallib" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/jallib?hl=en.
