Dart-code: Reduz a dependência de variáveis ​​passadas para adaptadores de depuração

Criado em 10 set. 2020  ·  3Comentários  ·  Fonte: Dart-Code/Dart-Code

Existem muitas variáveis ​​passadas para launchRequest como dartVersion e flutterVersion que são fornecidas apenas pelo código VS. Para tornar o adaptador de depuração mais reutilizável, essas coisas devem ser calculadas dentro do adaptador de depuração. Todas as variáveis ​​também devem ter padrões razoáveis.

in debugger is enhancement

Todos 3 comentários

@ericdallo FYI - como parte deste trabalho, estou removendo dartPath , flutterPath , pubSnapshotPath , dartVersion , flutterVersion do lançar argumentos para o adaptador de depuração e substituí-los por apenas dartSdkPath e flutterSdkPath . No momento, você está definindo alguns desses aqui .

Quando a maioria deles foi adicionada, o VS Code era o único consumidor, então era conveniente apenas transmiti-los. No entanto, o uso do adaptador de depuração fica mais complicado quando muitos dos valores podem ser calculados pelo adaptador de depuração. Isso deve simplificar um pouco as coisas (também estou revisando os padrões e certificando-me de que a maioria dos valores são totalmente opcionais).

(Observação: o uso deste adaptador de depuração por clientes não-VS Code não é "oficialmente" suportado, embora eu gostaria de garantir que funcione e não seja muito pesado! 🙂).

Muito obrigado @DanTup , concordo totalmente!
Vou atualizar lsp-dart e remover as variáveis ​​desnecessárias :)

Desculpe pela demora @DanTup , acabei de bater lsp-dart para usar Dart-Code extensão de depurador 3.17.0 removendo as variáveis ​​não utilizadas substituídas por dartSdkPath e flutterSdkPath , obrigado!
https://github.com/emacs-lsp/lsp-dart/commit/a2c30213326cbac11d805e94c770eec76d0e3624

Esta página foi útil?
0 / 5 - 0 avaliações

Questões relacionadas

shamrin picture shamrin  ·  5Comentários

FeimiSzy picture FeimiSzy  ·  4Comentários

lukepighetti picture lukepighetti  ·  4Comentários

ifredom picture ifredom  ·  3Comentários

e200 picture e200  ·  3Comentários