changelog

android-w.song.android.widget

	  NULL/0 so code can check whether or not the prompt has been saved
	- change rl_message and rl_clear_message to save and restore the
	  prompt if the caller has not already done it (using a simple
	  semaphore-like variable)
	- change rl_message to call expand_prompt, so that local_prompt and
	  local_prompt prefix are set before calling the redisplay functions,
	  in case the prompt is longer than a screenwidth (fixes bug
	  reported to debian by [e-mail])

lib/readline/doc/rltech.texi
	- make sure to note that rl_save_prompt should be called before
	  rl_message, and rl_restore_prompt before rl_clear_message

pcomplete.c
	- make sure to save and restore the parser state around the call to
	  execute_shell_function in gen_shell_function_matches.  Fixes bug
	  reported by [e-mail] (cute)

lib/readline/readline.c
	- fix _rl_dispatch_subseq in the case where we're recursing back up
	  the chain (r == -2) and we encounter a key shadowed by a keymap,
	  but originally bound to self-insert.  Calling rl_dispatch with
	  ANYOTHERKEY as the first argument will call rl_insert, but with
	  ANYOTHERKEY (256) as the char to insert.  Use the shadow keymap
	  and set things up to dispatch to rl_insert with the shadowed key
	  as the argument.  Fixes the bug reported by Thomas Glanzmann
	  ([e-mail])

				   1/13
				   ----
command.h
	- new word flag: W_HASQUOTEDNULL

make_cmd.c
	- new function to allocate a WORD_DESC * without doing anything with a
	  containing string:  alloc_word_desc

make_cmd.h
	- extern declaration for alloc_word_desc

dispose_cmd.c
	- new function to just free a WORD_DESC * without freeing the contained
	  string:  dispose_word_desc

dispose_cmd.h
	- extern declaration for dispose_word_desc

subst.c
	- change some places to use alloc_word_desc
	- make same changes to word_list_quote_removal as were made to
	  word_list_split
	- set W_HASQUOTEDNULL when a word is created with w->word[0] ==
	  CTLNUL and w->word[1] == '\0'

subst.c
	- parameter_brace_expand_word now returns a WORD_DESC * -- changed
	  callers to understand
	- parameter_brace_expand_indir now returns a WORD_DESC * -- changed
	  callers to understand
	- parameter_brace_expand_rhs now returns a WORD_DESC * -- changed
	  callers to understand
	- remove W_HASQUOTEDNULL from a word's flags when remove_quoted_nulls
	  is called on the word's enclosed string

				   1/15
				   ----
subst.c
	- param_expand now returns a WORD_DESC * -- changed callers to
	  understand
	- parameter_brace_expand now returns a WORD_DESC * -- changed
	  callers to understand
	- in expand_word_internal, only call remove_quoted_nulls after a word
	  is returned with W_HASQUOTEDNULL
	- changes to pass W_HASQUOTEDNULL flag out of expand_word_internal;
	  changed callers to call remove_quoted_nulls only if return value has
	  W_HASQUOTEDNULL set.  This is a mostly-complete fix for the
	  long-standing CTLNUL confusion between a quoted null expansion and
	  the expansion of a variable with a literal '\177' in its value
	- change string_list_dollar_at to compute the separator character the
	  same way as string_list_dollar_star:  using the already-computed
	  values generated in setifs()
	- when expanding unquoted $*, if $IFS is empty, check whether or not
	  we're eventually going to split the results (e.g., on the rhs of an
	  assignment statement) and concatenate the positional parameters as
	  if the expansion were within double quotes if we're not going to
	  split

tests/iquote.tests
	- test cases based on old bug reports about the quoted-null vs. 0177
	  problem the recent code fixes

				   1/16
				   ----
dispose_cmd.c
	- set w->word to 0 before putting a WORD_DESC * back in the cache in
	  dispose_word_desc; changed callers to delete those assignments

variables.c
	- change assign_random and get_random_value so that the random number
	  generator only gets re-seeded once in a subshell environment, and
	  assigning a value to RANDOM counts as seeding the generator.  This
	  makes the sequences a little more predictable

				   1/20
				   ----
lib/readline/history.c
	- fix replace_history_entry, remove_history to return NULL if
	  passed index is < 0

				   1/22
				   ----
lib/sh/netconn.c
	- fix isnetconn() to understand that getpeername can return ENOTCONN
	  to indicate that an fd is not a socket

