[PATCH] Updated Legacy code in i386 pc386

[Vipul Nayyar]

Hello Peter,

Thanks for the heads up. I put the cast their initially because the compiler gave me a warning that interrupt handler install function argument 4 requires a rtems_interrupt_handler type. 

So can you tell me, what is the right way to go about this ? What kind of problems can be caused ? How do I make sure that I'm actually fixing that part of code ?

Regards
Vipul Nayyar 



________________________________
 From: Peter Dufault <dufault at hda.com>
To: Vipul Nayyar <nayyar_vipul at yahoo.com> 
Cc: Sebastian Huber <sebastian.huber at embedded-brains.de>; "rtems-devel at rtems.org" <rtems-devel at rtems.org> 
Sent: Wednesday, 24 July 2013 5:15 PM
Subject: Re: [PATCH] Updated Legacy code in i386 pc386
 




On Jul 24, 2013, at 06:08 , Vipul Nayyar <nayyar_vipul at yahoo.com> wrote:

I put (rtems_interrupt_handler) casts because I was getting warnings. After I removed them now, I think it's working fine. Don't know how.ecaus
Be extremely careful about using casting to eliminate compiler warnings.   That can just cause problems to be hidden instead of fixed.


Peter
-----------------
Peter Dufault
HD Associates, Inc.      Software and System Engineering 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20130724/19f908f1/attachment-0001.html>