On Wed, 10 Jul 2013, Derek Gaston wrote: > On Jul 10, 2013, at 11:05 PM, Roy Stogner <royst...@ices.utexas.edu> wrote: > >> I'd rather avoid adding a mandatory extra 8 bytes to every DofObject, >> but I don't see any insuperable obstacle to adding an optional 8 >> bytes; this could even be run-time optional since we already have to >> have that variable-size _idx_buf in there. The only practical >> obstacle is that the _idx_buf code is *already* scary complicated. > > At this point I would just prefer a compile time option. We are going > to have code connected to this thing in quite a few places in MOOSE so > I would really like to be able to do elem->unique_id or some such.
Compile time option is okay with me unless others disagree. Just make it elem->unique_id() so that we can hide the underlying data in the index buffer and move to a runtime option sometime later. > Are we at the point where I can gin something up and put in a pull > request and we can discuss from there? Fine by me. There are still issues we haven't discussed but I can't think of any really hairy problems. > I really don't think it's going to be much code to get started with. Agreed. I also suspect your first crack is going to break my finally-passing-buildbot 64-bit-dof_id_type tests, but I'll fix that when we come to it. :-) You might want to strip out PackedElem and PackedNode while you're at it; that's been superceded by the Parallel::*packed_range stuff for a while now, and it's probably easier to delete the older code than to update it. --- Roy ------------------------------------------------------------------------------ See everything from the browser to the database with AppDynamics Get end-to-end visibility with application monitoring from AppDynamics Isolate bottlenecks and diagnose root cause in seconds. Start your free trial of AppDynamics Pro today! http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk _______________________________________________ Libmesh-devel mailing list Libmesh-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/libmesh-devel
