Skip to content
This repository was archived by the owner on Sep 2, 2021. It is now read-only.

Conversation

@JeffryBooher
Copy link
Contributor

@ghost ghost assigned redmunds Dec 6, 2013
@bchintx
Copy link
Contributor

bchintx commented Dec 6, 2013

@redmunds I noticed that there were merge conflicts w/ my other pull request. So, I took the liberty of updating this branch as well, so that it can be Merged.

@redmunds
Copy link
Contributor

redmunds commented Dec 6, 2013

Thanks.

@redmunds
Copy link
Contributor

redmunds commented Dec 6, 2013

@JeffryBooher This change increases the buffer size from 260 to 4096 which allows brackets-shell to handle longer file paths.

But the bug report states: "... the buffer should be large enough to accommodate collections of paths. In my particular installation, I needed to support several thousand file selections to deal with image sequences."

Does this add support for collections of paths that support "several thousand file selections" as requested in the bug?

@njx
Copy link

njx commented Dec 6, 2013

I wonder what the use case for selecting a thousand files in the Open dialog is...seems a bit outside our wheelhouse.

@redmunds
Copy link
Contributor

redmunds commented Dec 6, 2013

@njx I'd like to get feedback from original filer about a usage scenario, but I think this should probably be merged to get the longer file paths. Agreed?

@njx
Copy link

njx commented Dec 6, 2013

Yup, I definitely think this makes sense to merge--was just thinking that we don't really need to generalize it for the "thousands of paths" case yet until we understand why that's valuable.

@redmunds
Copy link
Contributor

redmunds commented Dec 6, 2013

Looks good. Merging.

redmunds added a commit that referenced this pull request Dec 6, 2013
Update all path buffers to accept long UNC paths
@redmunds redmunds merged commit 08ff9ab into master Dec 6, 2013
@redmunds redmunds deleted the jeff/fix-5270 branch December 6, 2013 23:00
@JeffryBooher
Copy link
Contributor Author

@redmunds @njx this change works
For collections because the File
Open dialog uses a null terminated strings in the buffer to delineate each entry and a double null to terminate the string. The first entry is the path followed by each individual file name selected so it doesn't blow out the buffer size requirements to handle multiple file selections. Hipefully this is big enough to handle most cases.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

appshell.fs.showOpenDialog: Windows wchar_t buffer is too small

5 participants