@ Dannii: Yes, it will essentially cheat the system. What you say about dodge/block is exactly what I intend to do. :-) The list is currently randomised after two random entries are added to a base set of movements. We can tweak this for the difficulty, though I can imagine that it would be more fun to leave the patter a standard length and perhaps tweak the actions in it. (More "attack" would almost certainly be more dangerous.)

I'm also contemplating having a particularly hard hit on Automatos (say, 8+ damage) damaging his artificial intelligence and causing one of the eight programmed behaviours to drop out.

@mciul: yes, I will add the hammer later.

About hammers and tension, I'll come back to that.


Spacing out like this doesn't help readability, and only causes trouble when you want to add something with a longer type name.


## test


That's only half an apology, but I suppose it will do.  Just ask in the future.


Using the relative import of `hscMapper` is good, but `transformRegistry` isn't either relative or absolute and is therefore dangerous.


Actually, gmp builds fine, unless it's gmp-4.3.1 which had a broken Makefile. modulesets and modulesets-stable had 4.3.2, modulsets-unstable had 4.3.1 until just now, and it builds fine, with one catch: The config.guess that comes with it thinks that my late 2013 Mac Pro has a 386 processor and so it wouldn't build in 64-bits. That was easily overcome with a jhbuildrc setting, also pushed.

Does libnettle need a newer gmp?


I'm currently looking into how the tool is sending information off to chapel.  At the moment, I'm hoping for inserting an argument to the function decorator named "depends" or something similar, whose contents would be sent to chapel with the -M option.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/10174/testReport/) for this commit**


It doesn't have to be a panic, it can be an error return value, no?


# Under, seu perv.
Perv.


Removing this line guts the file, leaving it with no raison d'etre.  If this persists, the file should be removed completely.


0.5.8 Stable is in there somewhere ...


This needs a docstring.


`angle` should be `elevation`.


I did a `git merge -s ours master` on SFML and then a `git merge SFML` on master. Should be okay now, except for lots of duplicate commits.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1931](http://teamcity.monogame.net/viewLog.html?buildId=4510&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


OK, I think that we can use this log pattern too. At least, it will work on more platforms. Thanks!


Copyleft year should be updated.


The reason I asked is that, I think traditionally, if I compile a file:

chpl foo/bar.chpl

that it's similar to adding a -M foo to the command line.  That is, if bar.chpl 'use's baz and there's a file baz/bar.chpl, it will be found.  It seemed like the similar thing ought to happen here, which is why I was curious about the command line.


@aocole Great, thanks Andrew!


@adam3smith : Twice `acta-neuropathologica`? I guess you want to delete the first occurence.


Should be a space between function arguments.  Similarly below.


That's still a hacky solution. Nowhere in the library we return "manual" errors. All errors are thrown from C side.


I'm a fan of using either setter or constructor injection.  It always seems to be unnecessary to allow for both.  I'll submit a PR to update the docs on the `next` branch if you don't beat me to it.


Done.  Thanks for the pointers.


What about the MIT Suprime-Cam?


This function is declared where it can never be reused, and then used only once.


Yeah, that's a bit kludgy :-) I'm pretty sure @chme can come up with a good solution.


I can see the font attribute in the textpaths in vml, font should remain there for vml.
(search for "font:" in http://www.w3.org/TR/NOTE-VML)


Thanks :+1:  I will fix (or remove this commit, if I can assure that this check is not needed. )


I don't remember exactly why this is here ...


I think you filled the wrong one on this one and on the next one. msgstr shouldn't be empty.


That's awesome! I had no idea there was a tool for that.


As stated in this thread https://stellartalk.org/topic/6913-new-created-account-are-merging-after-claim/?hl=charts. It's likely that cheaters had figured out a way to bypass this fix.


Prefer `Struct` to `dict` when you have standard key names.


*-Cough-*


Done. Thanks for taking a look!


@bdgould Can you take a look at this and see whether this makes sense for your use cases? Your feedback would be useful.


Don't worry about it for now since we gonna replace that part anyway


Hi @bjornerik, sorry for the troubles.  I have just reverted this commit in `master`.  If there is still any problem, please do not hesitate to let me know.

That said, when all the incoming pull requests are merged, may I re-apply those changes?  I find that having the variable definitions in some kind of tabular form makes it a lot easier to read, especially for newbies like me.  ;-)


This should not be setting the plate scale itself.  The plate scale differs between Suprime-Cam and HSC.


Commenting code out using a docstring is naughty.


why did you spend time on reversing the nice `"a".equalsIgnoreCase(tagName)` to the NPE-prone `tagName.equalsIgnoreCase("a")` ? :-)


The functions need docstrings.


For initial reference, this took 7.1 minutes on my MBP .


Aprenda a usar Markdown, baka
---
# Sun suga


For initial reference, this took 29.1 seconds on my MBP .


LGTM


The type in this case is right. The parsing failed. I think ```SyntaxError``` reflects the error better.


No problem


I'm not sure why this method is here. A submitting button should not be enabled if the form is disabled.
Maybe we should deprecate it and remove it.


Hi. Here you go https://www.dropbox.com/s/jlhr988hy9cjjpn/tutorial.pdf


I can never resist the opportunity to use one of my open source projects in another project.

@hltbra has accused me of running the equivalent of an MLM scheme, but for open source...


Not sure we should do the same here.
It shouldn't be possible to submit a disabled form.


ditto


>markdown
>2006


naprawione w PR


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/3301/console
Build was first broken at http://ci.cgeo.org/job/c-geo/3288/


Yes. This is what I meant in the mail discussion.
If it works we should apply it to both 6.x and 7.x.


Interesting! It was also my plan to further develop this idea and have a App and DB server separation. I'm sure we can make it configurable to do either or...

I'm working out how to make the script apply in general. So I'm going to add a check to see whether the folder is called configuration or not. Do you have a good suggestion for that in ansible? I'll quickly add my changes to my repo so you can see what I mean.


