Summary: | [GTK] Simplify Vector handling in WorkQueue::registerSocketEventHandler() | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Zan Dobersek <zan> | ||||
Component: | New Bugs | Assignee: | Zan Dobersek <zan> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | cgarcia | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | 130081 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Zan Dobersek
2014-04-29 02:21:05 PDT
Created attachment 230364 [details]
Patch
Comment on attachment 230364 [details] Patch This code should die once bug #130081 is fixed. It's not really possible to register a socket event handler for more than one descriptor. (In reply to comment #2) > (From update of attachment 230364 [details]) > This code should die once bug #130081 is fixed. It's not really possible to register a socket event handler for more than one descriptor. I'll mark this bug as dependent on bug #130081 so I get notified when this patch might become obsolete. Comment on attachment 230364 [details]
Patch
Removing the review flag until then.
(In reply to comment #3) > (In reply to comment #2) > > (From update of attachment 230364 [details] [details]) > > This code should die once bug #130081 is fixed. It's not really possible to register a socket event handler for more than one descriptor. > > I'll mark this bug as dependent on bug #130081 so I get notified when this patch might become obsolete. Bug #130081 landed, making this patch invalid/unnecessary. Closing. |