ToHeart2 ほか AquaPlus/LeafのGPLゲーをいじるスレ

**名前は開発中のものです。** · 2005/12/13(火) 01:33:48

弊社製品のムービー再生にxvid.orgのムービー展開ライブラリを使用していた件について。

弊社製品、
『アルルゥとあそぼ!!（ランチャとRoutesミニシナリオのみ）』
『Tears To Tiara』
『鎖－クサリ－』
『ToHeart2 XRATED』
の４商品では、動画再生に http://www.xvid.org/ において公開されいるXViD再生エンジンを使用しております。

この動画再生エンジンのプログラムはGPL(GNU General Public License)に基づくソフトウェアとして公開されていますが、これを使用した場合、GPLに定められている通り、オファーがあればソースコードを公開する義務があります。

このGPLのソース公開義務に従い、弊社の上記４作品のソースコードを全て公開する事に致しました。現在ソースコード公開に向けて社内のソースコードのまとめを行っておりますが、２年ほど前のタイトルも含まれておりますので、多少時間がかかります事をご容赦下さい。

今後、XViDの開発元に弊社より謝罪を含め連絡をとり、GPLの基準に合致しているかを確認を行った上で、全ソースコードの公開を致しますので公開まで少々お待ち頂けるようお願い致します。

http://leaf.aquaplus.co.jp/xvid.html

**名前は開発中のものです。** · 2006/01/18(水) 03:39:47

やっと動いた。TH2 readfile.cpp:320
ArcFile.reserve(15);
/*　reserveで↓はまずいのでコメント
for(i=0;i<15;i++){
ArcFile[i].handle = NULL;
ArcFile[i].fileCount = 0;
ArcFile[i].pack_file = NULL;
}
*/
とりあえずこれで音も出るし動くようにもなった。
ほんとはまずいと思うけど・・誰かSTLに詳しい人正しく直してー

**名前は開発中のものです。** · 2006/01/18(水) 03:49:32

>>232　XviDDec.libは\XVidに初めから入ってるのでパスが通ってないだけかと・・

**名前は開発中のものです。** · 2006/01/18(水) 04:19:22

>>233
作者の意図したところはその通りだと思う。
（reserveで領域が確保されると勘違いして、初期化コードを書いてしまったのかな？）

他の部分を見ても、はじめは要素が確保されていないことを前提としてるみたい。
つまりresizeで実際に確保してしまうと動かない。
openFileNum = 0;をopenFileNum = 15;にすれば動くかもしれんけど。

製品版がこのコードのまま動いているとしたら、Releaseビルドのせいで範囲外アクセスがチェックされていないだけの地雷だな。
どうも全体的に不自然なコードが多いなぁ。
STLの使い方にせよ、クラスの責務割り当てにせよ。

C++に慣れていない人が書いたのだろうか。

**名前は開発中のものです。** · 2006/01/18(水) 05:30:00

>>235
ぱっと見ほとんどCの記述なんだけど、
C++知らないプログラマなんだろうか。

**名前は開発中のものです。** · 2006/01/18(水) 06:24:52

Cしか使ったことがない人が、とりあえず1，2冊入門書なりを読んで
「こんな感じかな？」ってノリでC++を使ってる感じがする

**名前は開発中のものです。** · 2006/01/18(水) 07:10:49

>237
まるでオレのようだ。

**名前は開発中のものです。** · 2006/01/18(水) 07:52:57

>>233
# ちょっこれｗｗ
コンストラクタがあるから初期化はコンストラクタに
任せるのがセオリーだと思うんだけど
初期化されて無い要素もあるし地雷原を見ているような感覚だわ

**名前は開発中のものです。** · 2006/01/18(水) 08:46:54

エロゲの恥ずかしいソースｷﾀ━━━━━━（ﾟ∀ﾟ）━━━━━━!!!!
じっくりと弄んでやるぜ。うひひ。

**名前は開発中のものです。** · 2006/01/18(水) 09:10:16

>>235
このソース書き方すっげー俺と似てるんだよね…
Cで書いた方が書きやすいオブジェクト指向苦手人間ぽい。

**名前は開発中のものです。** · 2006/01/18(水) 09:12:03

誰かまとめページ作って

**名前は開発中のものです。** · 2006/01/18(水) 11:26:19

debugboxに所々中々笑える台詞が書いてあるな

**名前は開発中のものです。** · 2006/01/18(水) 11:55:14

"このみ環璃セットと、このみか、環が、単体で別々に出ています"

環璃って誰よｗ

