Comments (11)
x64 関連の Warning はスルーすると普通に動作不具合に繋がる系が多いと思っています。
たとえばこの Issue とは違う警告ですが warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data
等は明らかにデータが失われる状況ありますよね。こういうのは無視しちゃいけないやつです。
他 Warning についても同様に慎重に精査したほうが良いと思っています。
from sakura.
補足です。
X86版では主要な警告のいくつかをstdafx.hで切り捨てています。同じレベルの対応とすることは可能と思ってます。
x86 版でのキャスト処理等は割と問題起こらなかったりします。
たとえば上に挙げた warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data
については
x86 版だと sizeof(size_t) == sizeof(int) == 4 なので問題なし、
x64 版だと sizeof(size_t) == 8, sizeof(int) == 4 なので問題あり、
という感じです。
from sakura.
#89 を送りました。
from sakura.
一度何人かで話したいです。
from sakura.
一度何人かで話したいです。
論点は何でしょう。
PR へのレビューコメントでは足りない感じですか?
from sakura.
対応すべきか否か
Disable warningも選択の1つです。
from sakura.
それは、誤検出か、コンパイラのヘッダが原因の場合にする対策だと思います。
無効にするにしても、個別に判断することだと思います。
from sakura.
本業があるんで細切れですみません。
X86版では主要な警告のいくつかをstdafx.hで切り捨てています。同じレベルの対応とすることは可能と思ってます。
修正眺めた感じかなりの量ありそうでしたが、これは今やるべきか?ということを言ってます。
今やるかどうかです。
from sakura.
対応には賛成で、何件か見た限り修正は適切でした
いまこのprは特急でx64対応を進める作業の一環だと思っています。
特急で仕上げるにはみるべきことが少し多いように感じています。
リアルタイムでの参加は難しそうなので、一旦はそのまま進めてください。何かあれば後追いで質問するかも知れませんのでその時はよろしくお願いします。
from sakura.
自分としてはx64対応は特急じゃなくて良いんじゃないかなーと思っています。慎重にやらないと事故りそう。
from sakura.
#95 を追加しました。
from sakura.
Related Issues (20)
- 管理者権限非保持ユーザがインストーラーを使用するとエラーが発生する HOT 1
- スペースインデントされたファイルに対してはTabキーでスペースインデントするように自動で切り替えるべき HOT 16
- バックアップ方法を手動も追加してほしい HOT 1
- 開いた時の位置
- issueタイトルは一覧で見た時に内容がわかるタイトルにして欲しいので変更をお願いします。
- デフォルトの文字コードをUTF-8に一括変更する方法の提供 HOT 1
- Vistaスタイルのファイルダイアログでファイルの作成確認が余計に行われる HOT 6
- 丸カッコを含むURLの解釈が変 HOT 5
- GUIやグラフィック描画を考慮したプラグイン機構 HOT 2
- Grep実行結果に対してGrepを実行すると検索対象に存在しない行が検索結果に出力される HOT 5
- 結合文字(濁点・半濁点)のサポート
- ディレクトリ指定をプルダウンではなくディレクトリパス式にしてほしい HOT 3
- 複数行にまたがる文字列置き換えの対応 (正規表現)
- 正規表現での検索文字に\xHHが使えない HOT 2
- Build RequirementsでVisual Studio 2022対応明記 HOT 3
- ドキュメント類の整理やリンク切れの対策などを行う HOT 6
- Visual Studio 2019以前をサポート外とする HOT 9
- タイプ別設定の一括設定
- Azure Piplinesで発生しているMinGWビルドのエラーを対応したい
- Shift-JISのうち、IBM拡張文字由来のローマ数字が正常に表示されない HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from sakura.