configure.in
	- set BUILD_DIR to contain backslashes to escape any spaces in the
	  directory name -- this is what make will accept in targets and
	  prerequisites, so it's better than trying to use double quotes
	- set SIZE to the appropriate value if some cross-compiling tool
	  chain is being used; `size' by default (can be overridden by
	  SIZE environment variable)

Makefile.in
	- use $(SIZE) instead of size; set SIZE from configure

				   1/31
				   ----
arrayfunc.c
	- in array_value_internal, return NULL right away if the variable's
	  value is NULL, instead of passing a null string to add_string_to_list

				    2/1
				    ---
jobs.h
	- new struct to hold stats and counters for child processes and jobs
	- change some uses of global and static variables to use members of
	  new struct (struct jobstats)

				    2/2
				    ---

jobs.[ch]
	- change PRUNNING to PALIVE
	- new define INVALID_JOB
	- new macro get_job_by_jid(ind), currently expands to jobs[ind]
	- new define J_JOBSTATE, operates on a JOB * like JOBSTATE operates on
	  a job index
	- new function, reset_job_indices, called from delete_job if
	  js.j_lastj or js.j_firstj are removed
	- change various functions to keep counters and stats in struct jobstats

pcomplete.c, builtins/common.c, builtins/{exit,fg_bg,jobs,kill,wait}.def
	- change global variables (e.g., job_slots) to struct members
	  (e.g., js.j_jobslots)
	- use INVALID_JOB define where appropriate
	- use get_job_by_jid and J_JOBSTATE where appropriate

trap.c
	- change reset_or_restore_signal_handler to not free the exit trap
	  string if the function pointer is reset_signal, which is used when
	  the trap strings shouldn't be freed, like in command substitution

				    2/4
				    ---
jobs.c
	- new function, realloc_jobs_list, copies jobs array to newly-allocated
	  memory shrinking (or growing) size to have next multiple of JOB_SLOTS
	  greater than js.j_njobs
	- change compact_jobs_list to just call reap_dead_jobs and then
	  realloc_jobs_list, simplifying it considerably
	- discard_pipeline now returns `int':  the number of processes freed
	- slightly changed the logic deciding whether or not to call
	  compact_jobs_list:  now non-interactive shells will compact the
	  list if it reaches MAX_JOBS_IN_ARRAY in size

parse.y
	- move test for backslash-newline after pop_string in shell_getc so
	  that things like

		((echo 5) \
		 (echo 6))

	  work right

				    2/8
				    ---
jobs.h
	- new structs for holding status of exited background processes, as
	  POSIX specifies
	- new job flag: J_ASYNC

jobs.c
	- new functions to manipulate struct holding status of exited
	  background processes
	- new members in struct jobstats to hold pointer to last created job
	  and last created asynchronous job
	- initialize js.c_childmax in initialize_job_control
	- if the `async' arg to stop_pipeline is non-null, set the J_ASYNC
	  flag in the job struct
	- set js.j_last_made_job and js.j_last_asynchronous_job in
	  stop_pipeline
	- new function: find_last_proc, returns the PROCESS * to the last proc
	  in a job's pipeline
	- changed find_last_pid to call find_last_proc
	- change delete_job to call bgp_add on the last proc of the job being
	  deleted
	- change delete_all_jobs and wait_for_background_pids to call bgp_clear

				    2/9
				    ---
jobs.c
	- change wait_for_single_pid to look for pid in bgpids.list (using
	  bgp_search()) if find_pipeline returns NULL

				   2/10
				   ----
support/shobj-conf
	- change the solaris-gcc stanza so that it auto-selects the appropriate
	  options for ld depending on which `ld' gcc says it's going to run

				   2/11
				   ----
jobs.h
	- add support for PS_RECYCLED as a process state, add PRECYCLED macro
	  to test it.  Change PALIVE and PRUNNING macros to not count processes
	  in PS_RECYCLED state

execute_cmd.c
	- restore use of last_pid as sentinel value; use NO_PID as sentinel
	  only if RECYCLES_PIDS is defined

jobs.c
	- change find_job to return a pointer to the PROCESS the desired pid
	  belongs to, analogous to find_pipeline returning pointer to JOB
	- change find_job callers to add extra argument
	- change running_only arguments to find_pipeline and find_job to
	  alive_only, since we don't want recycled pids returned here and it
	  better describes the result
	- new function find_process, calls find_pipeline and searches the
	  returned pipeline for the PROCESS * describing the desired pid
	- in make_child, if fork() returns the same pid as the value of
	  last_asynchronous_pid when RECYCLES_PIDS is defined, avoid pid
	  aliasing by resetting last_asynchronous_pid to 1
	- use PRUNNING instead of child->running, since we, for the most
	  part, don't want to consider recycled pids (e.g., in make_child())
	- call find_process instead of find_pipeline in waitchld()
	- use PEXITED(p) instead of testing p->running == PS_DONE
	- in make_child, call bgp_delete to remove a just-created pid from the
	  last of saved pid statuses
	- in add_process, check whether or not pid being added is already in
	  the_pipeline or the jobs list (using find_process) and mark it as
	  recycled if so
	- This set of fixes mostly came from Pierre Humblet
	  <[e-mail]> to fix pid aliasing and reuse problems on
	  cygwin

