Roy, Yes, adding a static_cast resolved the warnings. I do not use periodic boundaries and so cannot check if this has any adverse effects but if this is the way to go, attached is a trivial patch for the same.
And I just noticed that you said that it shouldn't match invalid_id which seems to be static casted to -1 too. Is this a problem ? And sorry again about the sleepy message yesterday.. Vijay On 1/25/11, Roy Stogner <[email protected]> wrote: > > It's a deliberate change of sign - that constructor is trying to > initialize the periodic boundary ids to something that won't match > either any real id or the invalid_id constant. Would you see if > adding an explicit cast to unsigned int gets rid of the warnings? > > Thanks, > --- > Roy > > On Mon, 24 Jan 2011, Vijay S. Mahadevan wrote: > >> Oops. Sorry about my previous message, if that passed through and was >> empty. >> >> I was trying to compile a newly downloaded version of libmesh with >> intel c++ compiler and saw these warnings. I guess I am a little tired >> and I cant seem to figure out exactly why I am seeing this warning. I >> hope it is not any local modifications in my working copy but I >> figured you will be interested since it looks like a serious warning. >> >> /home/mahadevan/karma/contrib/libmesh_wc/include/base/periodic_boundaries.h(56): >> warning #68: integer conversion resulted in a change of sign >> myboundary(-1), >> ^ >> >> /home/mahadevan/karma/contrib/libmesh_wc/include/base/periodic_boundaries.h(57): >> warning #68: integer conversion resulted in a change of sign >> pairedboundary(-1), >> ^ >> >> /home/mahadevan/karma/contrib/libmesh_wc/include/base/periodic_boundaries.h(78): >> warning #68: integer conversion resulted in a change of sign >> myboundary(-1), >> ^ >> >> /home/mahadevan/karma/contrib/libmesh_wc/include/base/periodic_boundaries.h(79): >> warning #68: integer conversion resulted in a change of sign >> pairedboundary(-1), >> >> If you want more information, let me know. I will look at this again >> and see if I can figure it out after I get some sleep.. >> >> Vijay >> >> On 1/24/11, Vijay S. Mahadevan <[email protected]> wrote: >>> >>> >> >> ------------------------------------------------------------------------------ >> Special Offer-- Download ArcSight Logger for FREE (a $49 USD value)! >> Finally, a world-class log management solution at an even better >> price-free! >> Download using promo code Free_Logger_4_Dev2Dev. Offer expires >> February 28th, so secure your free ArcSight Logger TODAY! >> http://p.sf.net/sfu/arcsight-sfd2d >> _______________________________________________ >> Libmesh-devel mailing list >> [email protected] >> https://lists.sourceforge.net/lists/listinfo/libmesh-devel >> >
periodic_boundaries.patch
Description: Binary data
------------------------------------------------------------------------------ Special Offer-- Download ArcSight Logger for FREE (a $49 USD value)! Finally, a world-class log management solution at an even better price-free! Download using promo code Free_Logger_4_Dev2Dev. Offer expires February 28th, so secure your free ArcSight Logger TODAY! http://p.sf.net/sfu/arcsight-sfd2d
_______________________________________________ Libmesh-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/libmesh-devel