**名前は開発中のものです。** · 2006/01/18(水) 13:19:28

コメント削除して公開って、GPL的にOKなのか？

**名前は開発中のものです。** · 2006/01/18(水) 13:25:28

>>245
OK。
要はコンパイルが通ることが条件になってるだけ。

**名前は開発中のものです。** · 2006/01/18(水) 13:37:56

>>246
それってGPLのどの項に書いてある？

**名前は開発中のものです。** · 2006/01/18(水) 13:52:16

とりあえずTH2専用SOUND PAK展開TOOL v1.0うPしました。
サウンドは同じコードで展開できた。他はやり方違うので見てるとこです

**名前は開発中のものです。** · 2006/01/18(水) 13:55:59

>>247
コメントを消してもいい　って書いてあるわけじゃなく、
GPL中での「完全なソースコード」の定義が、
「プログラムに対応していて」「関連モジュールやインターフェースも含むもの」だから、
別にコメントを消したところでプログラムとの対応関係は崩れないし、
関連モジュールが減るわけでもない。

**名前は開発中のものです。** · 2006/01/18(水) 14:04:04

>>248
どこに？

**名前は開発中のものです。** · 2006/01/18(水) 14:06:20

Cマガが特集してくれんかなーって思ったんだが…休刊なのな

**名前は開発中のものです。** · 2006/01/18(水) 14:10:25

>>250 もち元のソースが置いてあったうpろだ。パス無しソース付GPLでw

**名前は開発中のものです。** · 2006/01/18(水) 14:21:20

DLしたお

でも何かソース足らなくね？
readFile.cppが開けないと言うか同根されてないお（；＾ω＾）

**名前は開発中のものです。** · 2006/01/18(水) 14:23:01

×同根
○同梱

恥ずかしいお（；＾ω＾）

**名前は開発中のものです。** · 2006/01/18(水) 14:25:18

readFile.cppはTH2の奴使っておくれ

**名前は開発中のものです。** · 2006/01/18(水) 14:36:15

>>251
２月号発売したとのメルマガきたけど、休刊なの？

**名前は開発中のものです。** · 2006/01/18(水) 14:39:42

>>256
今ぐぐってみたら・・・

ttp://www.fx-it.com/kihon/top.html

((((((；ﾟДﾟ))))))ｶﾞｸｶﾞｸﾌﾞﾙﾌﾞﾙ

**256** · 2006/01/18(水) 14:44:47

>>257
６年近く毎月買ってたのに……。(´･ω･｀)

**名前は開発中のものです。** · 2006/01/18(水) 14:56:22

>>236
betterCとして使ってるんじゃね？

**名前は開発中のものです。** · 2006/01/18(水) 14:57:11

まあ、最近は完全にウェブの後追いだったからな。雑誌じゃ小回りが効かん。

**名前は開発中のものです。** · 2006/01/18(水) 15:06:15

最新の情報を追うのは遅くなってたけど、まとまった情報源としての価値は大きいと思うんだけどなぁ。
今でも98､9年ごろの特集が役に立つことが多いし。

3月売り分ってことは4月号で休刊か…
刊行継続されることを祈るしかないな。

**名前は開発中のものです。** · 2006/01/18(水) 15:13:27

どこだか分かんないし
見あたんないからまあいいや。

**名前は開発中のものです。** · 2006/01/18(水) 15:16:58

>>262
>>178

**名前は開発中のものです。** · 2006/01/18(水) 15:18:11

>>179だった

**名前は開発中のものです。** · 2006/01/18(水) 16:24:51

＞２４８
GNU 一般公衆利用許諾契約書の複製物がないおｗｗｗ

**名前は開発中のものです。** · 2006/01/18(水) 16:47:08

しまった入れ忘れた、証拠隠滅))彡あと少しで全部解凍出来そうなので
そん時忘れずに入れときます

**名前は開発中のものです。** · 2006/01/18(水) 16:51:50

betterCまでたどり着けてないな。
まぁToHeart2延期の謎は全て解けたが。

・・・ここって、ソースのレビューしてやるスレ？

**名前は開発中のものです。** · 2006/01/18(水) 16:57:43

ここはソースのバグ修正やソースを利用した（当てずっぽうじゃない）ぶっこ抜きツールなどを作るスレ。

**名前は開発中のものです。** · 2006/01/18(水) 17:52:57

238は社員

**名前は開発中のものです。** · 2006/01/18(水) 18:20:28

とりあえず、最初にやることはリファクタリングだなｗｗ

**名前は開発中のものです。** · 2006/01/18(水) 18:30:35