variables.c
	- set $_ from the environment if we get it there, set to $0 by
	  default if not in env

doc/{bashref.texi,bash.1}
	- a couple of clarifying changes to the description of $_ based on
	  comments from Glenn Morris <[e-mail]>

				   2/15
				   ----
shell.c
	- use strstr instead of strmatch when checking whether $EMACS contains
	  `term' -- simpler and faster

				   2/18
				   ----
builtins/cd.def
	- implement posix requirement that `pwd -P' set $PWD to a directory
	  name containing no symlinks
	- add new function, setpwd(), just sets (and changes exported value)
	  of PWD

doc/bashref.texi
	- add note to posix mode section about pwd -P setting $PWD

doc{bash.1,bashref.texi}
	- added note that BASH_ARGC and BASH_ARGV are only set in extended
	  debug mode
	- expand description of extdebug option to include everything changed
	  by extended debug mode

				   2/19
				   ----
pathexp.h
	- new flag macro, FNMATCH_IGNCASE, evaluates to FNM_CASEFOLD if the
	  match_ignore_case variable is non-zero

execute_cmd.c
	- new variable, match_ignore_case
	- change call to strmatch() in execute_case_command so it includes
	  FNMATCH_IGNCASE

test.c
	- change call to strmatch() in patcomp() so that pattern matching
	  calls for [[ ... ]] obey the match_ignore_case variable

lib/sh/shmatch.c
	- if match_ignore_case is set, enable REG_ICASE in the regexp match
	  flags

builtins/shopt.def
	- new settable option, `nocasematch', controls the match_ignore_case
	  variable.  Currently alters pattern matching for case and [[ ... ]]
	  commands (==, !=, and =~ operators)

doc/{bashref.texi,bash.1}
	- updated descriptions of [[ and case to include reference to
	  nocasematch option

				   2/22
				   ----
builtins/mkbuiltins.c
	- add `times' to the list of posix special builtins

				   2/23
				   ----
builtins/cd.def
	- posix mode no longer turns on effect of -P option on $PWD if a
	  directory is chosen from CDPATH

doc/bashref.texi
	- clarified that in posix mode, reserved words are not alias expanded
	  only in a reserved word context
	- removed item about cd, $CDPATH, and -P from posix mode section

				   2/24
				   ----
builtins/reserved.def
	- minor cleanups to the description of `if'

				    3/2
				    ---
subst.c
	- change list_string and get_word_from_string to explicitly treat an
	  IFS character that is not space, tab, or newline *and any adjacent
	  IFS white space* as a single delimiter, as SUSv3/XPG6 says

builtins/read.def
	- check whether or not the number of fields is exactly the same as
	  the number of variables instead of just assigning the rest of the
	  line (minus any trailing IFS white space) to the last variable.
	  This parses a field and checks whether or not it consumes all of
	  the input (including any trailing field delimiters), falling back
	  to the previous behavior if it does not.  This is what POSIX.2
	  specifies, I believe (and the consensus of the austin-group list).
	  This requires a few tests in read.tests to be changed: backslashes
	  escaping IFS whitespace characters at the end of input cause the
	  whitespace characters to be preserved in the value assigned to the
	  variable, and the trailing non-whitespace field delimiter issue

				    3/7
				    ---
configure.in
	- add -D_POSIX_SOURCE to the LOCAL_CFLAGS for Interix

				    3/8
				    ---
bashline.c
	- make bash_directory_expansion a void function, since it doesn't have
	  any return value

				    3/9
				    ---
builtins/read.def
	- when testing for a pipe, use `fd' instead of hard-coding 0, since we
	  can read from other file descriptors now

lib/sh/zread.c
	- in zsyncfd, only set lind and lused to 0 if the lseek succeeds.
	  If the lseek fails, we might steal input from other programs, but
	  a failed lseek won't cause us to erroneously discard input

				   3/11
				   ----
builtins/evalstring.c
	- don't allow parse_and_execute to short-circuit and call exec() if
	  the command's return value is being inverted

				   3/15