mirror of
https://github.com/ip7z/7zip.git
synced 2026-04-21 06:03:40 +00:00
1eca44e7a7
English This patch addresses the review feedback around Windows installer metadata and update-path handling, and removes a partially implemented UI surface that duplicated the existing multi-output path controls. Installer / ARP metadata - The installer previously wrote fork-specific hardcoded values into the Windows Uninstall registry entry (`DisplayName`, `DisplayVersion`, `Publisher`). - That created drift between the installed product metadata and the actual upstream 7-Zip version/author macros used elsewhere in the tree. - The patch removes the hardcoded fork values and derives the registry values from the existing 7-Zip definitions: - `DisplayName` now uses the existing versioned product string (`7-Zip 26.00 (x64)` on 64-bit builds). - `DisplayVersion` now comes from `MY_VERSION`. - `Publisher` now comes from `MY_AUTHOR_NAME`. - This makes the Add/Remove Programs entry consistent with the rest of the build metadata and removes the need for manual source edits on each branch-specific release. Compression dialog cleanup - The `Paths...` button and its dedicated dialog were not part of the stable workflow. - At this point the main compression dialog already exposes the supported multi-output behavior through the output-count selector plus the existing archive path rows and browse buttons. - Keeping a second path-editing dialog in parallel would add maintenance overhead and duplicate UI/state management without providing new functionality. - For that reason, this patch removes the incomplete/duplicate surface entirely: - the `Paths...` button was removed from the dialog resource, - the handler declaration/definition was removed, - the auxiliary `COutputArcPathsDialog` class was removed, - the associated resource IDs and helper parsing/serialization code were removed. - Functional capability is unchanged: users still configure multiple output targets through the existing archive path controls already present in the compression dialog. Update path deduplication - In `UpdateGUI.cpp`, `AddUniquePath()` previously compared paths with a case-sensitive equality check. - On Windows, this could fail to deduplicate logically identical paths that differ only by casing. - The comparison is now case-insensitive (`IsEqualTo_NoCase()`), which matches expected Windows path behavior more closely and prevents duplicate item entries in the update/compression path collection stage. - This is intentionally a minimal fix: it addresses case-only duplication without changing the broader path normalization rules. Working directory safety - `PrepareWorkingDirForArchivePath()` previously ignored the return value of `MyGetFullPathName()`. - If full-path resolution failed, the code could continue with an invalid or empty path and derive an incorrect working directory. - The function now exits early when `MyGetFullPathName()` fails, leaving `options.WorkingDir` empty. - This is a conservative change: the downstream update flow already has fallback behavior for an empty working directory, so the patch avoids propagating bad path state without altering the existing fallback model. Header consistency / non-functional cleanup - `LangUtils.h` now provides a non-`Z7_LANG` fallback declaration for `LangString_OnlyFromLangFile()`. - This is not a functional runtime change for normal localized builds, but it keeps the header interface consistent across configurations and avoids IDE/static-analysis mismatches where `Z7_LANG` is not visible to the parser. Behavioral impact - No archive format logic was changed. - No codec behavior was changed. - No new persistence format was introduced. - The only runtime behavior changes are: - correct Windows ARP metadata, - removal of dead/duplicate UI, - case-insensitive deduplication of update input paths, - safer handling of full-path resolution failure before working directory derivation. Validation - The change set was validated with targeted code search and diff review to ensure the removed dialog/button plumbing no longer has remaining references. - `git diff --check` was also run on the touched files. - A full build/test pass has not yet been run in this environment. Portuguese Este patch responde ao feedback de revisão relacionado aos metadados do instalador no Windows e ao tratamento de caminhos no fluxo de atualização, além de remover uma superfície de UI parcialmente implementada que duplicava os controles já existentes para múltiplos caminhos de saída. Instalador / metadados do item no Windows - O instalador estava gravando valores hardcoded e específicos do fork na chave de desinstalação do Windows (`DisplayName`, `DisplayVersion`, `Publisher`). - Isso criava divergência entre os metadados exibidos em “Aplicativos instalados / Programas e Recursos” e as macros oficiais de versão/autoria já utilizadas no restante do código-fonte. - O patch remove esses valores fixos do fork e passa a derivar os valores diretamente das definições já existentes do 7-Zip: - `DisplayName` agora usa a string versionada já existente do produto (`7-Zip 26.00 (x64)` em builds 64-bit). - `DisplayVersion` agora vem de `MY_VERSION`. - `Publisher` agora vem de `MY_AUTHOR_NAME`. - Com isso, a entrada instalada no Windows fica consistente com os demais metadados do build e deixa de exigir edição manual do código a cada release da branch. Limpeza do diálogo de compressão - O botão `Paths...` e a dialog auxiliar associada não faziam parte de um fluxo estável/finalizado. - Neste momento, a janela principal de compressão já expõe o comportamento suportado de múltiplas saídas por meio do seletor de quantidade de saídas, dos campos de caminho de arquivo e dos botões de browse já existentes. - Manter uma segunda dialog para edição textual desses caminhos aumentaria o custo de manutenção e duplicaria a lógica de estado/UI sem adicionar capacidade funcional nova. - Por esse motivo, este patch remove completamente essa superfície incompleta/duplicada: - o botão `Paths...` foi removido do recurso da dialog, - a declaração e a implementação do handler foram removidas, - a classe auxiliar `COutputArcPathsDialog` foi removida, - os IDs de recurso associados e os helpers internos de parsing/serialização dessa dialog também foram removidos. - A capacidade funcional permanece a mesma: o usuário continua podendo configurar múltiplos destinos pelos controles de caminhos de saída já existentes na própria janela de compressão. Deduplicação de caminhos no fluxo de atualização - Em `UpdateGUI.cpp`, a função `AddUniquePath()` comparava caminhos usando igualdade case-sensitive. - Em Windows, isso podia falhar em deduplicar caminhos logicamente idênticos que diferiam apenas por maiúsculas/minúsculas. - A comparação agora é case-insensitive (`IsEqualTo_NoCase()`), o que se alinha melhor ao comportamento esperado para caminhos no Windows e evita entradas duplicadas durante a coleta dos caminhos de itens no fluxo de atualização/compressão. - A alteração foi mantida propositalmente mínima: ela corrige a duplicação por diferença de casing sem alterar as regras mais amplas de normalização de caminhos. Segurança na preparação do diretório de trabalho - `PrepareWorkingDirForArchivePath()` ignorava o valor de retorno de `MyGetFullPathName()`. - Se a resolução para caminho absoluto falhasse, o código poderia continuar com um caminho inválido ou vazio e, a partir disso, derivar um diretório de trabalho incorreto. - Agora a função retorna imediatamente quando `MyGetFullPathName()` falha, deixando `options.WorkingDir` vazio. - Essa mudança é conservadora: o fluxo subsequente de atualização já possui fallback quando o diretório de trabalho está vazio, então o patch evita propagar um estado inconsistente sem alterar o modelo de fallback já existente. Consistência de headers / limpeza não funcional - `LangUtils.h` agora fornece um fallback para `LangString_OnlyFromLangFile()` também quando `Z7_LANG` não está definido. - Isso não altera o comportamento funcional em builds localizados normais, mas mantém a interface do header consistente entre configurações e evita divergências em IDEs/analisadores estáticos quando `Z7_LANG` não está visível para o parser. Impacto comportamental - Nenhuma lógica de formato de arquivo foi alterada. - Nenhum comportamento de codec foi alterado. - Nenhum novo formato de persistência foi introduzido. - As únicas mudanças observáveis em runtime são: - correção dos metadados exibidos pelo Windows para a instalação, - remoção de uma UI morta/duplicada, - deduplicação case-insensitive de caminhos no fluxo de atualização, - tratamento mais seguro de falha ao resolver caminho absoluto antes de derivar o diretório de trabalho. Validação - O conjunto de alterações foi validado com buscas direcionadas no código e revisão de diff para garantir que a plumbing removida da dialog/botão não deixou referências residuais. - Também foi executado `git diff --check` nos arquivos alterados. - Ainda não foi executado um ciclo completo de build/testes neste ambiente. |
||
|---|---|---|
| .. | ||
| Util | ||
| 7z.h | ||
| 7zAlloc.c | ||
| 7zAlloc.h | ||
| 7zArcIn.c | ||
| 7zBuf.c | ||
| 7zBuf.h | ||
| 7zBuf2.c | ||
| 7zCrc.c | ||
| 7zCrc.h | ||
| 7zCrcOpt.c | ||
| 7zDec.c | ||
| 7zFile.c | ||
| 7zFile.h | ||
| 7zip_gcc_c.mak | ||
| 7zStream.c | ||
| 7zTypes.h | ||
| 7zVersion.h | ||
| 7zVersion.rc | ||
| 7zWindows.h | ||
| Aes.c | ||
| Aes.h | ||
| AesOpt.c | ||
| Alloc.c | ||
| Alloc.h | ||
| Asm_c.mak | ||
| Bcj2.c | ||
| Bcj2.h | ||
| Bcj2Enc.c | ||
| Blake2.h | ||
| Blake2s.c | ||
| Bra.c | ||
| Bra.h | ||
| Bra86.c | ||
| BraIA64.c | ||
| BwtSort.c | ||
| BwtSort.h | ||
| Compiler.h | ||
| CpuArch.c | ||
| CpuArch.h | ||
| Delta.c | ||
| Delta.h | ||
| DllSecur.c | ||
| DllSecur.h | ||
| HuffEnc.c | ||
| HuffEnc.h | ||
| LzFind.c | ||
| LzFind.h | ||
| LzFindMt.c | ||
| LzFindMt.h | ||
| LzFindOpt.c | ||
| LzHash.h | ||
| Lzma2Dec.c | ||
| Lzma2Dec.h | ||
| Lzma2DecMt.c | ||
| Lzma2DecMt.h | ||
| Lzma2Enc.c | ||
| Lzma2Enc.h | ||
| Lzma86.h | ||
| Lzma86Dec.c | ||
| Lzma86Enc.c | ||
| LzmaDec.c | ||
| LzmaDec.h | ||
| LzmaEnc.c | ||
| LzmaEnc.h | ||
| LzmaLib.c | ||
| LzmaLib.h | ||
| Md5.c | ||
| Md5.h | ||
| MtCoder.c | ||
| MtCoder.h | ||
| MtDec.c | ||
| MtDec.h | ||
| Ppmd.h | ||
| Ppmd7.c | ||
| Ppmd7.h | ||
| Ppmd7aDec.c | ||
| Ppmd7Dec.c | ||
| Ppmd7Enc.c | ||
| Ppmd8.c | ||
| Ppmd8.h | ||
| Ppmd8Dec.c | ||
| Ppmd8Enc.c | ||
| Precomp.h | ||
| RotateDefs.h | ||
| Sha1.c | ||
| Sha1.h | ||
| Sha1Opt.c | ||
| Sha3.c | ||
| Sha3.h | ||
| Sha256.c | ||
| Sha256.h | ||
| Sha256Opt.c | ||
| Sha512.c | ||
| Sha512.h | ||
| Sha512Opt.c | ||
| Sort.c | ||
| Sort.h | ||
| SwapBytes.c | ||
| SwapBytes.h | ||
| Threads.c | ||
| Threads.h | ||
| var_clang.mak | ||
| var_clang_arm64.mak | ||
| var_clang_x64.mak | ||
| var_clang_x86.mak | ||
| var_gcc.mak | ||
| var_gcc_arm64.mak | ||
| var_gcc_x64.mak | ||
| var_gcc_x86.mak | ||
| var_mac_arm64.mak | ||
| var_mac_x64.mak | ||
| warn_clang.mak | ||
| warn_clang_mac.mak | ||
| warn_gcc.mak | ||
| Xxh64.c | ||
| Xxh64.h | ||
| Xz.c | ||
| Xz.h | ||
| XzCrc64.c | ||
| XzCrc64.h | ||
| XzCrc64Opt.c | ||
| XzDec.c | ||
| XzEnc.c | ||
| XzEnc.h | ||
| XzIn.c | ||
| ZstdDec.c | ||
| ZstdDec.h | ||