まったくだ、混沌としすぎてるなこのソースは。。。

**名前は開発中のものです。** · 2006/01/18(水) 19:58:21

プロのコードってんで楽しみにしてたんだが……
力技っていうか……なんていうか……

**名前は開発中のものです。** · 2006/01/18(水) 20:15:05

まあ、クローズドのソースなんてどこもこんなもんさ。

**名前は開発中のものです。** · 2006/01/18(水) 21:05:10

誰かリファクタリングやってくんね？

てかどっかまとめ作って分散してリファしたいな。

**名前は開発中のものです。** · 2006/01/18(水) 21:11:36

あれ意外にソース少ないね、ノベルだからかな

**名前は開発中のものです。** · 2006/01/18(水) 21:55:04

予想通りの展開だな。ここでうれしそうにC++どうこうとかいってる奴は、
まともなゲームプログラムなんか作ったことのない奴だろうｗ
このくらいの規模のプログラムならCライクに作るという選択はありうる。
見たまんまで、クラスライブラリよりも読むのがはるかに楽だからな。
>272
派手な打ち回しなどいらないんだよ、静かに勝ってこそプロ。

**名前は開発中のものです。** · 2006/01/18(水) 22:04:22

そんな事より、コミパＰのコード公開してくださいよ。
あの酷い読み込みの嵐

**名前は開発中のものです。** · 2006/01/18(水) 22:10:14

>>276
何処のエロゲーメーカー所属ですか?

**名前は開発中のものです。** · 2006/01/18(水) 22:16:49

SourceForgeまだー？

**名前は開発中のものです。** · 2006/01/18(水) 22:22:32

人のソースにケチつけるようなプログラマって見っとも無いと思う。

**名前は開発中のものです。** · 2006/01/18(水) 22:30:37

俺サイコーっていう最低プログラマーに事実を教えないのは
人として冷たいと思うよ。

**名前は開発中のものです。** · 2006/01/18(水) 22:37:27

げんしけん7巻82ページを思い出した。

**名前は開発中のものです。** · 2006/01/18(水) 22:56:09

いまどきクラス構造もマトモにかけないプログラマは化石級だよ

**名前は開発中のものです。** · 2006/01/18(水) 22:59:54

お前の認識の甘さが化石級だｗ

**名前は開発中のものです。** · 2006/01/18(水) 23:00:22

>見たまんまで、クラスライブラリよりも読むのがはるかに楽だからな。

（　ﾟдﾟ）ﾎﾟｶｰﾝ

**名前は開発中のものです。** · 2006/01/18(水) 23:00:28

クラス構造ってなんでつか？

**名前は開発中のものです。** · 2006/01/18(水) 23:07:34

だから、オマエラの友達っつか知人に一人くらいいるだろ？

コメントアウト付けるプログラマは屑だ！
ソースから全てを読み取れ！
俺なら出来る！

って言う、奇才。

ちなみに漏れはそんな香具師見た事ありません。(´・ω・｀)ｼｮﾎﾞｰﾝ

**名前は開発中のものです。** · 2006/01/18(水) 23:07:46

>>280
せっかくのGPLだから
ケチつけるだけじゃなく直さないとな。

**名前は開発中のものです。** · 2006/01/18(水) 23:08:59

