Eliminate (or at least reduce) use of MAX_PATH
Bug #1187537 reported by
Paul J. Lucas
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Zorba |
Fix Committed
|
Low
|
Paul J. Lucas | ||
Bug Description
fs_util.h defines MAX_PATH if it's not already defined. For starters, under *nix systems, it's PATH_MAX. Regardless, both macros should be avoided whenever possible since they're often inaccurate. Path length limits can be filesystem-
Related branches
lp:~paul-lucas/zorba/bug-1187537
- Juan Zacarias: Approve
- Matthias Brantner: Approve
- Paul J. Lucas: Approve
-
Diff: 1011 lines (+242/-217)19 files modifiedChangeLog (+1/-0)
bin/zorbacmd.cpp (+1/-1)
include/zorba/util/error_util.h (+5/-1)
include/zorba/util/fs_util.h (+24/-42)
src/context/dynamic_loader.cpp (+3/-10)
src/runtime/full_text/icu_tokenizer.cpp (+3/-1)
src/runtime/full_text/stemmer.cpp (+4/-1)
src/runtime/full_text/stemmer/sb_stemmer.cpp (+2/-0)
src/runtime/full_text/stemmer/sb_stemmer.h (+2/-0)
src/runtime/full_text/thesauri/xqftts_thesaurus.cpp (+5/-6)
src/util/error_util.cpp (+7/-4)
src/util/fs_util.cpp (+80/-69)
src/util/fs_util.h (+17/-20)
src/util/locale.cpp (+4/-20)
src/util/mmap_file.cpp (+3/-5)
src/util/string_util.h (+57/-0)
src/util/uri_util.cpp (+12/-11)
src/util/uri_util.h (+11/-25)
test/apitest.cpp (+1/-1)
Revision history for this message
| Paul J. Lucas (paul-lucas) wrote | #1 |
| Changed in zorba: | |
| status: | New → In Progress |
| Changed in zorba: | |
| status: | In Progress → Fix Committed |
To post a comment you must log in.
Bug #1180220 has to be fixed/merged before work on this can begin.