redhat-developer / redhat-developer/vscode-java

Eclipse formatter option parentheses_positions_in_method_invocation not considered regardless its value

オープン
#1,986 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

bug formatter
主要言語
TypeScript
スター
2.3k
フォーク
546
平均マージ
20時間 1分
マージ済み PR(30日)
11

説明

Java code formatter in VSCode is not yet supporting the org.eclipse.jdt.core.formatter.parentheses_positions_in_method_invocation option. And identical issue was raised here: https://github.com/revelc/formatter-maven-plugin/issues/343.

Can it be that VSCode uses an older version of the Eclipse formatter plugin?

Environment
  • Operating System: MacOS
  • JDK version: 15
  • Java extension version: rehhat.java v0.79.2
  • Visual Studio Code version:
    Version: 1.57.0
    Commit: b4c1bd0a9b03c749ea011b06c6d2676c8091a70c
    Date: 2021-06-09T17:22:31.215Z (1 wk ago)
    Electron: 12.0.9
    Chrome: 89.0.4389.128
    Node.js: 14.16.0
    V8: 8.9.255.25-electron.0
    OS: Darwin x64 19.6.0
Steps To Reproduce
method(
    arg1,
    arg2,
    arg3
);

is always formatted to:

method(
    arg1,
    arg2,
    arg3);

regardless of the value given to the Eclipse formatter option org.eclipse.jdt.core.formatter.parentheses_positions_in_method_invocation (set to preserve_positions).

It would be nice to at least not join wrapped lines according to <setting id="org.eclipse.jdt.core.formatter.join_wrapped_lines" value="false"/>.

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

Java スニペットと parentheses_positions_in_method_invocation および join_wrapped_lines の Eclipse formatter 設定を使って、VS Code で問題を再現します。次に、extension の formatter 統合を追跡し、オプションが失われる箇所を特定します。join_wrapped_lines が false の場合に設定された値が尊重され、折り返された行が結合されなければ完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
java, typescript, vscode
領域
developer-experience, tooling
issue の種類
バグ
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
38/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。