Reformat long strings to avoid rustfmt bugs. #580
Conversation
Walkthrough今回の変更では、コードの可読性と保守性を向上させることに重点が置かれ、多くのファイルでクエリの整形やインポート順序の見直しが行われました。また、エラーメッセージのフォーマット改善や、マルチライン文字列割り当ての導入なども含まれています。 Changes
Poem
TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (1)
rustfmt.tomlis excluded by!**/*.toml
Files selected for processing (13)
- nusamai-citygml/src/parser.rs (1 hunks)
- nusamai-gltf/examples/make_gltf.rs (1 hunks)
- nusamai-gpkg/src/handler.rs (4 hunks)
- nusamai-kml/src/conversion.rs (1 hunks)
- nusamai-plateau/src/appearance.rs (1 hunks)
- nusamai/src/sink/cesiumtiles/mod.rs (4 hunks)
- nusamai/src/sink/cesiumtiles/slice.rs (1 hunks)
- nusamai/src/sink/czml/mod.rs (1 hunks)
- nusamai/src/sink/geojson/mod.rs (1 hunks)
- nusamai/src/sink/minecraft/region.rs (1 hunks)
- nusamai/src/sink/mvt/mod.rs (2 hunks)
- nusamai/src/sink/mvt/slice.rs (1 hunks)
- nusamai/src/transformer/transform/appearance.rs (1 hunks)
Files skipped from review due to trivial changes (6)
- nusamai-gltf/examples/make_gltf.rs
- nusamai-plateau/src/appearance.rs
- nusamai/src/sink/cesiumtiles/slice.rs
- nusamai/src/sink/czml/mod.rs
- nusamai/src/sink/minecraft/region.rs
- nusamai/src/sink/mvt/mod.rs
Additional comments not posted (12)
nusamai/src/transformer/transform/appearance.rs (1)
5-5:Schema、Entityのインポート順序が変更されました。これにより、コードの読みやすさが向上しています。nusamai/src/sink/mvt/slice.rs (1)
3-3: インポートの順序が変更され、flatgeomのインポートが他の依存関係よりも上に移動されました。これにより、依存関係の視認性が向上しています。nusamai-kml/src/conversion.rs (1)
3-3:flatgeomのインポート順序が変更され、kml::typesの前に移動されました。これにより、依存関係の管理がより一層明確になります。nusamai/src/sink/geojson/mod.rs (1)
276-276:nusamai_citygmlのObjectとGeometryRefのインポートが追加されました。これにより、ジオメトリの参照処理が可能になり、コードの機能性が向上しています。nusamai/src/sink/cesiumtiles/mod.rs (4)
284-288: tile_writing_stage関数内での経度と緯度の処理が改善されました。改行が追加され、コードの可読性が高まっています。
305-307: ログメッセージのフォーマットが改善され、可読性が向上しています。特に、緯度と経度の範囲を示す部分が改行されており、一目で理解しやすくなっています。
343-350: フィーチャーのデシリアライズ処理において、エラーハンドリングが適切に行われています。エラーが発生した場合には、PipelineError::Otherを使用して詳細なエラーメッセージを提供しています。このアプローチは、問題の診断に役立ちます。
405-417: ポリゴンのトリアンギュレーション処理において、正常に法線計算と2Dへのプロジェクションが行われています。また、earcutアルゴリズムを使用してインデックスバッファが適切に処理されており、効率的なメッシュ生成が可能です。nusamai-gpkg/src/handler.rs (3)
145-150: ジオメトリカラムのSQLクエリが複数行にわたって整形されています。これにより、SQLクエリの可読性が向上しており、メンテナンスが容易になっています。
219-220:gpkg_contentsへのテーブル情報の挿入処理において、SQLクエリが整形され、可読性が向上しています。クエリの構造が一目で理解しやすくなっており、エラー発生時のトラブルシューティングが容易になります。
330-338: バウンディングボックス情報の更新処理において、SQLクエリが整形されています。この変更により、クエリの可読性が向上し、将来的なメンテナンスや拡張が容易になります。nusamai-citygml/src/parser.rs (1)
256-259: パニックメッセージのフォーマットが改善されましたが、このメソッドが呼び出される前に適切なエラーハンドリングを行うことを確認してください。エラーハンドリングが不適切な場合、パニックが発生しやすくなります。Verification successful
skip_current_element()関数の呼び出し箇所を確認したところ、すべて適切なエラーハンドリングが行われていることが確認できました。
nusamai/src/source/citygml.rs:st.skip_current_element()?;の後にOk(())が続いています。nusamai-plateau/examples/parse_and_compress.rs:st.skip_current_element()?;の後にOk(())が続いています。nusamai-plateau/tests/common/mod.rs:st.skip_current_element()?;の後にOk(())が続いています。これにより、パニックメッセージのフォーマットが改善されたことと、エラーハンドリングが適切であることが確認できました。
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 説明: `skip_current_element()`関数の呼び出し箇所を検索し、適切なエラーハンドリングが行われているか確認する。 # テスト: 関数使用箇所の検索。期待結果: 適切なエラーハンドリングが確認される。 rg --type rust $'skip_current_element'Length of output: 579
Script:
#!/bin/bash # Extract surrounding lines of the call sites for detailed review # Call site in `nusamai/src/source/citygml.rs` rg --type rust -A 5 -B 5 'st.skip_current_element\(\)\?;' nusamai/src/source/citygml.rs # Call site in `nusamai-plateau/examples/parse_and_compress.rs` rg --type rust -A 5 -B 5 'st.skip_current_element\(\)\?;' nusamai-plateau/examples/parse_and_compress.rs # Call site in `nusamai-plateau/tests/common/mod.rs` rg --type rust -A 5 -B 5 'st.skip_current_element\(\)\?;' nusamai-plateau/tests/common/mod.rsLength of output: 1345
Codecov ReportAttention: Patch coverage is
Additional details and impacted files
📢 Thoughts on this report? Let us know! |
rust-lang/rustfmt#3863