Formatting overly long lines does not respect `--lines` range restriction

オープン

#1,114 2024年6月18日に作成コメント 1 件リアクション 1 件担当者 0 名 GitHub で見る

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

評価

難易度: 3/5
見積もり時間: 1〜2日
初心者へのやさしさ: 35/100

issue の種類: バグ
明瞭さ: 明確に書かれている
活発さ: 停滞
技術スタック: java
領域: tooling

調査の方向性

提供された Test.java の例と google-java-format --lines 1:1 Test.java を使って問題を再現し、その後、--lines の範囲が長すぎる行にどのように適用されるかを追跡します。完了条件は、選択した行がすでに正しくフォーマットされている場合にコマンドが diff を生成せず、要求した範囲外のフォーマットが変更されないことです。

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

説明

Let Test.java be:

class Test {
  void foo() {
    // Columns:
    //   1         2         3         4         5         6         7         8         9
    // ..0123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890
    System.err.println("xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx");
  }
}

What I did: $ google-java-format --lines 1:1 Test.java | diff Test.java -
What I expected to happen: No diff, since line 1 is already correctly formatted.
What actually happens:

6c6,7
<     System.err.println("xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx");
---
>     System.err.println(
>         "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx");

主要言語: Java
スター: 6.2k
フォーク: 936
平均マージ: 6分
マージ済み PR（30日）: 3

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

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

はじめの一歩

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

google/google-java-format のほかの issue

Misleading message for --dry-run with --replace オープン

難易度 2/5 1〜3時間初心者へのやさしさ 72/100

google/google-java-format#1094 · コメント 1 件 · 2024年4月7日に作成
Complementary Teamups オープン

難易度 5/5 1週間以上初心者へのやさしさ 15/100

google/google-java-format#1450 · リアクション 1 件 · 2026年9月6日に作成
IndexOutOfBoundsException - Wrong formatted content when dealing with latex String オープン

難易度 4/5 3〜5日初心者へのやさしさ 55/100

google/google-java-format#1439 · コメント 1 件 · 2026年8月6日に作成
unused import removal leaves extra blank line between package declaration and class Javadoc / declaration オープン

難易度 3/5 1〜2日初心者へのやさしさ 62/100

google/google-java-format#1436 · コメント 2 件 · 2026年8月4日に作成
Possible issue with Eclipse plugin オープン

難易度 3/5 1〜2日初心者へのやさしさ 48/100

google/google-java-format#1428 · コメント 3 件 · 2026年7月30日に作成

google/google-java-format の issue をすべて見る

似ている issue

[Bug][UI] Change the method of opening the Action Detail オープン

難易度 2/5 1〜3時間初心者へのやさしさ 65/100

Netcracker/qubership-integration-platform#913 · 2026年9月23日に作成
SEC-CORS-006 declares MEDIUM but can only emit HIGH or LOW オープン

bug

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

jdubois/boot-ui#1100 · 2026年9月23日に作成
build-scan-xunit: accept ISO offset timestamps in test reports オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

elastic/gradle-plugins#157 · 2026年9月23日に作成
OpenAPI document marks optional request body of `PUT /api/users/me` as required オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

cryptomator/hub#497 · 2026年9月23日に作成
No test covers the claim that only the last command's state is checked オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

johanhaleby/occurrent#1120 · 2026年9月23日に作成

Java の issue をもっと見る