Am 09.08.2011 um 06:15 schrieb kevin.buck...@ecs.vuw.ac.nz: >>> I think the original author tried to avoid several if-then-else tests >>> there, but maybe such an implementation is the only one being correct. >>> >>> -- Reuti >> >> You mean something like this, which does away with the macros >> altogether. >> ... > > And now find attached a patch for the > > static void pushlimit(int resource, struct RLIMIT_STRUCT_TAG *rlp, > int trace_rlimit) > > procedure from > > source/daemons/shepherd/setrlimits.c > > that does away with the backspace construct. > > The "many eyes" will no doubt also note that in the block near > the end for > > #if defined(NECSX4) || defined(NECSX5) > > the string that got created following a failure was never presented
What do you mean in detail here? You removed the clause? -- Reuti > to a shepherd_trace() for output, so I've fixed that too ! > > The patch is made against a ge-V62u5_TAG source tree. > > Hoping this is useful: it is for me, > Kevin > > -- > Kevin M. Buckley Room: CO327 > School of Engineering and Phone: +64 4 463 5971 > Computer Science > Victoria University of Wellington > New Zealand > <setrlimits.c.diff>_______________________________________________ > users mailing list > users@gridengine.org > https://gridengine.org/mailman/listinfo/users _______________________________________________ users mailing list users@gridengine.org https://gridengine.org/mailman/listinfo/users