develop (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


From my experience, Chapel tends to understand that anything living in the same directory as its source file could be referenced without a -M needed, which was why I tried leaving the other Chapel file in the same directory as the original Chapel sfile and Chapel bfile.  Failure under those circumstances convinced me that pych right now is grabbing the source file and the source file only when trying to compile, without looking at anything else in the directory from where the source file was found.


Are you compiling this? If not, please don't bounce! Thank you.


Suprime-Cam has `Ccd`s with `ptype` of `top` and `bottom`.  Please check that your result works with it too.


I'm not surprised @annevk suggested ```TypeError```. I think ```SecurityError``` would be better than ```InvalidAccessError``` but I do not have a strong opinion.


I'm not entirely certain how that works as yet.


82420068_GridViewMultiEdit (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-86.29%25-red.svg)


To check if the "Watching" role is present in the database and isn't deleted ?


Did not realize it in the push, but this commit addresses issue #32.


When records are deleted, they aren't hard-deleted, right ? so it'll show up in the result, even though it's deleted...


This commit shouldn't contain changes to hscLib.i.


looks good!


The plugin you made doesn't work, I looked it up and your numericranks is outdated. When will you update it?


How about using `WM_SETICON` messages after the window creation instead?


No, the case changes are unnecessary. Say if the driver is "foobar", we're converting it to "Foobar" which is still wrong. It should be "FooBar". PHP is not case sensitive with classes so your changes here are just wasting cpu time changing the case of something from wrong to probably still wrong.


What's the problem with the "element.id + gradient"?


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1931](http://teamcity.monogame.net/viewLog.html?buildId=4510&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 367, ignored: 6 Build time: 00:15:36


Hard deletion depends on deploymwent_settings.
However I think we can reasonably safely assume that the single instance running the SSF template doesn't have a duplicate role which is deleted.


Wouldn't `@data` always be nil on subsequent calls to initialize?


ops, eu deixei passar sem querer. @suuuzi, Obrigado pela correção :+1:.


This will work for now though I realize now it should be

`(mf->version_max > 0 || (mf->version_max == 0 && (mf->version_min > 1 || (mf_version_min == 1 && mf->version_rev >= 2))))`


yeh, i didn't lift this code from my digitalocean manager or anything...


@E3V3A, already compiled. Please see internal Testroom.


I don't like naming the HSC CCDs by their electronics-based name (e.g., 0_13), but would prefer their geometry-based name (integer from 0 to 111) because this is what people are now used to.

I don't see any FITS files for the MIT Suprime-Cam.


I know you didn't :)


82420068_GridViewMultiEdit (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


Hi @thdox, I introduced the change because the [refcard](http://svn.savannah.gnu.org/viewvc/*checkout*/trunk/doc/refcard/txirefcard.pdf?root=texinfo&content-type=text%2Fplain) states "Insert a question mark that ends a sentence."

But I'm also fine with removing this change from this pull-request, since it seems (from our current understanding) to add only little value to the documentation.


@pmuchowski could you remind me why we need to call this at the end of each test?


looks like a dick


Was literally just trying to figure out how to split this out into a separate pull request from the other one I have for you.

But it looks like you've only changed it for the onView bindings.  Making the document bind to keydown is useful as well, so that arrow keys work.


vat?


Yes that seems to be the preferable workaround (also described [here](http://stackoverflow.com/questions/2282349/specialization-of-templateclass-tp-struct-stdless-in-different-namespace)).


Do the camera.py` files contain the same information as in the FITS files?


I am a nice comment


82420068_GridViewMultiEdit (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2169 of 2188 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.13%25-yellow.svg)


Yeah, that seems to break FSMakeFSSpec (since it takes shortcuts).  I'll back it out until that gets fixed up, too.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/10175/testReport/) for this commit**


yup...it should


Ok I'll remove the case change.


Please split into library and executable parts, so parts can be reused simply.


Make this name consistent


Venga, CVs y demas historias


Ok see post it hier soir et au dessus !
Au pire  on voit que le commit sur les ? pris en compte par exemple return results?
Par contre un ? ou pas mis d espace ...
Et ensuite compile OK car reference


Definitely agree. I just have aligned the code (copy/paste) with how it is done in the other XxxLink (just inverting the condition). I will revert and change in the others...


is this "chromePhp" lib respecting debug vs non-debug modes? or are we adding new X-ChromeLogger-Data headers to all our output with this change?


Yes, we could leave it in, but it would be a section of code that could never be hit.


Fail fast


I might be wrong, but seems like this would potentially allow duplicates when checkpointing. If the first half of the stock data for a facility completed before the task failed, this would resubmit the first half, right?


This commit is doing two things (changing behaviour in `__init__` and removing `std_camera`).  The change in `__init__` is not desirable (and it says "This is a hack").


I will never learn to remember about this :)


Yes, make sense...


Don't get the difference between specifying icon before or after window creation.


Done.


¿Qué problema hay con la compasión? Según lo recuerdo, Nietzsche identifica hábilmente el mal con la debilidad. ¿No es natural, por tanto, que premiar la debilidad mediante la compasión sea una actitud nociva?


Fixed #47


This file rocks!


Ok, will try this as soon as possible...


Fixed. But clang 3.4 did not report this.

On 01/19/2015 02:29 AM, Ramiro Polla wrote:
>
> clang compilation fails with:
> src/tools/nltool.c:176:22: error: unused variable 'nc'
> [-Werror,-Wunused-variable]
> netlist_device_t *nc = m_setup->register_dev("nld_log", name);
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mamedev/mame/commit/cfb8782740a9b12b88ae07050a01828bf1b07a0e#commitcomment-9333057>.
>


Is it not better readable to put the assertion strings into constants? Especially if you make a string member acces after them it looks not really nice^^


No, we should not contribute `cordova-internal.conf` to Apache. It contains only internal config. I think we should keep it only on the MSOpenTech fork.


Please check that it compiles with Scali-MPI on INM cluster, it worked before.


A lot of people are parsing XHTML with it, so I prefer to keep it backward compatible


or just `'/'` ?


We don't really have a convention for it yet honestly. I'd prefer if we went camel case when possible, or even UpperCamel like we do for IDs. Thoughts @stevebosworth @mpiotrowicz?


I see how the solution prevents us from running "outputMinfiedCached" and hence avoid flooding cache with different URLs params for the include request; but the client implementation is still bad since every player has to be rendered with PHP and it breaks the iframe cache, since every player request is a distinct URL. The client should be told to not include these random strings on resource includes as well no?


But if you want to parse XHTML properly you have to deal with its entities too. In that case it would be better to have an unknown-entity-callback and a separate XHTMLParser class which utilizes this parser.


гэта не лічбы
`char a[] = "Мама";`


In future, if roles are added as well, then users can add roles, and possibly delete them as well, so I think it's better to be on the safer side,i.e, not assume that it'll never be deleted ?


@jesseolssondirt it depends on pull request https://github.com/mrdoob/three.js/pull/5502


Uploaded new build


і выводзіць праз `%s` лепш


After this commit im getting "HaxeParser.hx:235: lines 235-1394 : A generic class can't have static fields", is this correct?


also related to #111


There should be a corresponding algorithm on the LSST side.


You'll need latest Haxe.


:P


Nice! Nitpick: if bindScrollEvent handles setting up the binding, why not just toss the unbind call inside of it?


Wouldn't be better to redirect this to a drawable with a shape color as default?
My fear is that possibly redirecting colors to drawables could port to issues.


Instead of "overscan:" in the commit message, "isr:" would be more appropriate (but not a biggie).


Thank you. I accidentally deleted previous message, It was about getting this error "HaxeParser.hx:235: lines 235-1394 : A generic class can't have static fields", after this commint.


Start ever yitem with upper case.


Opa, obrigado amigo! :D


Uploaded new version

@kareem289
```php
$identity = urlencode( sha1( strrev($phonenumber), true));
```


Only SuperUsers...probably Admins here, should eb able to delete roles.
I think that making a slowness for a real edge case that an Admin should never let happen is not what I would do.
We'll probably have to agree to disagree.


Fold this into the previous commit, "overscan: rename overscanPolyOrder to overscanOrder".


"using the list module"


@esbenp There are more mail drivers than just swift mail...


Thanks for catching this @mdale - shouldn't be here - removed in next push


introduced color output


cosmetic change.


We should add a section describing the way to go when using t3extblog BE module.


This changes the behavior of curl_easy_recv when there is no data to receive on the socket. Before this change, curl_easy_recv would return CURLE_AGAIN. Now it returns CURLE_OK with 0 bytes received, as if the socket has been closed.


Yeah that was supposed to be in there...i was in a hurry...will update when i get in front of my computer


"Add at least a title and one category."


FWIW I nailed it now, it was indeed JtR-specific. I set sort and skip (from JtR option parser) before they were used... except after this very patch it was too late :-)  I need to look more carefully when merging.


Make this a headline.


Amazing! It works! Thank you!  Triggers seem to work, too!  Are they spotty for you or (the Liam M)


better: is determined by


I tried to run `incidents.php`, seems that after this line requires:

    if (is_array($args[0])) {
        $args = $args[0];
    }
since is used by call stack: `php/ctrl/SmallQ.php?oper=getActiveProviders` > `printXML(stored_query_XML_fields('get_all_active_providers'))` > `do_stored_query`


"click the 'create new' button to add new content elements


Make this a headline


@Neverlord: I yet have to create a minimalistic reproducible CAF example, but I just want to make sure you're aware of this issue.


Thanks! Do you have a suggested fix too?


@GrahamCampbell Yes, true, but this function explicitly uses Swift Mailer? The method is even named sendSwiftMessage - is it not fair to assume it is the driver? :-) I apologize if I am missing something obvious here.


"usage of TYPO3-categories" seems incorrect to me. We use own SQL and TCA for categories.


Categories and tags


This looks like bits and pieces of multiple separate commits have been combined.


Ok, I see now the point of `lastIndexOf`, somehow I got mentally blind to the second argument. Nice trick, I will remember it.


Maybe you can use my page to get fresh data if your predefined values fails? :)
https://coderus.openrepos.net/whitesoft/whatsapp_scratch


@stopiccot The latter way doesn't require modifying the SDL code.


Added docs to https://github.com/ppat/storm-rabbitmq/blob/master/README.md#rabbitmq-as-a-sink


@jgrandelli made changes


What about `return fmt.Sprintf(baseUrl, queryEnc, "&no_redirect=1")` where `baseUrl` becomes `http://api.duckduckgo.com/?q=%s&format=json&pretty=1%s`?


"you can find"
"between two sections"
"all posts are listed"
"'add' icon"
" the number of comments"
"corresponding to this"

This whole section is a little crumbly.


Nice idea to reduce the wait time for short tests while not being punishing to longer tests.


This function is protected so it's return value was discarded by the public function actually calling it.


Mmmm I keep getting same error "HaxeParser.hx:235: lines 235-1394 : A generic class can't have static fields" with latest Haxe git (e8cbe29), maybe this is an Haxeparser issue.


Any particular reason for not reusing `probability`-handling code from `AbstractMutation` as in `UniformPointMutation` ?


I'd remove this else, since you are returning this anyway.


My understanding is that part of Billy's point is about `std::swap(a, b)` and `a.swap(b)` yielding different results, which they would still do when having `std::swap()` call `swapPayload()`.

On the other hand, `operator=()` still behaves odd in that it does not copy all members. Unfortunately, fixing that one breaks existing code which relies on `operator=()` to keep its hands off some of the members.


It was allowed with this commit: https://github.com/HaxeFoundation/haxe/commit/344370c208b1a88533048a10bbbbb4ed6e137515

Chances are the nightly builds didn't build it yet. There are some issues at the moment with Windows at least.


Nice mate!!!


Sorry, yes.


you're right. I thought it was required for pypi, but on second look it's not. deleting.


Can you try and fix this in the `develop` branch? This is the file: https://github.com/actor-framework/actor-framework/blob/master/libcaf_core/caf/scheduler/coordinator.hpp#L44.

Should be enough to replace `size_t nw = std::thread::hardware_concurrency()` with something like `size_t nw = std::max(std::thread::hardware_concurrency(), 4u)`.


deleted + added to .gitignore


hmm, Okay, I'll change it..


Following up the comment above, this would become `return fmt.Sprintf(baseUrl, queryEnc, "")`


I'm an idiot and missed it, I grabbed a random BSD 2-clause not realizing there was already apache and thought it was pypi required, again no longer convinced. deleting.


Тут замыкание на ровном месте придумывается.


This is making a compilation error
/res/values-pl/rr_strings.xml:611: error: Error parsing XML: mismatched tag


Needs a docstring.


И здесь замыкание ненужное появляется.


Btw, seems like you are calling some blocking function at some point that depends on the result of another actor. I'm ok with setting a minimum nr. of threads nonetheless. 2 or 4 seems a reasonable minimum.


Very nice!


Prefer `Struct` to `dict` when using known keys.


Update composer...


Excellent!


Ok, will roll back to previous commit for the moment. Thanks.


Please make use of headline elements for "Insert plugin", "Archive", "Blogsystem", ...


modules -> plugins


Is it only for new compilers? Which gcc version? fwiw I tried to run locally with the CXX settings and my compiler doesn't support it:

``
cc -fno-strict-aliasing -fno-common -dynamic -arch x86_64 -arch i386 -g -Os -pipe -fno-common -fno-strict-aliasing -fwrapv -DENABLE_DTRACE -DMACOSX -DNDEBUG -Wall -Wstrict-prototypes -Wshorten-64-to-32 -DNDEBUG -g -fwrapv -Os -Wall -Wstrict-prototypes -DENABLE_DTRACE -arch x86_64 -arch i386 -pipe -I/System/Library/Frameworks/Python.framework/Versions/2.7/include/python2.7 -c src/annoymodule.cc -o build/temp.macosx-10.9-intel-2.7/src/annoymodule.o clang++-3.5 -fsanitize=undefined -fno-sanitize=float-divide-by-zero,vptr,function -fno-sanitize-recover
clang: error: no such file or directory: 'clang++-3.5'
clang: error: unsupported argument 'vptr' to option 'fno-sanitize='
clang: error: unsupported argument 'function' to option 'fno-sanitize='
clang: error: unsupported argument 'undefined' to option 'fsanitize='
clang: error: unsupported argument 'vptr' to option 'fno-sanitize='
clang: error: unsupported argument 'function' to option 'fno-sanitize='
clang: error: unsupported argument 'undefined' to option 'fsanitize='
error: command 'cc' failed with exit status 1
``


How about to name the patch ${PN}-tests-optional.patch?


"Use this plugin on a separate page!"


updated


Whey do you pass the mdb_txn if it is never used by LMDBCursor?


> Btw, seems like you are calling some blocking function at some point that depends on the result of another actor.

Sounds like the root cause to me.

> I'm ok with setting a minimum nr. of threads nonetheless. 2 or 4 seems a reasonable minimum.

Ack, this probably handles most cases where confused users wonder why the heck their program blocks.


You could make these default arguments in the `__init__` for a class that inherits from `Struct`.

`NAN` for `intensityUnits` doesn't seem reasonable.  Maybe `UNKNOWN`?


Which static template?


yes, readded


2015-01-15
kl 19:11.


Good idea, the clarifications are necessary.


"This plugin manages blog post subscriptions. All email links will point to this page."


Remove "Is commented well."


Although I too prefer camel casing, would you mind updating the new `*Timeval()` functions to use the format described in the style guide:

> lowercase, with words underscore delimited

Thanks :)


Please do it like in konsole, move all upstream changes to the patch. Then you can add it via PATCHES array. Same patch naming suggestion like in konsole comment.


Excellent, I've missed that update, thanks!


"RSS plugin"


Yep, that's OK


This seems not to create a headline. Please check.


This should probably be combined with the previous commit (within this branch) that originally wrote the `specialChipMap`.


Cool - I think we can now do something like...

```js
if (!directive.controller) {
  // There is no controller, there may or may not be a controllerAs property
  throw $compileMinErr('noctrl',
        "Cannot bind to controller without directive '{0}'s controller.",
        directiveName);
} else if (!identifierForController(controller, controllerAs) {
  // There is a controller, but no identifier or controllerAs property
  throw $compileMinErr('noident',
        "Cannot bind to controller without controller alias for directive '{0}'.",
        directiveName);
}
```


I discussed this change with Mike, it's not supposed to be permanent but is a quick fix until we have a better plan for dealing with multiple dbfiles. I left what we had before as a comment.


Would be great if you could add the ability to get the WiFi link speed.


script


Salute! @shirioko


*bug not big -OSX autocorrect is just as silly as iOS. haha


There's no trick, the randomizer replaces existing commands with new ones, so the max limit of about 32 total commands is still in effect. If you're wondering how the new commands work, they use the same basic principle as Shock and Health, which simply load existing spells under a different name. The "R" commands do the same, except they choose a spell randomly from a fixed list, first. The basic code is pretty simple:
```
A9 XX      Load spell ID XX
85 B6      Store spell ID in B6
A9 02      Load "Magic" command ID
85 B5      Store "Magic" command ID in B5
4C 5F 17   Execute the spell using the Magic command
```
The code pointers for battle commands are at `0x219C7`, properties and targeting for battle commands is at `0xFFE00`, and the names are `0x18cea0`.


Och, i know why ;)


Are this issue set to void warranty?


A better commit message would be something like:

    genCameraRepository: import shutil

    shutil is required for the --clobber command-line option.

The summary line tells you where in the package the change was made, and a simple statement of what changed.  Then comes the explanation (why it was changed).


I don't see how this could affect either OStatus or WebFinger....
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.


611 must be deleted and must be this:
611: <!-- Privacy guard notification -->
612  <string name="privacy_guard_notification_title">Powiadomienia</string>
613  <string name="privacy_guard_notifiation_summary">Show a notification if privacy guard is active</string>
614  </resources>


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1932](http://teamcity.monogame.net/viewLog.html?buildId=4512&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


This is not needed in my opinion and should therefor removed.


This guide does not need to cover any RealUrl default configuration. Just remove that whole section.


Very minor.  The line needs a period after the ) and before the ".


Adressed in actor-framework/actor-framework@cd72e6d.


@artwells: I have no clue how this commit went into the PR. This is not meant as an PR, but rather a preliminary thing that I just wanted to have in my fork.
Can you rollback this?


Can we keep rank 1 and rank 2 separate?


Wouldn't this also copy the .so to the config dir as well as the configs etc ?


I don't think we need to make this change in order to fix the bug, and the patch is already big enough as it is. Can you please remove this change? We can always file a separate 2.1 bug later if necessary.


@artwells: I have no clue how this commit went into the PR. This is not meant as an PR *yet*, because it is poorly formatted.

I would like to see it in your package, but it was not ready for Pulling due to formatting. I don't grok how this went into the origin PR.. strange..


IIRC, I think the [[...]] form of the conditional somewhat restricts which versions of Bash are supported.  This could also be done via [ -n "${message}" ].  There are some advantages to quoting the variables when doing quotes, too.


`s/the it/it/`


gofmt


Please add references (links) to section links, so one can jump to that section.


The name of the theme should probably be changed.


Ditto the description.


There is no constant in python. Effectively a variable is a name for an object, in this case we have a name for a string. _all_ strings are immutable in python (thus constant). Maybe upper case letters for the variable would be nice?


I'd love to hear more about those `foor` loops, they sound quite interesting! ;)


Hint: this is a reference one can link to


m-mas era um implemento não ter isso


@LegionXI/owners @LegionXI/editors
I started to actually re-write one of these and then said na, F this, ain't got time to keep doing this. Someone else's mess. **Do them in an unmodified clone DarkStar and push to a new shareback branch, then pull request it. DO NOT commit them directly to the LiveServer branch ! ** That way whatever issues exist will be worked out prior to any merging and darkstar ppl will help you learn.


Sure, I'm fine with letting `MockNavigatorSettings.mTearDown` do it for us. But please add a comment to explain this -- something like "Enable FMD to make sure its event listeners are setup. MockNavigatorSettings will remove the listeners on teardown".


This was a quick 'n' dirty attempt to fix the problem in

https://github.com/artwells/meteor-accounts-guest/issues/12

But I'm not sure at all that this is the right thing to do. Maybe you can help me.


There can not be two identical roles due to unique = True:
https://github.com/flavour/eden/blob/master/modules/s3db/project.py#L4713

That means, there will never be another "Watching" role - *if* there is one, then it's *the* one, hence you can safely ignore "deleted".


It wasn't accidentally, it was done, because nobody is consuming it from glance. By adding it there you make impression that user can change it via glance, which is not true.


@LegionXI/owners @LegionXI/editors
I started to actually re-write one of these and then said na, F this, ain't got time to keep doing this. Someone else's mess. **Do them in an unmodified clone DarkStar and push to a new shareback branch, then pull request it. DO NOT commit them directly to the LiveServer branch !** That way whatever issues exist will be worked out prior to any merging and darkstar ppl will help you learn.


Just "omit", without the "out".

Maybe simply write "semicolon" instead of `;`?


"static TypoScript template"
Remove "(this can be done in your root template or in an extension template corresponding to your blog in the pagetree."


We should advise people to use TS instead of the constant editor. Keep that part anyway.


Don't know if it's in the guide, but maybe mention that infinite loops are useful in servers and in general with goroutines?


Is it necessary to pass along this option using the WebGMEGlobal? Couldn't the keygen have a constructor like function and take an option object as an input argument?


I made a small mistake, the color for battery_history_chart_textcolor should be #ff000000.


Skype id: floatinghotpot


it's the 19th today


No, because the mupen64plus executable is installed in line 18 (./install) and shouldn't be located under $md_inst. But if you build a binary package the executable should be located under $md_inst. I will think about it.


@bors: retry


We should advise users to check all TS configs.
Isnt't it needed to set some more TS configs? Like some page pids?


Yeah. Took the guy working it long enough...


Use code formatting.


Just "Go" (without "the"), instead of "golang"?


Porque agregaste esas lineas adicionales. Por favor corregir y vuélvelo a subir


saw approval from larsbergstrom
at https://github.com/glennw/servo/commit/36e151dc955e111e420efe6451d44185562c29b7


merging glennw/servo/cef-glutin = 36e151dc into auto


glennw/servo/cef-glutin = 36e151dc merged ok, testing candidate = e4f0b816


pull request would be welcome :)


Just say its like in any other TYPo3 extension.


Oh that's right. sorry. i thought i had been overzealous in my cleanup of removing code for the overcloud specific role images. i'll undo this commit.


In fact - you must ignore "deleted", otherwise deleting the role would disable the functionality ;)