>>284
確かに認識があまかったようなだ社員さんよ(pgr

**名前は開発中のものです。** · 2006/01/18(水) 23:22:25

TH2でゲームプレイヤーに、18禁な名前が付けられないようにするコードを探してみた。

多分この関数で名前を登録している。

Winmain.cpp の、
BOOL CALLBACK NameDialogBoxProc( HWND hwnd, UINT wmes, UINT wparam, LONG lparam )

読む限り、もしかして18禁な名前をチェックしてない？

わくわくしながら探したのに30分無駄になった…

しかし、変数が_(アンダーバー)で始まるのが多いですね。_で始まる変数はC言語ではシステム予約
で使ったらダメだったと記憶しているのだが。(うろ覚え)

**名前は開発中のものです。** · 2006/01/18(水) 23:26:10

>290
そんなマナー規約、実際にエラー（名前のバッティング）が発生しなければ何ともないぜ！
それが職業プログラマクオリティ

**名前は開発中のものです。** · 2006/01/18(水) 23:27:17

チェックする前に卑猥な名前つけられるかどうか製品版で試してみればいいのに

**名前は開発中のものです。** · 2006/01/18(水) 23:27:33

STLをCライクに使ってどこがどう読みやすくなるのかと小一時間（ｒｙ

**名前は開発中のものです。** · 2006/01/18(水) 23:29:44

>>290
それ残骸だよ。
_で始まっている変数も残骸多数。

**名前は開発中のものです。** · 2006/01/18(水) 23:35:10

>>290
俺もちょっとうろ覚えなんだが、確か
_大文字が予約語なのよ
_小文字はOKのはず、GoF本でも使ってたはず

Leafのソースは観てないのでどちらかは知らないけど。

**名前は開発中のものです。** · 2006/01/18(水) 23:36:06

予約語ではない。あくまで紳士協定。

**名前は開発中のものです。** · 2006/01/18(水) 23:47:35

いや予約語だよ。俺も自信ないので結局めくってきたがｗ
uppercaseが大文字の事ね

17.4.3.1.2 - Global names [lib.global.names]

-1- Certain sets of names and function signatures are always reserved to the implementation:
Each name that contains a double underscore ("__") or begins with an underscore followed by an uppercase letter (lex.key) is reserved to the implementation for any use.
Each name that begins with an underscore is reserved to the implementation for use as a name in the global namespace.*
[Footnote: Such names are also reserved in namespace ::std (lib.reserved.names). --- end foonote]

http://www.kuzbass.ru:8086/docs/isocpp/lib-intro.html

**名前は開発中のものです。** · 2006/01/18(水) 23:48:40

これ、アンダーバー２つじゃね？

**名前は開発中のものです。** · 2006/01/18(水) 23:52:29

>>298
or begins with 【an】 underscore followed by an uppercase letter
こっちス

**名前は開発中のものです。** · 2006/01/18(水) 23:54:48

>>299
ほんとだ。すまん。

**名前は開発中のものです。** · 2006/01/18(水) 23:58:06

そのreservedは、
いわゆる予約語（キーワード（if とか int とか））のとは意味が違うんではないか。

**名前は開発中のものです。** · 2006/01/19(木) 00:20:07

JIS X3010(ISO/IEC 9899:1999)
7 ライブラリ
7.1.3予約済識別子

　下線に続き大文字一文字または下線に続きもう一つの下線で始まるすべての識別子は、
　いかなる使用に対しても常に予約済みとする。

　一つの下線で始まるすべての識別子は、通常の名前空間およびタグ名前空間の双方における
　ファイル有効範囲を持つ識別子としての使用に対して、常に予約済みとする。

**名前は開発中のものです。** · 2006/01/19(木) 00:20:56

結局ここには初心者にも満たない屑しかいねぇな。
プログラミングそれ自体を勉強して出直してこい屑ども。

**名前は開発中のものです。** · 2006/01/19(木) 00:28:14

ゲームなんて動けばいいんだよ。

バグがあったところで、別に人が死ぬわけじゃないんだから。

**名前は開発中のものです。** · 2006/01/19(木) 00:32:12

>>302
それ違う。
6.4.1のキーワードの方が予約語。X3014(C++)なら2.11。
識別子として用いられないものを予約語という。

ttp://www.jisc.go.jp/
ここで、X3011とX3014のpdfが見られるので見たい人はどうぞ。

**名前は開発中のものです。** · 2006/01/19(木) 00:35:55

識別子として用いられないものを予約語という

**名前は開発中のものです。** · 2006/01/19(木) 00:44:30

>>303
これくらいのソースで給料貰えるのは正直うらやましい
漏れのとこでこんなソース書いたら白い目で見られるわ文句言われまくりだわで大変だ。。。

**名前は開発中のものです。** · 2006/01/19(木) 00:56:26

折角のGPLなんだからエレガントにしてあげてください

**名前は開発中のものです。** · 2006/01/19(木) 00:57:31

>>303
口だけなら、誰でも出せるんですよ。(pgr

**名前は開発中のものです。** · 2006/01/19(木) 01:05:57

俺も自信ないのでドラフト見てきたら既に書き込まれてるし。

ていうか俺、インクルードガードに３連アンダースコア使ってた。１つや２つが予約されているのなら３つと。
いつどこで仕込まれたのか重い打線けど、規格見る限りこれまずいよな。次から改める。

**名前は開発中のものです。** · 2006/01/19(木) 01:17:05

Netscape Navigatorのソースよりも綺麗で驚いた

**名前は開発中のものです。** · 2006/01/19(木) 03:22:07

スクリプト解析して使えるようにして欲しい

**名前は開発中のものです。** · 2006/01/19(木) 04:43:42

ttp://www.pro.or.jp/~fuji/mybooks/cdiag/

>第２章　これでもプロ
>売りものであるにもかかわらず、超きたない！
>構造的な欠陥の指摘～引数、ポインタの活用

**名前は開発中のものです。** · 2006/01/19(木) 07:15:09

TH2専用 PAK展開TOOL v1.0うpしました。今回は全てのPAKに対応します。
PASS:nekoneko
注意点
1)コンパイルする時は必ず同梱のcomp_lzsとreadfileを使用してください。
2)Xvidは1.1で確認しました。2.2では再生しませんでした。

