[Nmh-workers] multiple -prefer options

[Paul Fox]

okay -- that was silly of me to say i didn't have time.

i have a patch that changes the priority of multiple -prefer switches
to ascending, and also fixes the man pages.  i won't push to master
yet, so as not to step on anyone's toes.

but a question:  even with this, is there still value in a -noprefer
option, to force the default behavior?  my use of -prefer is very
simple (just one -prefer switch, only used by a wrapper script) so i
think i'm not a good judge.

norm -- will the proposed priority order change (which lets -prefer
switches on the command line override those in .mh_profile) solve your
problem?

paul

i wrote:
 > ken wrote:
 >  > >Ken wrote:
 >  > >> Paul Fox, what say you?
 >  > >
 > 
 > thanks for mentioning my name.  i'd forgotten that i'd had anything
 > to do with -prefer.  :-)
 > 
 >  > >There's also the option of reversing the order, as in my earlier email.
 >  > 
 >  > I'd be fine with either.  But ... adding -noprefer is fine for post-1.7
 >  > for me.  But reversing the order post-1.7 seems wrong to me.
 > 
 > remind me: is -prefer not in 1.6?  from your comment, i guess it's not.
 > 
 > we definitely don't want to change the order later.
 > 
 > adding -noprefer would be the simpler change.
 > 
 > that being said, prefer_parts() in mhparse.c goes out of its way to
 > run the list of parts backwards, so that the first shall be last, so
 > to speak.  reversing that loop to run forwards would be trivial.
 > 
 > i think i agree with ralph that the order should be changed -- the MH
 > standard is for later options to override prior ones, which i'm sure i
 > didn't consider while implementing.
 > 
 > the man pages for mhshow/mhlist do currently give the ordering:  "The
 > -prefer switch can be used (one or more times, in order of descending
 > preference) to let MH know...".  it's not in mhstore(1), which refers
 > to the other two for more detail.  so the man pages must change, and
 > should probably also make explicit what happens when -prefer is used
 > both in .mh_profile and on the command line.
 > 
 > though this is my "fault", and i'd normally dive right in, i'm afraid
 > this isn't the best time for me to take on pre-release changes -- i'm
 > a little rusty, git-wise, and don't actually have a lot of time at the
 > moment anyway.
 > 
 > paul
 > =----------------------
 > paul fox, address@hidden (arlington, ma, where it's 55.8 degrees)
 > 
 > 
 > _______________________________________________
 > Nmh-workers mailing list
 > address@hidden
 > https://lists.nongnu.org/mailman/listinfo/nmh-workers
 > 


=----------------------
paul fox, address@hidden (arlington, ma, where it's 53.2 degrees)