Question is though whether you need a particular role here *at all* - or whether it would be sufficient to have an "unqualified" membership record (=without a specific role)?

Wondering that because I think that /any/ membership will imply "watching" - regardless of the role.

So I would, in fact, rather check whether the current user is already a member of the task team (to not unnecessarily add another membership link), and only if not, then add a new membership record - without a specific "default" role (i.e. leave role_id as None).


THis


Aside from all the nitpicks: this is pretty neat. :sparkles:


This seems like a unneeded duplication of what we have in AdministrationManual.


Stunning Job (Y) @shirioko


@Underground78: I guess I will push it now. Do you have any more thoughts?


![qq 20150119095651](https://cloud.githubusercontent.com/assets/10243528/5795063/c63a91c6-9fc1-11e4-8b5d-426a95eb3063.png)This is a bug?


I also check [commit d3de372](https://github.com/mozilla-b2g/gaia/commit/d3de3724170c776daace839031040a918cdefd90#diff-7e78b616bd4bff2333a5ac999fd46f6cR139) on master, so I feel maybe it's better to follow that, how do you think?


remove the file if sample not added


If recv_chunk and send_chunk were too small, this would be expensive. I think we could set these chunk more bigger automatically when using limiting download/upload rate.  How do you like this?

For me, s3cmd runs too fast to eat up all my bandwidth. if one use the rate limit, it maybe implies he cares qos more than performance.


Ah, I wanted those files in the /glsl/ subdir, because there will be more of them. I'd like the build directory to be organized.


I don't know whether this will work or not. This bitmap is only a reference from the function argument, I don't think set this local pointer to null will help GC to collect this resource. But it is worth a try that set the outside bitmap to null.


> Afaik this has some possibilities for collisions. I seem to remember some windows header files defining MIN/MAX. This could be avoided with #ifndef checks.

:+1:

> But have you thought about taking over my static inline header functions, which also don't have the double evaluation problem?

Where are these static line header functions?


That commit completely broke almost the conversation into mkv container


[High] The idea of item 1.2 is to apply a discount depending on the payment type (10% off by Credit Card, the cheapest item is for free by Paypal and 90% of the most expensive item is free by cash). In your implementation it is not clear.
For example, pay method in PaymentPaypal class should apply 90% off by the most expensive item. (A Payment has a Discount associated).


Actually, that does make sense.  If that scroller has to do with a mouse scroll.  I don't use one.


This commit seems to break imports for me; not sure if specific to my setup?
```
MrC C Compiler 4.1.0f1c1
Copyright (C) 1994-1999 by Apple Computer, Inc.

#include <Carbon.h>
                 ^
File "LightsOff.c"; line 3 #Fatal error: unable to open input file 'Carbon.h'
#-----------------------
```


update to English.


I don't know much vimscript, but what's the purpose of adding 'clang' to an empty list as opposed to just doing:
```vim
if neomake#utils#Exists('clang')
    return ['clang']
else
    return ['gcc']
end
```


Same here.


Wrong branch, please use dev branch for active development


Yes, it is. Thanks!


I found it could set 'findmydevice.enabled' to false in `teardown`, but `MockNavigatorSettings.mTearDown` already did the trick to clean it up.


In that case i might start to consider to begin a FF6 randomized playthrough, When that will happen, it will be posted in FF6hacking.com: i hope you'll see the run!

About the hex code, thanks... i implemented it in my new hack(FF6 CotG) and it works properly ;)
I won't forget to credit you for this piece of code.


We shouldn't have a compile time dependency on `sun.misc`. Use `Class.forName` instead.


cool :) .

*Hari K T*

You can ring me : +91 9388 75 8821

http://harikt.com , https://github.com/harikt ,
http://www.linkedin.com/in/harikt , http://www.xing.com/profile/Hari_KT

Skype  : kthari85
Twitter : harikt

On Mon, Jan 19, 2015 at 5:59 AM, Akihito Koriyama <notifications@github.com>
wrote:

