Feature Request: Update POL Core to use the new book header packet.

Bug reports and feature requests. New features can only be added to the current development version. Bug-fixes may be back-ported.

Current release: 099 / Current development: 100
Post Reply
Yukiko
Distro Developer
Posts: 2825
Joined: Thu Feb 02, 2006 1:41 pm
Location: San Antonio, Texas
Contact:

Feature Request: Update POL Core to use the new book header packet.

Post by Yukiko »

Currently POL uses the old book header packet:

Code: Select all

 Packet: 0x93
Sent By: Both
Size: 99 Bytes

Packet Build
BYTE[1] 0x93
BYTE[4] Book Serial
BYTE[1] Write Flag (see notes)
BYTE[1] 0x1 (unknown)
BYTE[2] Page Count
BYTE[60] Title
BYTE[30] Author

Subcommand Build
N/A

Notes
Write Flag

    0: Not Writable
    1: Writable

Server version of packet is followed by packet 0x66 for Book Contents.

Client sends a 0x93 message on book close. Update packet for the server to handle changes. Write Flag through Page Count are all 0's on client response
There is a "new" packet available:

Code: Select all

 Packet: 0xD4
Sent By: Both
Size: Variable

Packet Build
BYTE[1] cmd
BYTE[2] length
BYTE[4] book ID
BYTE[1] flag1
BYTE[1] flag2 ( writable ? )
BYTE[2] number of pages
BYTE[2] length of author (0 terminator included)
BYTE[?][2] author (Unicode, 0 terminated)
BYTE[2] length of title (0 terminator included)
BYTE[?][2] title (Unicode, 0 terminated)

Subcommand Build
N/A

Notes
Packet sent from server when opening a book: opening book writeable: flag1+flag2 both 1 (opening readonly book: flag1+flag2 0, unverified though)

client side: flag1+flag2 both 0, number of pages 0.

Example::
Opening books goes like this:
open book ->
server sends 0xd4(title + author)
server sends 0x66 with all book data "beyond" title + author

if title + author changed: client side 0xd4 send

if other book pages changed: 0x66 client side send.
DevGIB
Grandmaster Poster
Posts: 248
Joined: Mon Feb 06, 2006 6:12 am

Re: Feature Request: Update POL Core to use the new book header packet.

Post by DevGIB »

I started to look at this.
But from what i'm reading it seems like the change mostly just implements unicode support.
I wondered if this was a good change to include in the UTF8 branch thats still outstanding as i believe that will simplify the unicode strings and not make the books more complicated than they are now.
Yukiko
Distro Developer
Posts: 2825
Joined: Thu Feb 02, 2006 1:41 pm
Location: San Antonio, Texas
Contact:

Re: Feature Request: Update POL Core to use the new book header packet.

Post by Yukiko »

Yeah. I tested the packet for the new book gump. I had hoped that it would allow longer book titles to be entered in the gump because, as it is, the title area is rather limited in length. The results of my test revealed that there was no change in the text length for titles. In fact, I couldn't see any change in the gump's appearance using the new book gump packet.
Post Reply