- Maniphest Tasks
- T1125: Eliminate ICU From SIMPLEXpress
- rSb851268c4524: Replace ICU U_Char and UnicodeString with PawLIB onechar and onestring
No tests have yet been written for this project. The tester CLI builds, however.
Successfuly build with "make ready" and "make tester". Tester parses latin basic strings, breaks on extended Unicode as expected "simplexpress-source/src/utf_tools.cpp:138" with a segmentation fault. Valgrind shows memory leaks, haven't started hunting.
This is really just one of those commits that I expect to be pretty fairly broken. You're not so much adding a feature as you are preparing for a refactor; the kitchen looks like a mess because you're demolishing, removing, and replacing some structural elements. Because of that, it's pretty reasonable to suspend P1 on this Differential.
The commented-out code is acceptable here for the same reason. This isn't production code. You're just getting the code into a state where you and @mduminer can work on it.
Go ahead and invert these. I prefer Authors to be in alphabetical order by surname, just to ensure there's no "politics" surrounding who's first.
I suspect StringPiece will still just be a onestring.
These namespaces turned out to be a mistake on my part. This just doesn't look nice...
...and this isn't particularly convenient...
using simplexpress::char_sets char_sets::arr_greek_lower
Let's drop the simplexpress namespace from the whole project, and then just come up with good class names (or specific namespaces) for things that need to be grouped, like char_sets.
Isn't ^[ 0-9 <AEBTXaebtx> ]/ inherently Latin anyway?
...at least it shouldn't be. If it is, something needs to change in onechar.
As it stands, this will only look at the first part of a multibyte Unicode character.
Hrm - @jcmcdonald I'm having trouble landing this, and was unable to clone a new copy. Permission denied. Did anything change with the poermissions for this repo, or is this problem likely local on my end?
TARGET Landing onto "master", the default target under git. REMOTE Using remote "origin", the default remote under git. FETCH Fetching origin/master... firstname.lastname@example.org: Permission denied (publickey,keyboard-interactive). fatal: Could not read from remote repository. Please make sure you have the correct access rights and the repository exists. Usage Exception: Fetch failed! Fix the error and run "arc land" again.
You have push permissions; I double-checked on that just now.
Notice this line...
Permission denied (publickey,keyboard-interactive).
That means your SSH key is not being accepted. That almost alwas means you don't have your SSH key added to your Phabricator profile. Make sure you've created an SSH key on your computer (you can use an existing one if you prefer), and then upload your public key to your Phabricator profile.