[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Bug-gnubg] Request for merge: image.backgammonbase.com support pat
From: |
保坂範行 |
Subject: |
Re: [Bug-gnubg] Request for merge: image.backgammonbase.com support patch |
Date: |
Fri, 9 Apr 2010 21:56:34 +0900 |
Well, I refined a bit. I hope Oystein like this.
This version uses fixed size buffer for formatting url, uses no fp
Some people might think too many sprintf. :P
If you want more cleaner thing, it is better to organize broader range of code,
I think.
Nori
2010/4/9 Øystein Johansen <address@hidden>:
>> Hi.
>>
>> I wrote a patch to support image.backgammonbase.com url generation.
>>
>> What it does:
>> Let you copy position as an url, using image.backgammonbase.com.
>>
>> Motivation:
>> 2009 World Champion Mochy requested me to ease use of
>> image.backgammonbase.com
>> This site needs gnubgid, but gnubg do not proved it in UI to copy &
>> paste.
>> With this patch, only single step of copy and paste from edit menu,
>> let user to generate a URL for intended position image.
>>
>> Reference:
>> http://image.backgammonbase.com/form
>>
>
> Nice.
> Just out of curiosity: Why does you code use a temporary file? Could it not
> be simpler to use a string instead of fiddling with the temporary file?
> -Øystein
>
image.backgammonbase.com-support-v2.patch
Description: Text Data