[PR] [IO-783] Remove NTFS ADS separator check in FilenameUtils.getExtension [commons-io]

2024-05-18 Thread via GitHub
baezzys opened a new pull request, #629: URL: https://github.com/apache/commons-io/pull/629 ### Summary This PR addresses issue [IO-783](https://issues.apache.org/jira/browse/IO-783) by removing the NTFS ADS separator (colon) check from the `FilenameUtils.getExtension` method to ensure

Re: [PR] Cleanup help formatting [commons-cli]

2024-05-18 Thread via GitHub
garydgregory commented on code in PR #277: URL: https://github.com/apache/commons-cli/pull/277#discussion_r1605818691 ## src/main/java/org/apache/commons/cli/HelpFormatter.java: ## @@ -129,6 +129,16 @@ public Builder setShowDeprecated(final boolean useDefaultFormat) {

Re: [PR] Cleanup help formatting [commons-cli]

2024-05-18 Thread via GitHub
garydgregory commented on code in PR #277: URL: https://github.com/apache/commons-cli/pull/277#discussion_r1605818691 ## src/main/java/org/apache/commons/cli/HelpFormatter.java: ## @@ -129,6 +129,16 @@ public Builder setShowDeprecated(final boolean useDefaultFormat) {

Re: [PR] Cleanup help formatting [commons-cli]

2024-05-18 Thread via GitHub
garydgregory commented on code in PR #277: URL: https://github.com/apache/commons-cli/pull/277#discussion_r1605797848 ## src/site/xdoc/usage.xml: ## @@ -466,145 +465,141 @@ catch (ParseException exp) { deprecated method. Usage of the deprecated op

Re: [PR] Cleanup help formatting [commons-cli]

2024-05-18 Thread via GitHub
garydgregory commented on code in PR #277: URL: https://github.com/apache/commons-cli/pull/277#discussion_r1605797762 ## src/main/java/org/apache/commons/cli/HelpFormatter.java: ## @@ -129,6 +129,14 @@ public Builder setShowDeprecated(final boolean useDefaultFormat) {

Re: [PR] Cleanup help formatting [commons-cli]

2024-05-18 Thread via GitHub
garydgregory commented on code in PR #277: URL: https://github.com/apache/commons-cli/pull/277#discussion_r1605797762 ## src/main/java/org/apache/commons/cli/HelpFormatter.java: ## @@ -129,6 +129,14 @@ public Builder setShowDeprecated(final boolean useDefaultFormat) {

Re: [PR] Cleanup help formatting [commons-cli]

2024-05-18 Thread via GitHub
Claudenw commented on code in PR #277: URL: https://github.com/apache/commons-cli/pull/277#discussion_r1605796694 ## src/site/xdoc/usage.xml: ## @@ -466,145 +465,141 @@ catch (ParseException exp) { deprecated method. Usage of the deprecated option

Re: [PR] Cleanup help formatting [commons-cli]

2024-05-18 Thread via GitHub
Claudenw commented on code in PR #277: URL: https://github.com/apache/commons-cli/pull/277#discussion_r1605796400 ## src/site/xdoc/usage.xml: ## @@ -466,145 +465,141 @@ catch (ParseException exp) { deprecated method. Usage of the deprecated option

Re: [PR] Cleanup help formatting [commons-cli]

2024-05-18 Thread via GitHub
Claudenw commented on code in PR #277: URL: https://github.com/apache/commons-cli/pull/277#discussion_r1605795923 ## src/main/java/org/apache/commons/cli/HelpFormatter.java: ## @@ -129,6 +129,14 @@ public Builder setShowDeprecated(final boolean useDefaultFormat) { }

Re: [PR] Improve memory allocation and validation [commons-imaging]

2024-05-18 Thread via GitHub
Marcono1234 commented on PR #401: URL: https://github.com/apache/commons-imaging/pull/401#issuecomment-2118834773 @garydgregory, yes I know that it has conflicts, and I can try to split this. My main goal was to show the general idea I had, the main important changes are in `Allocator` and

Re: [PR] Improve memory allocation and validation [commons-imaging]

2024-05-18 Thread via GitHub
garydgregory commented on PR #401: URL: https://github.com/apache/commons-imaging/pull/401#issuecomment-2118829400 @Marcono1234 Thank you for your PR but it conflicts with the current state of the master branch. As a general process issue, you should start with a smaller PR, 51 files

Re: [PR] Cleanup help formatting [commons-cli]

2024-05-18 Thread via GitHub
garydgregory commented on code in PR #277: URL: https://github.com/apache/commons-cli/pull/277#discussion_r1605789846 ## src/main/java/org/apache/commons/cli/HelpFormatter.java: ## @@ -129,6 +129,14 @@ public Builder setShowDeprecated(final boolean useDefaultFormat) {

[PR] Improve memory allocation and validation [commons-imaging]

2024-05-18 Thread via GitHub
Marcono1234 opened a new pull request, #401: URL: https://github.com/apache/commons-imaging/pull/401 This is a general proposal for how the commons-imaging code could better handle allocations to protect against malicious image data causing high memory consumption. The changes do not show a

Re: [PR] COLLECTIONS-854 : renaming bloomfilter components and methods as per ticket [commons-collections]

2024-05-18 Thread via GitHub
garydgregory commented on code in PR #492: URL: https://github.com/apache/commons-collections/pull/492#discussion_r1605789594 ## src/main/java/org/apache/commons/collections4/bloomfilter/BitMapExtractor.java: ## @@ -61,33 +61,33 @@ public boolean forEachBitMap(final LongPredica

[jira] [Created] (COLLECTIONS-855) Update the EnhancedDoubleHasher to correct the cube component of the hash

2024-05-18 Thread Alex Herbert (Jira)
Alex Herbert created COLLECTIONS-855: Summary: Update the EnhancedDoubleHasher to correct the cube component of the hash Key: COLLECTIONS-855 URL: https://issues.apache.org/jira/browse/COLLECTIONS-855

Re: [PR] COLLECTIONS-854 : renaming bloomfilter components and methods as per ticket [commons-collections]

2024-05-18 Thread via GitHub
Claudenw commented on code in PR #492: URL: https://github.com/apache/commons-collections/pull/492#discussion_r1605689991 ## src/main/java/org/apache/commons/collections4/bloomfilter/BitMapExtractor.java: ## @@ -61,33 +61,33 @@ public boolean forEachBitMap(final LongPredicate p