**名前は開発中のものです。** · 2006/01/19(木) 13:17:57

2.2の方は間違い、別物でしたm(_ _)m

**名前は開発中のものです。** · 2006/01/19(木) 13:24:42

>>311
それは言うな、悪夢を思い出した。

そもそも、Mosaicから続いからてるんだ。一発屋のエロゲと一緒にするな。

**名前は開発中のものです。** · 2006/01/19(木) 16:10:51

>293
そもそもこの程度のことにＳＴＬ使うのが間違い。

**名前は開発中のものです。** · 2006/01/19(木) 16:29:25

>>317
楽したもの勝ち
コの程度のことに、ライブラリ使わなくてどうするの

**名前は開発中のものです。** · 2006/01/19(木) 16:30:56

どうするのって何言ってんの

**名前は開発中のものです。** · 2006/01/19(木) 16:38:33

この処理がライブラリ使って楽になるとも思えんが。
STLを使うには、STLの使い方という、C言語の言語仕様の外の知識が
必要になるからな。
しかし、葉鍵板のほうがまだソースを利用しようとしてる奴が
多いというこの逆説。ここじゃ文句つけてるだけだものな。
あ、「弄る」って、利用するって意味じゃなく、ネタにするってことか？
いわゆる関西芸人的な。

**名前は開発中のものです。** · 2006/01/19(木) 17:04:23

>>320
日本語でおｋ

**名前は開発中のものです。** · 2006/01/19(木) 17:10:04

このソースを利用しろと言われても困ってしまうな。正直、情熱のあるエロゲオタに譲るよ。

**名前は開発中のものです。** · 2006/01/19(木) 17:37:22

実際のところゲームプログラムの参考にするなら吉里吉里の方がよくまとまってるしなぁ…
吸い出しや移植以外に用途が思いつかない

**名前は開発中のものです。** · 2006/01/19(木) 19:13:13

>323
吉里吉里は吉里吉里で、ちょっと大げさすぎるからなー。
どっちも部品単位で参考にする目はあると思うけど。

**名前は開発中のものです。** · 2006/01/19(木) 19:26:40

は？　え？　規模の問題じゃねえよｗ

**名前は開発中のものです。** · 2006/01/19(木) 20:19:55

規模というより、吉里吉里は汎用性をもたせようとしすぎ。
あそこまでやる必要ないだろう。オブジェクト指向とか。

**名前は開発中のものです。** · 2006/01/19(木) 22:27:39

でもスクリプト周りがしっかりしてたらわざわざプログラム変えてあんな妙なことをやらなくてもイイと思うんよ。

**名前は開発中のものです。** · 2006/01/19(木) 23:21:45

話を聞いてると吉里吉里のソースのほうがおもしろそうなのでそっち行くわ。んじゃ。

**名前は開発中のものです。** · 2006/01/19(木) 23:32:31

別の商品を同じもので統一するわけにはいかんのだろう。
汎用プログラム＝商品ごとに使わない無駄が出る＆バグが全商品に影響

あとそういう仕事を知ってる人なら分かると思うけど
もし完全自社内開発でなければ大人の事情もあるわけで。

**名前は開発中のものです。** · 2006/01/19(木) 23:37:37

俺は吉里吉里のゲームは例え法人による販売だとしても同人ゲームだと思ってる。

**名前は開発中のものです。** · 2006/01/19(木) 23:40:32

>327
あれは社内人事絡んでそうじゃね？
メインプログラマが違うんじゃないかと思えた。

TH2のソース見る限り、サポートはこっちのほうが楽だな。
ちょっとだけコンシューマの開発やったことがあるけど、どうせ毎回一作限りで
キツいチェック通さなきゃいけないなら、ストレートにハードコーディングして
一回きりのソースとして考えたほうが楽なんだよな。

**名前は開発中のものです。** · 2006/01/19(木) 23:42:49

たとえば、あらゆるタイトル画面を実装するために、
細かい演出処理をスクリプト仕様に組み入れるぐらいなら、
タイトル画面自体をハードコーディングしたほうが楽。