| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
| |
The original goal was to comply the Uninit warning, but this is now a
futile effort as Uninit is no longer planned to be the default in Nim.
Setting `res' to its type's default value is therefore just a waste of
cycles when it's already zero-initialized.
|
|
|
|
| |
It has caught some minor bugs.
|
|
|
|
| |
Fixes acid3 tests 31, 32
|
|
|
|
| |
Fixes acid3 test 30
|
|
|
|
| |
Allows branching on the scheme without string comparisons.
|
|
|
|
| |
I wonder if it would be better to just resume in the buffer.
|
| |
|
| |
|
|
|
|
| |
Clearing the entire dependedBy array in applyStyleDependencies is wrong.
|
|
|
|
|
| |
* ignore username/password if undefined
* do not throw on subsequent fetch with username/password
|
| |
|
| |
|
|
|
|
|
|
|
|
|
| |
This time, I've also ported over the consistency check to prevent some
ownership bugs.
Unfortunately, the check is very limited, and it is still possible to
double-free or leak JSValues. I think it would be possible to make
coverage 100%, but only with ARC...
|
|
|
|
|
|
|
| |
* optimize toJS set
* change defineProperty wrappers to return an enum
If we're going to wrap defineProperty, then let's do it properly.
|
|
|
|
|
|
|
| |
It's simpler and more efficient to handle this in the DOM.
(The interface was also confusing/broken in that it only really accepted
one htmldda class.)
|
| |
|
| |
|
|
|
|
|
| |
This adds a runtime check to packet readers to ensure that all fds have
been read, and switches to seqs for packet writers.
|
|
|
|
|
| |
Conflating buffered streams with non-buffered streams is generally a
bad idea.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Now we just pass down the value of n and check errno, plus
readDataLoop/writeDataLoop returns a bool indicating whether it failed.
For now this seems to work OK, but maybe I'll add a better abstraction
in the future.
EOFError is still used for handling failed packets; this is brittle,
and should be replaced once we have a proper buffering mechanism for
them. (That will also let us kill BufStream.)
Unrelated: this also fixes a bug in buffer with cacheId.
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
Instead of hashing the class name for isInstanceOf, we now just reuse
the Nim type pointer -> JSClassID map, which should be more efficient.
This removes getClass and hasClass; these can be replaced by just
reusing the class ID returned from registerType.
|
|
|
|
| |
ref. https://todo.sr.ht/~bptato/chawan/43
|
| |
|
|
|
|
| |
Event handler functions can be set twice now.
|
|
|
|
|
| |
A map isn't so bad with three pointers, but it won't be viable once we
start adding more pseudo-elements.
|
|
|
|
|
| |
Base InlineBox is a fair bit smaller now, and (most) strings are
cached using RefString.
|
|
|
|
|
| |
This isn't great, but neither was passing around a pointer that pointed
to a single object.
|
| |
|
|
|
|
|
|
|
|
| |
* do not trigger change event if selection did not change
* do not destroy old selection on option insertion steps
* position select popup correctly
* reflectors for disabled attribute
* immediately redraw container when select disappears
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
* fix iterator assertion failing if finalizer del's after incl
* fix strange mismatch in id
* fix missing HTMLOptionsCollection finalizer
* optimize refreshCollection
* fix root liveCollections not being updated by the finalizer
Now it shouldn't crash pages anymore.
|
|
|
|
| |
Not only was it wrong until now, it was also non-deterministic...
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
ok now I understand why I made shareCachedItem crash...
|
|
|
|
| |
createTextNode is redundant...
|
|
|
|
| |
probably breaks with TLA
|
|
|
|
|
| |
All uses of these could be delegated to other processes with more
privileges.
|