bug-gnubg
[Top][All Lists]
Advanced

[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
>

Attachment: image.backgammonbase.com-support-v2.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]