> I did. @harikt <https://github.com/harikt> thanks !
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/auraphp/auraphp.github.com/commit/615e64d46ebfc351b3dd82accbfe2c9349fc6f70#commitcomment-9332788>
> .
>


Yes.
I think this problem was completely resolved, but seems still not resolved ...


I don't like this. It requires that users explicitly include meta-gtk-osx-bootstrap in their module-list, and if they do that in .jhbuildrc-custom it has no effect.

The actual problem is that one or more local repositories are marked "dirty". In my case it was libxml2 and libxslt. After I went to each and reset them and rebuilt again jhbuild correctly recognized that they were up to date and skipped them.


Pls update the readme :)


I couldn't get my pure `Class[_]` working with the requirement to instantiate a new subclass of `SignalHandler` down below. If you don't cross over the relevant bytecodes, doesn't the JVM's lazy-loading mean you'll never try (and fail) to load the class?


今回は、テストの練習かと思うので、良いのですが、ActiveRecordのテストを書いている気分になるようなテストは、本番のアプリでは書かないことが多いです。


Or if you want to return both if both are installed:

```vim
let makers = []
if neomake#utils#Exists('clang')
    call add(makers, 'clang')
endif
if neomake#utils#Exists('gcc')
    call add(makers, 'gcc')
endif

return makers
```


Remind me to talk to you about naming you functions on Tuesday. It'll make debugging 100 times easier.


切り分け方もいい感じだと思います。


I wonder if we ought to try to `interrupt` the thread first on the off chance that it is executing code that checks for thread interruption. `Thread#stop` gives me the heebie jeebies...


I like the idea for the flavour, but are you sure coconut oil works in baking this way?


Someone might want to compile this code on a non-Oracle JVM, though.


Yeah I can try doing that.


quickpost_before_validation --> quickpost_before_submit


~~~
$userID = get_current_user_id();
~~~
とし、必ずログインしているユーザのIDを使うようにしましょう。
外部から引数として渡せてしまうと、ユーザのIDがわかれば誰でも編集できてしまいます。


Asi es


It will be a hassle, but you could spin up the subclass of `SignalHandler` with ASM.


This needs to be updated to the download version.  I have spent a couple of hours searching for the solution.  Not funny what this is 3 months old.


What if we compile this code at run-time? We already have a compiler lying around for the REPL. It'd be kind of sketchy, but I found no way of reflectively creating subclasses. I'd just ask `IMain` to create a string and eval it there and then.


It is getting the actual scrollbar component from the view itself, and doesn't have anything to do with your mouse scroll.


This little guy is still commented out; I uncommented it locally to get things working


LOL That was it!? anyway...perhaps we should look at all the other places that deal with these types of params and see if they need to be changed (this might be the only one).


Here's some relevant history:

317a1056cd8062331964d1bc65f1bfd945538551

1fcb86507062f9923465cdf72e6b70c3a1cfe96b


As I mentioned... elsewhere... the "using namespace std;" statement throws the entire STD namespace into the program and can cause conflicts with other libraries. Refer to [this post](http://stackoverflow.com/questions/1452721/why-is-using-namespace-std-considered-bad-practice) for further reading on that.


Rename the file names as well.


dotnet


Hmm seems like this is the case. So am I right that while the locals are visible, the variables declared in scriptlets are not?


Actually, you can use a JDK dynamic proxy. That's what Paul did in his version of this feature before he removed it.


Will fix.


Oops! Fixed.


How about "escaped variable expansion"?


rename kre* -> runtime*


you don't have to use DOTNET_APPBASE anymore (or shouldn't have).


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1926](http://teamcity.monogame.net/viewLog.html?buildId=4499&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


sss


[Low] Be careful when calling to getSequence method. Perhaps you can rename one of them to avoid future errors.


Toggle `false` to `true` to restore sole, single, only page pagination e.g. Page 1 will show back up as the site default.


なるほど。
標準機能の validation のテストは書いてもあまり意味がないということですね。
カスタムの validation を書いたらテストがあった方がいいですよね？


These are necessary.  try this:
```
$ dash -u
$ echo "$unset"
```


I noticed there's a notion of execution wrapper already. It's added to the "template" in which the user code is embedded. I've never tried it.


r+


saw approval from jdm
at https://github.com/Ms2ger/servo/commit/31edd21d6827c8b2d82bec1f23f1433e275f8485


merging Ms2ger/servo/bindings-docs = 31edd21d into auto


Ms2ger/servo/bindings-docs = 31edd21d merged ok, testing candidate = 578203fa


I'm wondering when this happened.


Graphs! :) Fancy.

With me knowing as little as I do about AI, I can only say this looks good :confetti_ball:

The only thing I notice is there are a few Malloc's with no Free's, but for a test it probably doesn't matter at all.


@ioquatix I forgot about these.


Also do you prefix struct initialisers with ".", why is that? Just curious.


This resolves the sed errors on osx for me! Thanks!


Actually, it is what properly sets HACKING_DIR on some shells (zsh for sure).  Unless this is breaking other shells we should leave it in.

(Unless you know of a portable way to do this... All the examples I found were a sequence of tests targeting subsets of shells and a fallback if the shell didn't implement anything above POSIX).


The load function is doing more that just returning an array. Perhaps you should test more cases ?


This commit will fix #53


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1926](http://teamcity.monogame.net/viewLog.html?buildId=4499&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 376, ignored: 6 Build time: 00:15:34


IMO inexistence of `return` is causing crashes leading to failures on travis-ci.

Please either add `return` or stop using goto and move the code that handles the HEAD request into the if statement.


(Couldn't *HELP* myself)


是的


@simi we see exceptions every day for this


是否缩放或者缩放后拖拽的结果，这种情况下 不应该翻页


Same with this as the comment on $0.  Remember, the aim is not to write a script that only runs on the original bourne shell or only on a 100% POSIX compatible shell.  It's to write a script that runs on as many roughly POSIX compatible shells as feasible.  The reason for restricting ourselves to the subset defined by POSIX when possible is so that  the code is portable.  In this case, if the various tests we're doing to set HACKING_DIR do not cause failures when run on other roughly POSIX shells then we need to leave them in so that we can set it more accurately on more shells.


I think 404 should be better here. Are those requests triggered by real users? Also is there any referrer link?


I dont think we should show a 404, if we could just default to the first page.


Oh wow I didn't know that was a thing, I'll try that


nbna mátenme.


This is not async.
```Temasys.WebRTCPlugin.callWhenPluginReady```  is async.
```Temasys.WebRTCPlugin.WaitForPluginReady```needs to be blocking as it is used to return a value (not by callback)


Hat tip to @ignaciouy's screenshot patch


qq


The load function is supposed to return an array of processed queries, why are you returning a promise ?


The ; needs to be on the end of the .h file.

class xyz{
};


what about just:

```ruby
@page = [1, params[:page].to_i].max
# or
@page = [1, params.delete(:page).to_i].max
```


geof_opt (Unknown) - x86_64-MacOS-10.9-clang: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-68.1%25-red.svg)


some tests failed:
failure: http://build.servo.org/builders/mac2/builds/775


Don't forget to wrap your .h file with
#IFNDEF TASKCONTROLLER_H
#DEFINE TASKCONTROLLER_H
<Your class Definition>
#ENDIF


I found a thread describing why the feature was removed. It was partly because it was problematic, partly because of the larger removal of non-core code.

https://groups.google.com/forum/#!topic/scala-internals/PqK6GkZhJK0

We should discuss the proposal to add a signal handler with this history in mind. If we want it back, it might be best to resurrect the old code, rather than rewriting it.


How can this issue be reproduced ?


can you throw a new line in here for good measure :D


Are you planning on creating a .h/.cpp combination?  The .cpp file contains a class definition, but looks like you're planning on using this .h file.


Div?  or Span?  following text reads as span, not as div with display:block by default


Also you know this is a destructor, and not a constructor right?


Sure thing, where?


The commit message doesn't make sense:

> With this change we go from hard to reason about errors like:

>     Error 65537: The GLFW library is not initialized

> To more clean ones:

>     Error 65537: The GLFW library is not initialized

They're the same strings.


These would normally be defined outside the class


in chat.jade  You will notice the red mark in the diff above... it is a warning that there is no newline at the end of the file


Move this to the FAQ


My recommendation would be to abolish the "Watching" role - if a membership record has role_id==None, then the membership is "unspecific" (which is equivalent to "just watching"), and *any* membership implies "watching".


yeh - it should be changed to install to /opt/retropie - most likely just needs an adjustment to the build script to do so to add a configure prefix. We can't sensibly ship a binary for it without that either.


Again: move this to FAQ


Are these driver changes related? Otherwise, they fit better into their own commit, "git add -p" makes this super easy.


I'll merge this for now anyway and we can sort it after. thanks.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1932](http://teamcity.monogame.net/viewLog.html?buildId=4512&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 367, ignored: 6 Build time: 00:15:24


Yes, because this way we can just call the public `cancel` API instead of having to do something special.


Updated the description and linked relevant github pages.


"Example code for changing the pagination values:"


Add `sudo: false` to enable [Docker builds](http://blog.travis-ci.com/2014-12-17-faster-builds-with-container-based-infrastructure/) :wink:


+1 :)


Errr...but yes: no need to check for pre-existing membership ;) of course. This is a create_onaccept, I forgot.


Add a section about needed configs.


You are right, it could have be done in other ways.
I was using the globals as this option is read on both client and server sides.
And I do not want to modify the file which using it.
I will check it again, and if it can be done in a 'nicer' way I will do so.


