toObject method of my message class uses an undeclared variable
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 42/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- javascript
- 領域
- tooling
調査の方向性
minimal.proto から始め、提供されている protoc --js_out コマンドを実行して、M2.toObject の生成結果を再現します。生成されたメソッドを期待される出力と比較し、未宣言の変数 f が出力されなくなっていることを確認します。
索引モデルが issue の本文から書いたものです。
説明
What version of protobuf and what language are you using?
Version: 3.7.0
Language: Javascript
What operating system (Linux, Windows, ...) and version?
Ubuntu 16.04.5 LTS (Bitnami LAMP)
MacOS Mojave 10.14.2
What runtime / compiler are you using (e.g., python version or gcc version)
node js v11.10.1
What did you do?
Steps to reproduce the behavior:
file minimal.proto:
syntax = "proto3";
message M1 {
uint64 x = 1;
}
message M2 {
M1 m = 1;
}
run protoc --js_out="import_style=commonjs,binary:." minimal.proto
What did you expect to see
proto.M2.toObject = function(includeInstance, msg) {
var obj, f = {
m: (f = msg.getM()) && proto.M1.toObject(includeInstance, f)
};
if (includeInstance) {
obj.$jspbMessageInstance = msg;
}
return obj;
};
(or at least that's what it was doing when it worked with version 3.5)
What did you see instead?
proto.M2.toObject = function(includeInstance, msg) {
var obj = {
m: (f = msg.getM()) && proto.M1.toObject(includeInstance, f)
};
if (includeInstance) {
obj.$jspbMessageInstance = msg;
}
return obj;
};
note that the declaration of f is missing in this code
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
Anything else we should know about your project / environment
- 主要言語
- JavaScript
- スター
- 471
- フォーク
- 91
- 平均マージ
- 3時間 57分
- マージ済み PR(30日)
- 2
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
protocolbuffers/protobuf-javascript のほかの issue
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
-
難易度 5/5 1週間以上 初心者へのやさしさ 30/100
protocolbuffers/protobuf-javascript#248 · コメント 1 件 · リアクション 13 件 ·
-
question
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
protocolbuffers/protobuf-javascript#222 · コメント 9 件 ·
-
enhancement port-fix triaged
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
protocolbuffers/protobuf-javascript#185 · コメント 1 件 ·
-
enhancement port-fix triaged
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
protocolbuffers/protobuf-javascript#182 · コメント 3 件 · リアクション 1 件 ·
protocolbuffers/protobuf-javascript の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
mksglu/context-mode#1200 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
neondatabase/website#5944 ·
-
module: core
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
bigbluebutton/bigbluebutton#25849 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
jaegertracing/jaeger-ui#4506 ·