]> sourceware.org Git - newlib-cygwin.git/commit
Cygwin: path_conv: decouple path_types from mount types
authorCorinna Vinschen <corinna@vinschen.de>
Sun, 6 Jan 2019 16:44:20 +0000 (17:44 +0100)
committerCorinna Vinschen <corinna@vinschen.de>
Sun, 6 Jan 2019 19:30:14 +0000 (20:30 +0100)
commitc1023ee353705671aa9a8e4e1179022277add2aa
treef999d1038b40a48b2b358d3878ab969cd3ed45e6
parentfde4eaa105912625b6a78fba79d1fc39ee0fd201
Cygwin: path_conv: decouple path_types from mount types

- Remove another unfortunate amalgamation: Mount flags (MOUNT_xxx)
  are converted to path_types (PATH_xxx) and mixed with non-mount
  path_types flags in the same storage, leading to a tangled,
  pell-mell usage of mount flags and path flags in path_conv and
  symlink_info.

- There's also the case of PC_NONULLEMPTY.  It's used in exactly
  one place with a path_conv constructor only used in this single
  place, just to override the automatic PC_NULLEMPTY addition
  when calling the other path_conv constructors.  Crazily,
  PC_NONULLEMPTY is a define, no path_types flag, despite its
  name.

- It doesn't help that the binary flag exists as mount and path
  flag, while the text flag only exists as path flag.  This leads
  to mount code using path flags to set text/binary.  Very confusing
  is the fact that a text mount/path flag is not actually required;
  the mount code sets the text flag on non binary mounts anyway, so
  there are only two states.  However, to puzzle people a bit more,
  path_conv::binary wrongly implies there's a third, non-binary/non-text
  state.

Clean up this mess:

- Store path flags separately from mount flags in path_conv and
  symlink_info classes and change all checks and testing inline
  methods accordingly.

- Make PC_NONULLEMPTY a simple path_types flag and drop the
  redundant path_check constructor.

- Clean up the definition of pathconv_arg, path_types, and mount flags.
  Use _BIT expression, newly define in cygwin/bits.h.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
winsup/cygwin/include/cygwin/bits.h [new file with mode: 0644]
winsup/cygwin/include/sys/mount.h
winsup/cygwin/mount.cc
winsup/cygwin/path.cc
winsup/cygwin/path.h
winsup/utils/path.cc
This page took 0.031993 seconds and 5 git commands to generate.