closed is a bit of a special case since closing auto-releases, so it is not necessary I think. (At least, I couldn't come up with a test case that made it necessary).


:laughing:


Add a "see in TS file" notice


Thank you for the suggestion, Chris! I will research about it a little bit more, but in my understanding, [[...]] is preferred if we don't care about the portability.

http://mywiki.wooledge.org/BashGuide/TestsAndConditionals#Conditional_Blocks_.28if.2C_test_and_.5B.5B.29


develop (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (334 of 334 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Please use `is` not `like` to replicate the exact match it was before. Otherwise this would match at `foo-two-bar` too which it should not.


Is `atof` good at handling malformed inputs?


Please use `is` not `like` for to match the exact string.


I suggest to remove this change, as the current version is more "lean"/simple to read from my point of view.

I also search via google if there is any other reference to what is written in refcard, but could not find.


Add a word about the realurl autoconfig file.


Do we only expect mosaicing to assemble 4 tiles?


Yes, I need to prevent driver creation in the tests. I introduced this method so that I can override driver creation.


what's the purpose of this?


We need a section for this. Perhaps add it as a important notice (there is a formatting for that).

When using EXT:realurl or similar extension you will need to add additional staticTS template "T3Extblog: additional RealUrl config (t3extblog)" (ADD PATH HERE). Add this static TS to the blogsystem plugin page only to preserve cache!


Yes, will you make edit?


(pickiest of notes)  After the {, I'd start a newline.


In this test, that is the expectation. But it can assemble more than 4 chunks.


It's going to return zero, IIRC. Will not crash for sure.


Why not take `t` in milliseconds?


Excellent!


Make these headlines.


This demo is for your grabs :-)


The following call of `ok` will check `$server`. Better use this and exit after the `ok` call if `!$server`.


I'm concerned about this:
> http://www.cplusplus.com/reference/cstdlib/atof/
> On success, the function returns the converted floating point number as a double value.
> If no valid conversion could be performed, the function returns zero (0.0).
> If the converted value would be out of the range of representable values by a double, it causes **undefined behavior**.
> See strtod for a more robust cross-platform alternative when this is a possibility.


Thanks for the fix


r+


OK thanks :)


"template files"
"via TS or constant editor."


Please use `is` not `like` to get the exact match back.


Agree, this should not be in production.

This ```.cc``` file is going out of ```Bricks``` soon anyway.


I think it's okay, because it's already a helper function around the C-api, since it's dealing with go types anyway. Therefore you should be at liberty (as a library writer) to supplement with additional error state.


OK let's not forget about these hacky demos and examples. :)


Remove "Your posts should all listet."
"Add an optional 'limit' parameter for very large blogs. Example: '&limit=100'"


since this change i've had player-tracker select a portal that wasn't the player's most recent one

however: a suggestion. as the recent activity is now shown in a popup, rather than the tooltip used long ago, we could
1. make all portal names in the popup clickable links to select the portal
2. don't select a portal at all when clicking to open the popup


together with the next `ok` this would print `not not ...` on failure which is not the intended behavior. Better just remove this print and use only `ok`.


oh right!


"cases?"


Was this done as a part of running genCameraRepository, or independent?


Roger that.


Again, please use `is` not `like`. I will no longer comment this fact, but please check the rest of the changes too.


having build errors during the compile . somthing wrong with your first pull request . pls check again. thanks


Why is this necessary here do delete __del__?


You know, that this has the same effect as previous version?


No, I don't think so. I will push a small LAV Filters update later tonight since the nightly will be triggered anyway.


I think this commit originated from confusion over the use of `SUPRIME_DATA_DIR`, and should be dropped.


Yes, I believe I mentioned that in the email.  The Docker container replicates it with clang-3.5; gcc was 4.9.

Can you use a Docker container?


Start with upercase


testlib is in nearly all tests needed because it checks for support for fork and skips test if not available. Please add it back everywhere.


Commit log should describe why this change was made.

This commit shouldn't include the whitespace changes.

Since this commit is fixing up a file that was introduced as part of this same branch, we should probably combine the commits.


Sounds like you changed the meaning of these method. A better approach would be to add another method hiding all non_current tabs and calling this. hide usually is called to hide the visible parts of a widget, this code seems to hide the non visible parts/tabs?


Perhaps add some more of the list in this file: https://github.com/fnagel/t3extblog/blob/typo3_62/README.md


This is not a "version constraint" btw. Just to be sure clear for anyone coming across this and getting confused by it.


Probably want to add an `immediate=True` to the `get` (to bypass the butler's proxy objects).


Thanks @pikeas! Was on my way to do this just now! Looking forward to an npm release so I stop seeing this in my console:

> npm WARN package.json jshint-loader@0.8.0 No repository field.


Need a space after a comma in function arguments.


better use `pass` instead of `ok(1,..` which matches also the `fail` you used above.


r+


If you're updating commit by commit, it would be easiest to nuke the three folders inside /ext/bootstrap/ and then replace them with full copies from a new download of this repo.


It should select the same portal as before. I noticed this behavior, too, but I believe this happens when the last action was creating a link, so player tracker has two LatLngs for the most recent timestamp.

+1 for not selecting a portal. Will take a look at this.


And hes only half done.....wtf..???


Added def for Sony Nex 7


This should be folded into the previous.


Please terminate the last line with a newline.


Use `functools.wraps` here.


To get an idea what some code does, I recommend to take a look at the git commit msg. Often it is helpful. Afaik there is a simple delay added. It was necessary to keep the players synchronized in MP games, because the widget wasn't inited quickly enough. Now it will be broken again.


Same comments as before.


Use `functools.wraps` here


##Remove


Didn't get the meaning of the comment. Any chance to rephrase or maybe there are typos?


Current ffmpeg is leaking single frame on flush when decoding h264. It was fixed here https://github.com/FFmpeg/FFmpeg/commit/e0e33ebe6bd71fdb211c9af7c7af17d6472c535b Maybe it would be ok to include the fix, I don't know how often people are seeking though and how if it would cause troubles.


Oh, I know where it's defined now, that's not the problem (usr/include/AssertMacros.h, in the SDK). What's more, according to the comments in that header Apple has realized they made a mistake and they have versions with a capital and prefixed with __ for some time now, like __Check for this one. And there's even a way to disable the old macros:
```
#define __ASSERT_MACROS_DEFINE_VERSIONS_WITHOUT_UNDERSCORES 0
```
I'm not sure what the best solution would be. We could set the above as an extra define in the CMake files on Mac OS X. Or we could define it somewhere in our sources. Or, as you suggest, we could simply #undef the macros.

That said, I'm still interested to see the includes leading to the issue. Thinking about it, the best way to get to the bottom of this is getting the preprocessed output. I assume @cala is using cmake + make to build. In that case, he can run something like `make -C src/game Level2.i SpellAuras.i` in his build folder. Note that Level2.cpp was causing the problem, SpellAuras.cpp was not, even though it also used check as variable. Then, after running that command, make the two resulting .cpp.i files - in the src/game/CMakeFiles/game.dir subfolder of your build folder - available somewhere. I suggest a zip file, because I expect the preprocessed output to be large.


I had thought of the same thing, but how do you enforce uniqueness, especially if the data is spread across multiple resources?

I had thought of making a type that described how params (whether they're secrets, 'normal', or something else) are addressed and presented, and applying this to different sources.  The type might look something like:

    type ParameterAdapter struct {
        Name   string
        Value  string // would allow adapting values of parameters or composing them from other params.
        EnvVar *EnvVarPresentation
        File   *FilePresentation
    }

The pod spec would incorporate these like so:

    type PodSpec struct {
        // other fields omitted
        Parameters []ParameterPresentation
        Secrets    []ParameterPresentation
    }

The kubelet would use these to present the parameters / secrets to the pod.  Going to think through how this might apply to service info.


When do you plan to re implement the deprecated method?


You should remove this temporary fix, and do things the standard way (empty defect files?).

In general, you should avoid `print` in favour of using a `Log` (e.g., `self.log.warn("…")`).


Please factor out `testpath` into a constant.


Oh I got this at last... after reading `// The only legit constructor is HTTPServerConnection(Connection&&).`


all tests pass:
success: http://build.servo.org/builders/linux1/builds/571
success: http://build.servo.org/builders/linux2/builds/570
success: http://build.servo.org/builders/android/builds/607
success: http://build.servo.org/builders/mac1/builds/788
success: http://build.servo.org/builders/mac2/builds/788


Ah, I see. Caught it this morning didn't put that much thought into it.


This needs a test too, so we don't do it again.


Ran `npm test` and `npm run-script cover` with no errors. Other than the formatting and small refactor comments, LGTM.


Not sure how much we want to say about general upgrading. Depending on your system its a complicated tasks.
So why say something about DAM but not about TemplaVoila? Not sure if we want this here. Perhaps splitting in upgrade with TYPO3 and just switching would be a good start.


I see the variable name has changed to `text` but I don't see it updated in the HTML. Think of the `variables` property as `root`, with whatever you put under it building objects or arrays. With that in mind, what you have here defines a top-level variable `text`, which you would call using `{{text}}`. If you would like it to be be `{{p.text}}` like you have in the HTML, please define `variables->p->text`.  That being said, I'd recommend namespacing it `{{paragraph.text}}` to be consistent with our other elements.


Should cherry-pick d63e53fa4fc63e5db6a909be77d000711296fee8 from HSC instead of using this.


"You will need to adjust your templates and TS configuration, but"


I am not sure that we don't need recurlyjs-errors and recurlyapi-errors


Should cherry-pick 61824969547ec61997ec4639c06f6adc3333c721 from HSC instead of this.


fast-forwarding master to auto = e4f0b816


Can we localize the exception messages as per TRUNK-4500?


This was Solar's format, and it was totally busted. Salt dupe remove logic was totally busted.  It would have been better to only strcpy in the actual 'salt', but I simply wipe out the binary part later in the function.  Works the same.


what bugs are fixed with this commit ?


Think I fixed this one too…

> On Jan 19, 2015, at 3:27 PM, Paul Price <notifications@github.com> wrote:
>
> Should cherry-pick 61824969547ec61997ec4639c06f6adc3333c721 from HSC instead of this.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/LSST/obs_subaru/commit/d578b6afc68eec1180bbb3b034898668c696a386#commitcomment-9345655>.
>


"Use a separate page for this plugin!"

This could be formatted as a notice!


Done (for all of the #:fail-when).


I think we have to accept that funky colored backgrounds won't work to well on the wiki. It just looks much cleaner without the bg.


Done


Why in master branch?


I think you can call `plan skip_all` only in subtests or before you did another plan. Otherwise you get `You tried to plan twice...`.


I would use `VALID_ESCAPE_CODES` here as you use it in the functional test, but OK.


Oh, switching results in a LOT more stuff.  Great!

P.


True although the method signature could also get quite long, e.g.:

```` java
    ResponseEntity<ResponseBodyEmitter<MyPojo<MyParameter>>>

Also I wonder if it wouldn't be quite common to send objects of a different type. For example the SseEmitter sub-class "sends" a series of objects of type Object (actual data serialized as JSON serialization for example) and String (SSE formatting around the data).


saw approval from larsbergstrom
at https://github.com/servo/servo/commit/eaa600a8c6e199abca6e9454934a1b2312448573


merging servo/servo/glennw-patch-1 = eaa600a8 into auto


servo/servo/glennw-patch-1 = eaa600a8 merged ok, testing candidate = 0a787ddb


:+1: Good! I Forgot to change those


:+1:

Yes, it could have been commented cleaner, but I don't think it's that necessary for this inner-level code.


Mike, only the malleus doesn't benefit from tension. (Granted, that used to be the only hammer.) This is because Nomos rewards methodical attacks, not emotional states. Well. It's actually so we could give the player a hammer with a nice attack bonus, something she can use when Nomos commands her to attack even if concentration and tension are low; without making it overpowered.


Will match http://imgsee.me/login.html


I think ast is a jmespath submodule. So instead of `import ast` you should try `from jmespath import ast`.


Also, PEP8 says we should avoid importing multiple stuff in the same line.


Hi Jakub,

I cannot confirm that on my side I'm missing the last bundle. This seems a bit different on my system than on yours one.
This is almost the same patch as in 1.1.2 but you have additional \n in line 87:

    +        printf "%s\n" "${output}" | tr '\t' "${SEP}" | ${COLUMN} -xt -c 5 \

Please have a look at my cqosgi patch in master

    git diff HEAD~1 cqosgi

    diff --git a/cqosgi b/cqosgi
    index 6f968b3..625ca62 100755
    --- a/cqosgi
    +++ b/cqosgi
    @@ -65,7 +65,7 @@ symbolicName\":\"([^\"]+)\",\"category\":\"([^\"]*)\".*"
         items=$(printf "%s" "${filtered_items}" \
             | ${SEDX} "s#${REGEXP}#\1${TB}\6${TB}\4${TB}\5${TB}\2#"\
             | sort -nk1 -t "${TB}")
    -    total=$(printf "%s" "${items}" | wc -l | tr -d ' ')
    +    total=$(printf "%s\n" "${items}" | wc -l | tr -d ' ')
         if [ "${machine_friendly}" -eq 0 ]
         then
             printf "%s bundles\n\n" "${total}"
    @@ -76,7 +76,7 @@ symbolicName\":\"([^\"]+)\",\"category\":\"([^\"]*)\".*"
         fi
         if [ "${machine_friendly}" -eq 1 ]
         then
    -        printf "%s" "${output}"
    +        printf "%s\n" "${output}"
         else
             COLUMN=$(which column)
             if [ ${?} -ne 0 -o -z "${COLUMN}" ]

For my system it seems that I don't have problems in any mode for both dash and bash shells.
Please have a look at my instance bundle line state in OSGI:

> Bundle information: 324 bundles in total, 316 bundles active, 8 active fragments, 0 bundles resolved, 0 bundles installed.

My tests (`-j` option is JSON raw format obtained directly from CQ) for master branch:

    $ cqosgi -j -u admin | grep -o 'Fragment' | grep -c 'Fragment'
    8
    $ cqosgi -j -u admin | grep -o 'Active' | grep -c 'Active'
    316

Machine format:

   $ cqosgi -m -u admin | grep -o 'Active' | grep -c 'Active'
    316
   $ cqosgi -m -u admin | grep -o 'Fragment' | grep -c 'Fragment'
   8

Pretty printed format:

    $ cqosgi -u admin | grep -o 'Fragment' | grep -c 'Fragment'
    8
    $ cqosgi -u admin | grep -o 'Active' | grep -c 'Active'
    316

I even have correct line ending for pretty printed format. The last line is correctly ended with `\n`.

Could you please confirm if you have similar tests on 1.1.2.
Default shell:
    $ readlink -f /bin/sh
    /usr/bin/bash

My bash version:
    $ bash -version
    GNU bash, wersja 4.3.33(1)-release (x86_64-unknown-linux-gnu)

I'm not against merging this patch but I would like to figure out potentially dangerous differences between our systems.

Thanks in advance,
Arek


Such a minor difference, but since you're looking for the _shortest_ thing, I might write this as

```ruby
if item.length < shortest_so_far.length
```

just because it reads a bit more smoothly, as in "is this item shorter than the shortest", not "is the shortest longer than this item". Again, this is super minor and nitpicky.


Yes I realized about `lastIndexOf` with your last commit. Nice trick, I like it.


This will not work ... instead we should either introduce `Slicer_VTKSYS_SYSTEMTOOLS_USE_STRING` or patch VTK/KWsys to expose a similar macro and/or have more elaborated version information.


TeamCity Ylab :: Feodorov/ylab [Build 55](http://192.168.30.225:8111/viewLog.html?buildId=566&buildTypeId=Ylab_FeodorovYlab&guest=1) is now running


These tests definitely add more robustness to the code.


Could you please assign the indent to some variable instead of doing this all inline?
Example:

```js
var indent = mode === 'bitbucket.org'  ?  '    '  :  '  ';
// ... then use indent further below
```


@alanclayton this code example is incorrect. It should be:

`
var url = "https://store-dot-rvaserver2.appspot.com/v1/company/" + companyId + "/product/status?pc=" + productCode;

$.ajax({
  dataType: "json",
  url: url,
  success: function(data, textStatus) {
    if (data && data[0]) {
      // API was successful
      // Product status is available via data[0].status
    else {
      // API failed, verify the request
    }
  },
  error: function() {
    // Server error
  }
};
`


Alright, I'll change that for consistency with HttpMessageConverter javadocs. BTW I tried to find an alternative meaning of POJO, just for fun but it seems to have virtually no competition :)

http://www.acronymfinder.com/Plain-Old-Java-Object-(POJO).html


Nice work! Implementations are straightforward and readable.

My only question `word_count` is: should it handle `/` or other non-whitespace word separators? If so, you might want to add some more tests (and, of course, write code to make them pass).


This undermines the point of the intent of the original code.


Yes. But looking at the SFML documentation it seems like this is a more idiomatic way of doing this. It also seems that the draw methods for our classes derived from sf::Drawable should be private.

http://www.sfml-dev.org/documentation/2.0/classsf_1_1Drawable.php


@kbenne ... develop is now showing the bad commits again unless I'm looking at some cached version...


Rounded rectangles are no longer used in OS X (as of 10.8? 10.9? can't recall).


I can not see where the additional test comes from. Could you point me to the relevant place?


Good variable naming. Can't see any way to improve this.


well, i made a stupid mistake :) like i said, might be a giant fail :) I'll fix it tomorrow ;)


this commit broke DTLS - RTP/AVP (browser - to sip trunk) call
SDP towards browser does not have DTLS in it.


82420068_GridViewMultiEdit (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2172 of 2188 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.27%25-yellow.svg)


userId https://github.com/CollectionFS/Meteor-CollectionFS/issues/449


I actually like `array[0]` as it was before more than `array[ 0 ]`. Could you please change this and the following cases back?


A separator is still required here for the chunked multi-JSON response ("JSON-per-line").  Likely a LF `'\n'` but maybe CRLF, just need a comment explicitly stating it has nothing to do with the chunked encoding specs (that's our own way to separate multiple JSONs).

It's not necessary only after the single-JSON response in `SendHTTPResponse` above.


Should merge this into previous commit within this same branch.


Does not this kind of kill the whole point of `checkTempDir`?


Fixed here: #1847


Again, solid code. And great suite of tests - I'm glad to see you exercising the edge conditions.

Please please fix your indentation settings though! 2 spaces per indent, not tabs.


Probably want to add `immediate=True`.


@shirioko  Please , the id with the code not give me the same result
$identity = urlencode( sha1( strrev($phonenumber), true));

![untitled](https://cloud.githubusercontent.com/assets/6575879/5807473/57d96106-a02c-11e4-8e56-269c023d51c5.png)


Need space between function arguments.


Hmm, here is how I understood your comment:
1. For a single JSON, let's do no newline (I personally would prefer to keep it, but see no good reason beyond my personal preference)
2. For chunked format with multiple JSON-s, leave "one JSON per line" as something the user of the chunked output format should handle.

Did you mean something different?


Good question! And one without a direct answer. You will have to decide how to handle this case, and whichever way you do, it would probably be good to document it somewhere (perhaps as a comment) so that whoever uses this code won't be surprised by the behavior.


Sorry…

> On Jan 19, 2015, at 3:29 PM, Paul Price <notifications@github.com> wrote:
>
> Oh, switching results in a LOT more stuff. Great!
>
> P.
> —
> Reply to this email directly or view it on GitHub <https://github.com/LSST/obs_subaru/commit/d578b6afc68eec1180bbb3b034898668c696a386#commitcomment-9345692>.
>


Fold this into the previous commit.


`result` is used outside the `try`, maybe it should be declared outside the `try` too


Ah, yes, I get it now. And I like it.

2015-01-19 13:54 GMT+01:00 Mark Pizzolato <notifications@github.com>:

> Hi Jordi,
>
> But I'd suggest to add another layer of abstraction to make things even
> easier: provide a standard communications protocol for "detached" panel
> implementations.
>
> The API, as described, in fact already does exactly what you are
> proposing. How the communication is achieved is done internally within the
> API. The initial 'constructor' creates a local process on the same machine
> and provides a communication path to control and display information in
> that process. How the communication is achieved is not a concern of the
> application using the API.
>
> The idea is to build upon the API to make a piece capable of communicating
> with external processes (in the same machine where the simulator is running
> or thru the network) implemented in whatever platform is available (ie,
> java, or .NET, or python). It should be necessary to establish a generic
> "PANEL block" format, which would be sent to the "panel server" using UDP
> (or TCP, but I'd suggest UDP to minimize the impact in the simulator
> itself). In the same way, a "response block" should also be defined,
> allowing the panel implementation to send commands to the simulator.
>
> Front panel applications needn't be concerned about this detail. These are
> 'under the covers' issues that the API is intended to address.
>
> It would certainly be reasonable (once working panel applications exist)
> to provide an alternative 'constructor' which could use alternative
> channels or protocols to transport information, and potentially to run on a
> different system from the simulator.
>
> That basic implementation would also be a "reference implementation" for
> the API, allowing the SIMH developers to test the PANEL API implementation
> in each simulatior without needing to have a real, working PANEL.
>
> I think you are missing part of the point here. The goal is that none of
> the simulator specific code will have any particular knowledge of whether
> the simulator is running under the control of a front-panel application or
> merely a more traditional simh initialization command file (or a user
> invoking it from the command line). No special code will exist within the
> PDP11, PDP10, NOVA (or any other) simulator code to support panel
> operation. I believe that all simulators which exist today already have
> defined simh register variables which contain the bits of information which
> any panel for that simulator would need or want to both display and/or
> change. Am I missing something?
>
> The plan is to implement the API I've described (extended as needed for
> missing pieces) so folks who want to implement panel applications can focus
> on what is necessary to provide the user facing experience. The person
> implementing the panel will certainly need to know what registers in the
> system being simulated are important for the particular panel they are
> building, That knowledge and the panel API should be sufficient.
>
>    - Mark
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/simh/simh/commit/b198e13c09b784e37dcdfa334d3213dc4f09b850#commitcomment-9338732>
> .
>

--
Jordi Guillaumes i Pons
Barcelona - Catalunya - Europa


Hmm indeed, maybe @nevcairiel plans to cherry-pick that soon?


Same deal as 5bbde05 - good tests, good question, please change indentation settings.


That shouldn't matter—`try` doesn't introduce a new scope and that's how it's done in willie too.


poniendo comentarios por doquier


We should probably go with the "standard" way of dealing with this (e.g., blank FITS files for CCDs with no defects?), and dump this "temporary fix".


TeamCity Ylab :: Feodorov/ylab [Build 55](http://192.168.30.225:8111/viewLog.html?buildId=566&buildTypeId=Ylab_FeodorovYlab&guest=1) outcome was **FAILURE**
Summary: Tests failed: 7 (7 new), passed: 114 Build time: 00:10:33
### Failed tests
```
ru.biocad.run.tecan.TecanMappingGeneratorTest.TecanMappingGenerator should work in degenerate cases: class java.util.NoSuchElementException: key not found: ()
java.util.NoSuchElementException: key not found: ()
      at scala.collection.MapLike$class.default(MapLike.scala:228)
      at scala.collection.AbstractMap.default(Map.scala:59)

ru.biocad.run.tecan.TecanMappingGeneratorTest.TecanMappingGenerator should throw exceptions on invalid argument values: class org.scalatest.exceptions.TestFailedException: Expected exception java.lang.IllegalArgumentException to be thrown, but java.util.NoSuchElementException was thrown.
org.scalatest.exceptions.TestFailedException: Expected exception java.lang.IllegalArgumentException to be thrown, but java.util.NoSuchElementException was thrown.
      at org.scalatest.Assertions$class.newAssertionFailedException(Assertions.scala:496)
      at org.scalatest.Assertions$.newAssertionFailedException(Assertions.scala:1538)

ru.biocad.run.tecan.TecanMappingGeneratorTest.TecanMappingGenerator should work in basic tests: class java.util.NoSuchElementException: key not found: ()
java.util.NoSuchElementException: key not found: ()
      at scala.collection.MapLike$class.default(MapLike.scala:228)
      at scala.collection.AbstractMap.default(Map.scala:59)

ru.biocad.run.tecan.TecanMappingGeneratorTest.TecanMappingGenerator should work in larger tests: class java.util.NoSuchElementException: key not found: ()
java.util.NoSuchElementException: key not found: ()
      at scala.collection.MapLike$class.default(MapLike.scala:228)
      at scala.collection.AbstractMap.default(Map.scala:59)

ru.biocad.run.tecan.TecanScriptGeneratorTest.TecanScriptGenerator should insert wash instructions appropriately if configured so: class org.scalatest.exceptions.TestFailedException: "A;[dst;;;1;;1;;;1
D;src;;;1;;1;;;1
W
A;dst;;;3;;1;;;1
D;src;;;3;;1;;;1
W
A;dst;;;2;;1;;;2
D;src];;;2;;1;;;2
W
" was not equal to "A;[src;;;1;;1;;;1
D;dst;;;1;;1;;;1
W
A;src;;;3;;1;;;1
D;dst;;;3;;1;;;1
W
A;src;;;2;;1;;;2
D;dst];;;2;;1;;;2
W
"
org.scalatest.exceptions.TestFailedException: "A;[dst;;;1;;1;;;1
D;src;;;1;;1;;;1
W

ru.biocad.run.tecan.TecanScriptGeneratorTest.TecanScriptGenerator should transfer volumes above and below tip capacity: class java.util.NoSuchElementException: key not found: TecanCoordinates(src,1)
java.util.NoSuchElementException: key not found: TecanCoordinates(src,1)
      at scala.collection.MapLike$class.default(MapLike.scala:228)
      at scala.collection.AbstractMap.default(Map.scala:59)

ru.biocad.run.tecan.TecanScriptGeneratorTest.TecanScriptGenerator should skip tips having 0 or negative volumes: class java.util.NoSuchElementException: key not found: TecanCoordinates(src,1)
java.util.NoSuchElementException: key not found: TecanCoordinates(src,1)
      at scala.collection.MapLike$class.default(MapLike.scala:228)
      at scala.collection.AbstractMap.default(Map.scala:59)

```


@Bitti09 It is a test. At this moment I don´t know if works well.


Many thanks for you work. But there are some issues which prevent the direct commit for now (see comments).

Regards,
Steffen


Should cherry-pick d63e53fa4fc63e5db6a909be77d000711296fee8 from HSC instead of this.


Inline code blocks strangly use two pseudo elements to simulate padding. Apparently in Firefox, one cannot start a selection when dragging over these pseudos. I 'fixed' it for now by using real padding.


Combine with previous commit.


Wouldn't this break all other systems that don't have fseeko64 and related? I'd suggest to use mingw-w64 instead of mingw32 that has fseeko and the current version shoud work fine with (also for 32-bit code).


Mehmet beni parse hesabindan collaborators olarak mi eklemen gerekiyor
benimde push yapabilmem icin ? Eger oyleyse pro hesaba ihtiyac var sanirim.
30 gunluk trial veriyorlar fakat 30 gun sonra yine sadece 1 kisi atabiliyor.

19 Ocak 2015 10:18 tarihinde Arif Gencosmanoglu <a.gencosmanoglu@gmail.com>
yazdı:

> Aynen geldi :D Ariiff yaziyor
>
> 19 Ocak 2015 10:13 tarihinde mrgenco <notifications@github.com> yazdı:
>
> Apk cihazinda yukluyse bildirimi almissindir :)
>> 19 Oca 2015 17:07 tarihinde "0014" <notifications@github.com> yazdı:
>>
>> > Aynen local olarak goremezsin ama GIT hesabin uzerinden gorebilirsin
>> > sanirim. Bu git isi guzelmis ya sevdim :D menejer gelene kadar biraz da
>> > parse`a bakayim hehe.
>> >
>> > 19 Ocak 2015 09:28 tarihinde mrgenco <notifications@github.com> yazdı:
>> >
>> > > Henuz pull yapmadan yaptigin degisiklikleri goremem abi :) Su an
>> > > bankadayim
>> > > aksama bakarim :)
>> > > 19 Oca 2015 16:22 tarihinde "0014" <notifications@github.com> yazdı:
>> > >
>> > > > Libbraryler senin klasorun altindaydi o nedenle hata aliyordum.
>> > Sanirim
>> > > > suan, projenin libs klasoru altinda [image: :smile:] Bir sikinti
>> > > > olusturdu mu sende ?
>> > > >
>> > > > —
>> > > > Reply to this email directly or view it on GitHub
>> > > > <
>> > >
>> >
>> https://github.com/mrgenco/strategy-game/commit/b773c6fd7804b4b00e8acd5577221520a23fbf23#commitcomment-9339900>
>>
>> >
>> > >
>> > > > .
>> > > >
>> > >
>> > > —
>> > > Reply to this email directly or view it on GitHub
>> > > <
>> >
>> https://github.com/mrgenco/strategy-game/commit/b773c6fd7804b4b00e8acd5577221520a23fbf23#commitcomment-9340002>
>>
>> >
>> > > .
>> > >
>> >
>> >
>> >
>> > --
>> >
>> > Arif Gencosmanoglu
>> >
>> > Software Developer
>> >
>> > International FoodSource.
>> >
>> > Mobile: 201-745-0004
>> >
>> > arif.gencosmanoglu@intlfoodsource.com
>> >
>> > —
>> > Reply to this email directly or view it on GitHub
>> > <
>> https://github.com/mrgenco/strategy-game/commit/b773c6fd7804b4b00e8acd5577221520a23fbf23#commitcomment-9340666>
>>
>> > .
>> >
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/mrgenco/strategy-game/commit/b773c6fd7804b4b00e8acd5577221520a23fbf23#commitcomment-9340746>
>> .
>>
>
>
>
> --
>
> Arif Gencosmanoglu
>
> Software Developer
>
> International FoodSource.
>
> Mobile: 201-745-0004
>
> arif.gencosmanoglu@intlfoodsource.com
>

--

Arif Gencosmanoglu

Software Developer

International FoodSource.

Mobile: 201-745-0004

arif.gencosmanoglu@intlfoodsource.com


Optional advice: At line 54 of ddl.sql, there is a discussion of a hypothetical case with 4k inserts per second and 4 partitions.
Sure, it's only hypothetical, but I think that it might be slightly more useful/clear to hypothesize the actual default rate and partition settings that we use and let the user extrapolate from there.
Otherwise, good to go.


Don't think this will work in LSST land, as I'm not sure "classification.extendedness" exists there.


1. Correct. If you prefer to keep it, let's keep it. Should not break anything.
2. I think of the chunked transfer-encoding as a transport-level packets that enable data streaming in HTTP (though I could stream data from node.js without such encoding, we decided to go the RFC way). You said before that a proxy could split/join our prepared chunks into other chunks at will, so a separator **inside** the data stream is still required to split the app-level data into packets of JSON.


vendor/xiaomi/cancro/cancro-vendor-blobs.mk:446: *** recipe commences before first target.  Stop.


Actually I subsequently found the 'official' answers and they had the same basic shape as ours


Does the "NO_DATA" mask plane exist on the LSST side?


maybe int64_t and < 0?


return array() for backwards-compatibility


Did these changes find their way into the new geometry files?


Awesome, was just looking into this issue.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/10176/testReport/) for this commit**


I hope this is no longer necessary on the LSST side, following the cameraGeom change.


1. Great, will keep re-add the newline after a single-JSON response (just ```'\n'``` would be sufficient, I guess). Let's do it after the code is merged, which is happening as we speak :-)

2. For the chunks: On the pedantic side, technically, it is the user's job to introduce separators. They might want something else than the ```'\n'\\\. On the practical note, quite frankly, I would lean on the side of re-inserting it here as well.

Will do so if there are no objections from your side.

@deathbaba: Will do so in a separate PR to keep things simple. Thanks!


landmark class created by Adam, he requested for me to upload it for him. All coding for this class was done by Adam.


This is a callback, not a decorator, and `handle_future` isn't returned. Not sure what `wraps` would do here.


I'd prefer something like this:
```
var config = config || {};  // make config param optional; ensure it's an {}.
var deleteLocations = (config.deleteLocations === true);


@smarterclayton, any thoughts about the above?


Unhandled Fundamental Failure


You could return GLFW_INVALID_VALUE error, or a more specific glfw3 error.

Or, just undo this commit and pass a byte slice directly to `C.glfwCreateCursor`, that way the user will work on getting an `*image.RGBA` or whatnot and pass in its underlying byte data.


Document that all non-RGBA `image.Image` are converted to RGBA


The naughty person who wrote this combined two features in one commit.  We don't want the rotation following the cameraGeom changes, but it seems that changing where the defects are interpolated is desirable.


* Across all files: can you please don't use abbreviates for variable names. config => configuration

* ('ctor') Could you please rename the following variables to reflect the 'Features' naming convention from the SDK this.ARModeGeo = (1<<0); this.ARModeIR  = (1<<1) and also update the documentation for that variables. You can refer to the iOS documentation for further reference;

* (loadARchitectWorld)
    - Could you please change the key 'AugmentedRealityMode' to something similar that the SDK uses, like 'Features', to be consistent across source files.
    - Could you please change the mode parameter name to something similar that the SDK uses, like 'Features', to be consistent across source files.
    - Could you please update the function documentation to reflect the function signature changes
    - Could you please move the optional json object documentation from the exec call documentation to the function documentation.

* (isDeviceSupported) Could you please update the function documentation to reflect the function signature changes. Please also refactor the variable name arMode (See a point above).

Otherwise nice work ;)


again, allow 3rd param #config to be optional.
```
config = config || {};
var interval = config.interval || 5000;
```


@Bitti09 Thanks for the info! :D In a moment i will try to compile and try to fix errors.


Published to http://vui-docs-internal.s3-website-us-east-1.amazonaws.com/16_bbf11b48fd2efbc1cf0630a0d6060985e5d2baa9


all tests pass:
success: http://build.servo.org/builders/linux1/builds/572
success: http://build.servo.org/builders/linux2/builds/571
success: http://build.servo.org/builders/android/builds/608
success: http://build.servo.org/builders/mac1/builds/789
success: http://build.servo.org/builders/mac2/builds/789


This is to support a feature that doesn't exist on the LSST side, but it doesn't hurt.


Let me try. I've never touched Docker so it might take a while (this is good reason to learn it though).

If I can replicate the compiler error I'm pretty sure there's some easy fix. I suspect just casting the array to a naked pointer would do the trick.


Has this been used to adjust the new cameraGeom files?


Relevant: https://bugzilla.mozilla.org/show_bug.cgi?id=1123450


The settings table has only two rows, which are both varchars as they are used for other data as well. `1` and `0` would be the best values for `$medium`. Not sure if I did the same thing with other vars. Let me know if you experience the same problem elsewhere. Will fix it for `$medium`.


done


Why?


fast-forwarding master to auto = 0a787ddb


There are more build errors.  Add me in hangouts: bitti06(at)gmail.com


1. Thanks. Please note my edit about `Content-Length`, the newline is a part of the body.
2. The separator could be passed from the outside as a parameter (maybe defaulting to `'\n'`, the current frontend implementation assumes `'\n'` as the separator). But be aware that accepting a `"`, `{`, `]` or the like as a separator would break data parsing or make it very complex (and you specifically patched `cereal` to force it to a single line to avoid the conflict with `'\n'` separator).


This should be combined with the previous commit that operates here.


Did you mean to invert this IsEmpty check?


@joemcgill I'm curious as to why it's better to use metadata?


Could you do the same for Studio?


Where are `lockfile` and `errorFile` variables defined? It looks like you're just creating global vars here.


Oh, right. Didn't catch that. Thanks.


Duplicate - removed here: cfa7300ae2ed158bfd3ae400dfe4b611cd6ddd90


I don't know – I thought that you would know =)

The purpose of `checkTempDir` is to verify whether directories created by `mkdir` are writable. If `checkTempDir` returns `FALSE` then using `mkdir` is not safe. And you added a new `mkdir` call which does not care whether using `mkdir` is safe or not.


Fixed here: cfa7300ae2ed158bfd3ae400dfe4b611cd6ddd90


yes, understanding is good.
Adding the safety to cmake part for mac os is probably also best choice here..


@roliveri This looks good!  It looks like there is also an extraneous (and incorrect) d_time method in the same file (as well as a reference to the field in the dump method) you may want to remove as long as you're fixing issues with time-related methods in the xfs code.


download the image to a folder like: `/images/background.jpg`


@Bitti09 Done.


pinging @goya

@rocco what is your example scenario and plugin for this? The intent for this change is that a plugin cannot alter or read anything outside of the project. This change is for shared server environments where you don't want the plugin to access (r or w) system files (like /etc/passwd or any config files) for example

@sgrebnov let's patch that


Probably want to fix this comment


I've asked @terminalmage to also take a look here, since this is his code IIRC. However, I'm afraid I don't follow the point that you're making here and I fail to see how your change modifies the return in any way. We'll still return a tuple in the three conditionals you modified. Could you please elaborate on why you think it's better to split the assignments up as you have so I can understand where you're coming from? Thanks.


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-53.7%25-red.svg)


Apparently `link_to` doesn't work for `destroy` requests if js is disabled (it turns into a `GET` request), but `button_to does:
http://stackoverflow.com/questions/4606860/rails-3-link-to-to-destroy-not-working


Is there a timeline for this  TODO? Did you mean to leave this here?


please use a better name for this field


نبنثقلتخقحلتطسسسسسسسسسسسسسس


@rodrigopavezi I don't think you need this anymore?


Whoops, I'm on an early commit. My bad.


Yes some sort of thing, but just the string line wraps do really look unmaintainable^^
To keep the indentation you can also use the + operator to concatenate them in the "constant".
For example I would suggest something like this:

```python
ASSERT_STRING1 = "\n\nproj/white\n" +
                 "| 1| 1| test line\n| 2| 2| line 2\n" +
                 "|  |  | [{}] SpaceConsistencyBear:\n" +
                 "|  |  | Trailing whitespace found"
```
Or wrap them to the next line:
```python
ASSERT_STRING1 = \
    "\n\nproj/white\n" +
    "| 1| 1| test line\n| 2| 2| line 2\n" +
    "|  |  | [{}] SpaceConsistencyBear:\n" +
    "|  |  | Trailing whitespace found"
```
It's possible I missed `\` escapes since I'm not a python pro (not yet :D).


We do have a convention, all variables ( objects, properties ) use underscore, they always have.

Sent from my iPad

> On Jan 19, 2015, at 3:16 PM, Carson Shold <notifications@github.com> wrote:
>
> We don't really have a convention for it yet honestly. I'd prefer if we went camel case when possible, or even UpperCamel like we do for IDs. Thoughts @stevebosworth @mpiotrowicz?
>
> —
> Reply to this email directly or view it on GitHub.
>


PLS REBUILD/pr-scala@fb043761806f8ef2c7a662f15ba5ac5731426dbf


Last thing, please make this the designated initializer


The image height and width attributes were set from the original image, so if the original was a thumbnail and I changed it to a medium size (for example), the target ratio would be 1 (i.e. the ratio of the thumbnail) instead of the ratio of the new image. Using the metadata height and width attributes works as expected.


There are a lot of nested divs here. Is there a way where you could just target the native semantic elements?


You may wanna throw an assert in here to stop fools from sending nil


I am not overly confident about that change, can anybody review it ? @Stapelzeiger @nuft ?


Is this div necessary?


Updated this on master, doesn't necessitate a new release.


wann braucht man den nicht?


all links given by spyder are down..
so i give you my own, until spyder update the site
here the link, then add all updates on core/SQL/updates

https://dl.dropboxusercontent.com/u/63485661/ArkCore%20NG/ArkDB_r7.7.0_NG_CATA_4.3.4_19-10-2014.zip


updated to v0.0.7, replaced keypress with keydown for document event listeners


I see an over use of divs in this document. Try to use semantic elements as stand-alone as possible. If you're using divs for positioning, that's likely unecessary.


@krisselden this line is the source of problems when trying to use a vendored version of ember pulled in from an addon


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Hrm, what if the presentation thing contained an `ObjectReference` to the source?


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-0.0%25-red.svg)


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-0.0%25-red.svg)


It works with custom directory `$this->tempDir . '/cache'`, so it should create it.


Three double quotes are a multiline string so theres no need for escapes. This makes the expected output way more readable than having it in concatenated single line strings because it looks just as it should.


Unhandled Fundamental Failure


What reference errors are being addressed here? I'm not seeing any reference to process/Buffer anywhere in the vm's code...


typo: `autoderefs`


Bitte in "bezogener" ändern


Fixes #1


I've added the file to the fileserver now (thanks @gpn39f) ;)

http://files.arkania.net/index.php?direction=0&order=nom&directory=ArkDB-NG&


It starts with a number, and that's [not valid](http://www.w3.org/TR/xml/#dt-name).


guh


I've send you a mail with my changes ;) Please read through the changed files and push them to this feature branch.


This way of adding extra functionality seems pretty hacky. Both cases are just picking a file...why not unify them?


I adapted this tests from my forked version on ```exp4j```, which doesn't generate duplicated entries on validation. That's why ```testValidation1()``` will fail.


oh, nm, I see it's the closure variable.


So it will not cause any trouble on hosting where `checkTempDir` returns `FALSE` and `%tempDir%/cache` does not exist? Because I'm afraid (not sure, just afraid) that user will have to manually delete `%tempDir%/cache` and create it once again with different owner/group which is very unintuitive.


Was there not some scheme changes (.res files in resources folder)?


But then you begin the next line in column zero.
I think this is kind of weird since it breaks somehow the code alignment:
```python
 self.assertEqual("""\n\nproj/white
| 1| 1| test line\n| 2| 2| line 2
|  |  | [{}] SpaceConsistencyBear:
|  |  | Trailing whitespace found
""".format(RESULT_SEVERITY.__str__(RESULT_SEVERITY.NORMAL)), self.get_str_from_queue(q))
```
But that's only a suggestion, if you want to keep it, I'll never mind :O


one arg per line


and here


Could you add a comment here please


D'ou venaient les conflits ?


prefer `let resolved_t = match ...`


The string is easier to change this way which in turn _increases_ maintainability ;) We use this style on several places in coala and I think its better because you can change it faster. Especially tests are changed often (probably more often than functional code, remembering that every functional change usually comes with at least one test change if not more for dependent objects.)


It seems working. Buying multiple Armor/weap. are slow, but it happens once so it matters not.


Looks like I also fixed upper and lower weaponskill thresholds. You can now weaponskill at Lower <= Target HP Value >= Upper.

For example
Target HP = 50
Lower = 25
Upper = 75

You can weaponskill as long as the mob's hp is between 25 and 75 inclusive.


I attempted to add a unit test for this scenario but was unable to
replicate it inside a unit test. It seemed to pop up when using odata,
specifically ODataQueryOptions<T>. Just didn't have the time to go any
further with it. I'm ok with however you would like to use the existing
pull request. I was thinking you just use my pull request as a prototype
and incorporate it however you see fit.

Please let me know if/when you add it to a nuget package so I can abandon
my custom nuget package.

Thanks
Steven

On Sat, Jan 17, 2015 at 2:40 AM, Kasper B. Graversen <
notifications@github.com> wrote:

> First, thanks for the pull request! I cannot accept it as it is now, but
> with a little tuning we will get it in! :)
>
>    1.
>
>    its great you found a bug, but please add a test that shows the
>    existing way is errorneous. This will help us in the future when we
>    refactor the code into something unsafe again was unable to replicate the
>    test. It only showed up in production for some reason. I would have added
>    it.
>
>
>    1.
>
>    There already exists a "HarvestHelper" which does recursive lookup for
>    fields, I think we better place your method harvesting there. That way all
>    harvestimplementations can enjoy it.
>    2.
>
>    I had the same problems with line endings and have committed changes.
>    I guess you did the same - I was unsure how to take part of your pull
>    request in
>    3.
>
>    you use a different indenting. We should use the same. 4 is probably
>    better than 2 - but we need to coordinate this and apply it to all files.
>    in a separate commit.
>    4.
>
>    its a nice idea with a fallback.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sjdirect/StatePrinter/commit/fadd4e062a7dc99801c4496cd2b80d93b3008617#commitcomment-9325646>
> .
>


Should this be DoubleCalibrationActivity?