> A new mod that fixes the mod? Would have to look up the individual licenses, but that might be an option.

Licensing doesn't work like that. Whether you distribute an update as a patch (i.e. an esp file depending on the original) or in full doesn't matter. It is still a derivative work.

I am not sure why you are looking at this from a cost / benefit point of view. I didn't ask you to implement it yourself, I just asked what you would do if someone happened to present us a finished implementation.
The benefit is obvious, better compatibility and removing cruft from our script compiler.
Regarding cost, is it really that difficult to do? The laborious part, I assume, would be getting the various opcodes used for each script instruction. But we we should be able to automate that part, even without reverse engineering the executable. Just create an ESM file with a specially crafted testing script for each script instruction. We already have a list of these in the OpenMW source code (with argument types, etc), so the ESM file can be created automatically. Then compile that file in the CS and look at the compiled code.
Anyway, looking at implementation cost makes no sense if the implementation is created by someone who otherwise wouldn't be contributing to OpenMW.


note that for this particular driver you always know it is not external if it is packaged as part of a MPU9250, so you could (via a #ifdef) force _external[0] to 0.


You are removing an empty line here...


I actually have some doubts about that. In cases like that I prefer to define db column traditionally, user_id (consistent with the DB naming conventions) and only use different name (author, owner, etc) in relation name in Rails model class.

What do you think about that?


:confetti_ball: Thanks for all the hard work!


I'd prefer including the file unconditionally here and wrapping all code in #ifdef __ANDROID__


Definitely add javadocs for this method. Particularly, the thinking around priorities: what higher and lower would mean when returned by this call.


I think the description should be generic (unless that's really the only arg that's supported ;) )

Maybe something like: `'--gradleArg=<gradle command line arg>': Extra args to pass to the gradle command. Use one flag per arg. Ex. '--gradleArg=-PcdvVersionCode=1'`


Yes, unfortunately it is the case that they have disabled the exact timing
stuff in the EyeX API - possibly because they realize they are eating into
their own market? Or possibly they just haven't added it in yet.

For now, I am using option 3 and setting the delay to 0.

I'm back at the lab now and am looking at calibration.

On Sat, Dec 20, 2014 at 11:26 AM, Sol Simpson <notifications@github.com>
wrote:

> Regarding the timing stuff, looking at this code, I think what is being
> calculated is the /offset/ between the tobii sample time stamps and the
> iohub time base, not a measure of the sample /delay/. To be able to
> calculate the delay of a sample, we need a way to know what the tobii
> clocks time is when the sample is received by iohub:
>
> # delay: the amount of time between an eye tracker sample time stamp # and the time when that sample is received by iohub#
> delay = current_tobii_software_time - sample_timestamp_time
> # where:# current_tobii_software_time: is the current time from the tobii software time base#    that is used by the tobii SW to set sample time stamps   # sample_timestamp_time: the time reported in the tobii sample being processed
>
> Do you agree, or am I reading the code incorrectly?
>
> If you agree, then the following maybe considered:
>
>    1.
>
>    The iohub - tobii time base offset being calculated includes the eye
>    sample delay. It can not be used to calculate a sample delay, but can be
>    used as a way to convert a sample time stamp into an iohub time. The
>    calculated iohub sample time will be influenced by the sample delay of the
>    system though. So accuracy of the time base conversion is kind of unknown.
>    2.
>
>    Given 1), It may be more accurate to calculate the timebase offset
>    between iohub and tobii by collected the offset values based on N samples,
>    and then use the smallest offset collected as the offset to use. This way,
>    the time bases are being offset using a value that was least affected by
>    the sample to sample delay that can not be corrected for.
>    3.
>
>    Set delay to 0 for eyex samples, since there is no way to calculate it
>    using the new tobii gaze C api.
>
> If the above is an accurate assessment of the new tobiigaze api, it is
> very unfortunate.\, as it means there is no way to accurately calculate
> sample delay and the time base conversion between iohub and tobii times
> will be influenced by the delay, which compounds the issue.
>
> Even if tobii removed the time base conversion code from the api, which
> actually tried to factor in code call time / network transmission duration
> into the time base conversion, at minimum the new api should simply have a
> function ( say getTobiiTime ) that can be called and would return the time
> of the tobii software when the function call was made. This way people
> could write their own timebase conversion code and delay calculations that
> could at least be as accurate and precise as the accuracy and variability
> of the getTobiiTime function.
>
> Sorry, starting to ramble here..
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dgfitch/psychopy/commit/30d8f1f732fe0902c13222e3b9dc690a0de4d75b#commitcomment-9053635>
> .
>


Doesn't change anything, still get nothing on liveposts


:+1: thanks!


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-warnings-yellow.svg)


ah, many thanks


maybe name this rewriteActionsFor() ?


Just testing the commit and push with netbeans :)


Instead of introducing the new variable FASTD_LDFLAGS, you can just add the PIE flags using `set_property(TARGET fastd APPEND PROPERTY LINK_FLAGS "...")` (the `APPEND` keyword will just add the flags to the once previously set without `APPEND`)


Good. Thanks


Shouldn't rootdev be initialized to NULL prior to passing it to arm64_getdev? There are fail conditions inside in arm64_parsedev which would return without touching rootdev, so it could retain a previous stack garbage value and this condition here would not hold.


@someara, is there a reason this changeset and the version bump to 1.3.0 changeset aren't on the master branch?


Nice thanks for updating.


Bam Bam


All functions should have Doxygen comments (although it isn't super important for the imported code, it would still be nice to have them, so `make doxygen` doesn't print any warnings besides some false positives in `crypto.h`).


Is 19 still right? I don't recall anymore :(


This should be moved to `fastd_context_t`.


:+1: awesome! :D


@someara, is there a reason this changeset and the version bump to 1.3.0 (changeset 682f33e4a8924b8f30d5dc92177c9b37d97b9dae) aren't on the master branch?


I noticed a bug. Trying to find a fix right now.


This function should be renamed to `fastd_receive_android_tunfd` or something like that.


Line 2 clearly asks not to edit the file.


Может "Что ж" а потом запятую ? http://ru.wiktionary.org/wiki/%D0%BD%D1%83_%D1%87%D1%82%D0%BE_%D0%B6


@petebacondarwin do you want me to amend the commit?


develop (macumber) - 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)


Please add a separator below this option.


Somehow in this commit the permissions for VideoRecStation got changed so that it's no longer executable... maybe it was copied over on a FAT32 USB stick or something? Will push a fix.


1. I commented that old commit cause I didn't know how to comment a line, only a commit (I just found out how now!)

2. OK, I understand. I didn't know variableNode had been introduced after the pattern matcher. Although I would think that having a variableNode can benefit also the algorithms working on the atomspace, not just the user, but I see your point. However I don't think it is worth modifying, so I'll that aside for now.


No, not normal. Can you create an issue with the crash data.


this condition like the others can never be true.


After thinking about this again, I think this call should be moved to `bind_socket` in `socket.c` (which would mean changing the argument back to an int...)


If the old variant works, so does the new variant.

That is, when you have a "request.application" context (and the old variant requires that) - then you are importing it within the web2py application context. And if you are importing it within the web2py application context, then it will be a local import (=same "modules" folder), so you can just use normal import.

If you do /not/ execute this in the application context (but e.g. importing s3migration from a websetup co-app), then you can't use request.application, so the old variant won't work. But you can still use normal import /if/ you import s3migration into websetup like:

    from applications.eden.modules import s3migration

...because then s3cfg is within the same Python package, so it's a package-internal (~relative) import.


initial commit


Vamos vamos


Исправлю.


Calling the internet police.


si si si
lets gooo


vamo vamo vamo


> Remove eventing from iD.Connection and pass result data exclusively via callbacks. The context has wrapper methods for loadEntity and loadTiles that push the data into the default history. Everything uses these wrappers except conflict resolution, which calls the connection methods directly. (This is the approach I favor.)

Yes, this is better.. It's in ed49292


ye ye ye ye


Is `django.template.engines` going to be documented elsewhere? Most readers won't need to know to simply upgrade there code, but how the indexing works needs to be explained for other uses.


> I am not sure why you are looking at this from a cost / benefit point of view. I didn't ask you to implement it yourself, I just asked what you would do if someone happened to present us a finished implementation.
The benefit is obvious, better compatibility and removing cruft from our script compiler.

If we can get someone interested in doing the vanilla VM, we may also be able to get that person to do something more useful. As I wrote before I am not convinced that better compatibility with defects is desirable, at least if it means silently ignoring these defects. And the cruft is actually not that bad, even if I occasionally rage about it. Just a couple of lines that can easily be prefixed with an if statement, when we get the improved scripting language version in OpenMW 1.1.

> Regarding cost, is it really that difficult to do?

I believe it is. We are not just talking about instructions and functions (which is bad enough considering the number of these and the that the vanilla opcodes and arguments are probably as irregular as the vanilla script compiler). We would also have to deal with variable management, including variable access in other scripts.
And then there is the issue of broken byte code. I don't know what happens when the vanilla compiler encounters junk scripts. It might just create the good parts as byte code. Or it might create the defective part too. If we are unlucky, we could face a repeat of the script compiler implementation workarounds.


Line 22: change z to player z instead of 0?


Bro you have console.logs.....


Done


Thanks, Mark! Did you see my inline comment, as well? Looks like there's a problem when offset is 0.


huh?!


I only changed some lines on top, don't know why github show everything afterwards as changes, they are just line number changes


Hey Alberto Did u finaly manage to find a way to fix the manual network selection error issue yet or not yet
And by the way what does really this patch do ( ril: implement a new ExynosXMM6360 RIL which unsupports CB/CellInfo )


suggested edits for this commit: http://dpaste.com/380C1P6


I switched to callbacks.


Should be "y root coordinate value"?


    unless id
Is probably a bit more readable ^^


@mkgold: Missing <li> tags around the content—I've fixed this in the commit I'm about to push, but you asked where it was.


Everything in the geocoding stack is now lat/lng


However, I intentionally left the old variant in place just in case you'd like to revert it. So feel free...


Changing the `epoll_pwait` to `pthead_sigmask` and `epoll_wait` opens a race condition: if a signal is received between unblocking the signals and `epoll_wait`, `epoll_wait` will wait for its whole timeout when there are no epoll events, but the wait should be skipped in this case.

There are two possibilities to fix this:

The first is a workaround in `on_signal` in `fastd.c`, which is already used on systems without EPOLL: a NOP event is inserted into the event queue.

The second possibility would be directly calling `epoll_pwait` though `syscall` (caveat: the call signature is different from the libc wrapper). This would also reduce the number of #ifdefs...

I'm currently unsure which way to go here. Normally, I would prefer option 1, but the `epoll_pwait` wrapper in uClibc is broken on x86 (at least in the version used in OpenWrt), so that's another thing that would be fixed by option 2...


I'm afraid I don't understand this bit ...


I actually think this is pretty good. Would we want to have a default?


Another removed empty line...


I added template ``<script>`` tags to the page and the JS now manipulates and clones the existing markup rather than generating it.


Shouldn't you use
```
oldl3 | ATTR_AP_RW_BIT)))
```
instead? Otherwise you are not clearing permission to write.


Deleted `extraArgs = extraArgs || [];`. These are non-exported functions and extraArgs is never undefined.


ditto.


Please try not to move source lines in the generic Linux `fastd_tuntap_open` implementation...


Should be "y root coordinate value"


Whoops! didn't mean to do this! reverted.


Thanks!


:dancer:


Apart from the above mentioned remark - it is OK


LGTM


It is used in example/createKeys.php and only needed if you want to Setup a primary. It is not needed for verification.

No problem about the "mess". Your fork is a copy that belongs to you and with which you can do whatever tests you want. But if you just want to version something without making it public, just don't sync your local repo back to github. Your commits will just be saved to a local repository on your Computer from which you can also revert to any state.

If you want to Test something and want to make it
Public or want to remember the Point before you started your test, create a branch. You can Switch between all different branches with just one click or one Command line :)


maybe make this protected ?


```
TypeError: this.img is undefined
TypeError: this.$fig is undefined
```
Don't really see where the error is happening, but it's happening somewhere.


Whichever you like.  Nested double-negatives always make me do a triple-take and then I'm still afraid I've misinterpreted it. Thanks :)


Use `phenotips.globalHeader.search.placeholder`. See http://dev.xwiki.org/xwiki/bin/view/Drafts/L10N+Conventions#HRulesforstandardformelements


close RuleML/Issues-ruleml#46


Shouldn't the branch for `conf.android_tun == false` look like the generic Linux implementation? With the `ioctl` etc.?


It's what was there before (within cordova.gradle). Believe it's still right (but it's what we want to change for Lollipop)


all tests pass:
success: http://build.servo.org/builders/linux1/builds/479
success: http://build.servo.org/builders/linux2/builds/477
success: http://build.servo.org/builders/android/builds/514
success: http://build.servo.org/builders/mac1/builds/695
success: http://build.servo.org/builders/mac2/builds/696


@MarkusTeufelberger You are correct about online_delete. This setting in conjunction with online_delete would serve no purpose, and would be somewhat counter-productive since rippled would try to reach the ledger_history index after every deletion cycle. @miguelportilla You may want to check for the existence of online_delete greater than 0 on startup, and throw an exception with an error message that both options cannot be configured at the same time.


Don't put wiki markup here, only the text.


tank you very much


Leave `**` here.


Nice!


I think this is possible to achieve in only data driven for the non-self part similar to this https://github.com/Pizzalol/SpellLibrary/blob/SpellLibrary/game/dota_addons/spelllibrary/scripts/npc/abilities/crystal_maiden_frostbite_datadriven.txt


We need add here:
self._player = None
before checking self.enabled(), if we don't add this, will be undefined when called in get_all_*voices()


develop (macumber) - i386-Windows-7-VisualStudio-12: OK (2165 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-98.94881170018282%25-yellow.svg)


Keep the properties in alphabetical order.


we can do just:
```
return _check_modules():
```
If we need add more logic later we can do it.


Same, wiki markup outside the translation.


.gitreview
. . . . . . @@ +0,0 -1,5 @@
          1 -[gerrit]


The he -> The


fast-forwarding master to auto = 14587092


I think that it would be better to have interpret.extensions[ext] simply return the package name, e.g. coffee-script or iced-coffee-script and so on.

Using this, we can now look up the configuration from the newly introduced interpret.configurations object, e.g.

packageName = interpret.extensions[ext];
if (!packageName) return;
config = interpret.configurations[packageName];
if (config) do the extra stuff
else
   just require the package

What do you think?


This would mean that blocks will be processed only when the operator is idling?


This is unnecessarily slow as they are only *5* combining ranges.


We got by without MediaTypeOf until we hit a type without a sensible default, which makes me tend to think that there should not be a default.


Would be better
if manager is None:


Well, I'm sure you'll be doing something every frame, like a DSP interrupt or vsync or or something. As soon as you have such a periodic task, you never need downcounts longer than that, and I don't think you have 2 billion cycles per frame...

But yeah, not exactly critical :)


Yeah, spoiler toggling just don't work right on liveposts. It doesn't read them or something.


Awesome! That definitely works.


pip


It should be
```
oldl3 | ATTR_AP(ATTR_AP_RO)))
```


.gitignore
. . . . . .  @@ +0,0 -1,11 @@
           1 -*~


Wouldnt it be idling when there is no input


` »` should be included in the translation, since a RtL language might want to use `« اقرأ المزيد` instead.


`camera` -> `cameras`


Thanks, this happens when do a plain copy&paste ;-)


- fix typos
- fix formatting issues
- fix some translation issues
- improve the translations significantly (almost every paragraph).
- add useful comments and links to explain some terminologies.
- try to be more consistent from beginning to the end.
- add my name, since I think it is not minor change.


Same, include ` »` in the translation.


saw approval from Ms2ger
at https://github.com/mattnenterprise/servo/commit/5a548431119b1bf8f45958f97df72c0cf35a2bea


merging mattnenterprise/servo/globalref-by-value = 5a548431 into auto


mattnenterprise/servo/globalref-by-value = 5a548431 merged ok, testing candidate = 7800d987


thanks, will push this later tonight. BTW does the phantomjs collecting of data work for you? something has happend to my windows image, so right now it doesn't work for me (collecting using slimerjs works though).


This is a note lul


cool i can comment on lines


Or I can change it to emit head but don't see much reason unless input is given more priority over idle callbacks when there is no input.


Would be better
if speech.get_speech_manager() is not None:


This is something we want. However, the goal is to allow templates to be compiled into special objects that can render themselves, rather than needing a special function to get them into this form.

I'm not sure if it's worth changing the API for the short-term. That said, the code is good.


Use `phenotips.` as a prefix.


Looks good. I think there are places where we check whether this is user or kernel address (means we are testing MSB). Maybe some common macro would be more useful and clear (i.e. KERN_VA(va) or etc.). Just saying.


This implementation is unnecessarily slow for the common all-ascii case. And you can do without the 'tmp' sequence as well.


This line being removed is causing the Observation Length Historgram to not render in Achilles Web. We need to have both INTERVAL_SIZE and INTERVALS populated.


Right, I fixed it.


Here is a doodle: http://doodle.com/2rpctdpze4k9zi5s
rgds


why sort by class name if same priority? Any value in that?


Tests pass and reports the following:

>JPM info console.warn: firebug: 237 warnings or errors were logged to the platform's nsIConsoleService, >which could be of no consequence; however, they could also be indicative of aberrant behavior.
>
>JPM info
>34 of 34 tests passed.

which shows the warning/errors have reduced by 1 from 238 to 238.


@LuisZ54 Si, debes hacer una versión más en bower. (Un tag más)


These are used by tasks that will break if we remove them.


The first one is still being used in the footer for the target of the "Documentation" link.  Shall I remove it?  Or update it to go somewhere else?  If so, where, and should I rename it as well?


OK


Include the date in the message, like above, for consistency, and also to be nice with languages where there's more text after the date. For past tense, German usually has a verb at the very end of the sentence.

    patient.header.modificationWithoutModifier=Last modified on {0}


84205966-DefaultHVACConstruct (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-warnings-yellow.svg)


One of our goals is to ensure that all aspects of the front-end code can be built via Gulp.  Can the jsdoc gen be handled via a Gulp task that is invoked by Gradle?


People would keep reporting bugs there so I agree that it's probably not a good idea.


это был просто коммент что бы сделать коммит. ничего особо не менял. просто
нужно было что-то писать. ты же сам говорил что нужно больше коммитать. вот
и коммитаю [?]

8 января 2015 г., 21:20 пользователь Alexander Osin <
notifications@github.com> написал:

> Пробуй более детальное описание комитов. Что исправил, что немного изменил
> и т.д.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/majesteetlikkus/edupoll/commit/e401e8c96aa1c05d199b5e26a7ca42c4c3d97944#commitcomment-9207483>
> .
>

--
Best Regards,
*Maksym Kondratenko*


I just saw this.  I'll pull this and remove the unneeded routes.


The notification:

```
rx <notification from="39xx-1420654608@g.us" id="1977265161" participant="31xx@s.whatsapp.net" type="w:gp2" t="1420748185" notify="Wa">
rx   <subject subject="ciaoooo" s_t="1420748185" s_o="31xx@s.whatsapp.net"></subject>
rx </notification>
```


`more` is not a nice segment... It's not obvious what it refers to, use `moreActionsMenu`:

    patient.header.moreActionsMenu.label=More actions
    patient.header.moreActionsMenu.viewMode=Open in view mode
    ...


Test


Thanks, but this branch has very unstable RIL. You should patch master branch...


The token should be moved into the following (per the [Travis documentation](http://docs.travis-ci.com/user/code-climate/#Ruby):

```yaml
addons:
  code_climate:
    repo_token: 9b8c....
```


Uhhh? Tabs?


The warning will always be there. It is a warning it is not blocking.


nerd


The last word is misspelled : *physically*.


added real json schema validator (validating the end results), more schema tests and loosened up some key requirements in the swagger2-schema. One can now pass just `{}` and ring-swagger completes it to be correct. Same with empty Operations.

- https://github.com/metosin/ring-swagger/blob/master/src/ring/swagger/swagger2_schema.clj
- https://github.com/metosin/ring-swagger/blob/master/test/ring/swagger/swagger2_test.clj


Formatting is busted in this file


And actually `patientRecord` is better than just `patient`:

  phenotips.patientRecord.header.creation=


Hi @tobibeer,

it is not. It is just a starting point to figure out what browser is active. If a browser is not using webkit as javascript engine, then the property is undefined and it moves to the next property.

-Felix


OK


There's another label to translate a bit lower: `Push to $serverName`.

Anyway, this should be translated in `patient-data-sharing-api`, not in `navigation-api`.


ghay


we might need to add some logging here ...


This is f#cking awesome!!!


Kinda looks like something that should have been implemented as a unit test though... Wonder why they didn't make us do any unit tests, especially like these... Way to go Hanze :unamused:


These keys shouldn't be here, but in their corresponding modules. If a key is used in `xyz-ui`, it should be defined in `xyz-api`.


I suggest to make this a `:default`


The timeout is actually occurring on push with the git publisher.


Sure, glad it's useful.

I am on version 3.0.3, and confirm that phantomjs isn't working here in
Windows 2012. In the log, get about 18 repeats of this 5-line message for
each of the 3 default executions...

TypeError: 'undefined' is not an object (evaluating
'window.performance.timing')

phantomjs://webpage.evaluate():2
phantomjs://webpage.evaluate():3
phantomjs://webpage.evaluate():3

But I saw your issue on Github. I'm not having a problem with YSlow at all.

Hope that gives you an idea. I can try to debug it some but I'm still not
finished fully with the upgrade yet (tied back into Jenkins / JUnit)

Stephen

On Thu, Jan 8, 2015 at 3:13 PM, Peter Hedenskog <notifications@github.com>
wrote:

> thanks, will push this later tonight. BTW does the phantomjs collecting of
> data work for you? something has happend to my windows image, so right now
> it doesn't work for me (collecting using slimerjs works though).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/thebostik/doc.sitespeed.io/commit/e8a2757d4f1e0ec216f54494805dbba2e5008a69#commitcomment-9208326>
> .
>


Quick review the new OS strings are great, but I am thinking the best way to manage host groups is likely going to be through the view model per #5, no?

I'll go through some further review on this this weekend and hopefully get it merged in.


If you're seeing a timeout on push, that would usually indicate that the authentication information is not correct. Unless your Jenkins job is adding enormous content to the repository, the push should be a very fast operation.


@lgalfaso i don't think it makes sense to ammend this coomit, lets just try to follow commit conentions more stricly in the future


I was thinking an organization you own unless they fit in LlamaKit. My initial response is "Because that's how it's done?" Every popular framework I can think of lives in its own org. I'll address motivations anyway:

* I think the migration toward grouping repos in orgs supports the trend toward microframeworks. I was doing JavaScript when this trend sprang up. cujoJS is archetypal for grouping minimally related minimal utilities. But even Mantle was moved from GitHub's org to its own (not its maintainer's user).
* The only reason I'm aware of these libraries is because I follow you personally since researching Swift set implentations. I was pleasantly surprised skimming your other Swift repos. But you have 93 mostly original, mostly unrelated, mostly unsupported (logistically speaking) works. From the perspective of a developer searching for paragons of a Swift stack it's difficult to assess mature (many user's projects haven't been updated since beta) general utilities versus experiments, presentations, UI controls, or even alternative platforms (of which you have all).
* They're difficult to discover because there's no link to a shared entity. I can follow Traversal's externals (making them arguably interrelated) but your user page is conflated (although awesome). By comparison ReactiveCocoa's org includes a layout concept I expect a degree of continued support which I discovered through their org page. And their fork of LlamaKit lends credibility to it as a de facto standard (because it seems like everything from GitHub or Facebook should be standardized).


These are entries in the menu, which is not the same as the header. It gets out of the header and follows you when you scroll down. Use this key format:

    phenotips.patientRecord.actionMenu.editMode.save=Save
    phenotips.patientRecord.actionMenu.viewMode.edit=Edit


It should be `resources :posts`


Hello,
There is a problem with the armor stand, a no-member of the zone can exchange an item that is placed on it.
(ex: exchange a diamond helmet with another iron ...).

sorry for the post in a section may be inappropriate


How do you feel about putting FetchInterceptorProtocol in a separate file?


sounds good thanks


Why was this lead paragraph removed?


jijiji


@felixhayashi

I understood that, however, here we're not in css land where you have the browser default set last instead of first.

in JS, the browser default should be applied first and then any engine specific version if that browser default is not implemented, imho

in the above the standard always comes last in all four cases


rm :)


This can just be `> $file`


This is a little bit misleading to me. I always suppose that `belong_to some_name` is correlated with `some_name_id`. What is more there can be a situation when in the feature you would need to add second relation to the user...


Weil dein merge extrem unnötig war und nichts gebracht hat (abgesehen davon, dass die GUI zerschossen wurde xD)
Sprechen wir morgen nochmal drüber. Und Jochen und ich haben auch noch ein zwei Fragen an die Allgemeinheit.
Ich werden auch mal nach dem allgemeinen Stand fragen und ggf. Hilfe anbieten,w eil Jochen und Ich quasi fertig sind und nur noch debuggen müssen.


PostsController


Yes. ATTR_AP_RW_BIT == ATTR_AP(ATTR_AP_RO) but the second one should be set.


ok


r=mbrubeck


Buildbot [supports this pretty simply](http://docs.buildbot.net/0.8.10/manual/cfg-schedulers.html#configuring-schedulers) (Look for the `fileIsImportant` callable).

I think the best way to handle this would be to have Github send webhooks to a server that we run, we'd filter the webhooks there, and then pass them on to Travis/Appveyor if we deem them "important".  Github's webhook format is [pretty rich](https://developer.github.com/v3/activity/events/types/#pushevent), we don't even need to perform any on-disk manipulations like checking out repositories or anything; we can just look at the bundle of commits coming in on the current payload.


The timeout on push occurs because heroku builds all of the resources as part of the push, so for large web apps it can take up to 15 minutes.  Looking at the implementation for the other git 'additional behaviours' it seems like it should be easy to implement a similar timeout override for the push command.


I wasn't clear. I want the relation to be called author. The point with second relation is valid, though. Ok, let's leave it like this.


Codestyle: missing a space between the two parameters.


What's up ? :-)


The programs don't exist in the `$PATH` variable. As you state below, path just holds on to references to where the programs might be able to be found. I might correct the wording on this part just for clarity.


Note the comment about Buildbot isn't to suggest that we pipe everything through Buildbot; it was just to show that such a feature can be implemented very simply if we build the underlying scaffolding (e.g. identifying all changes accurately) properly.


Then, if it finds one, it executes the program, and if it doesn't it throws an error :grinning:


This will cause an extra whitespace at the start, remove the space between `#if()` and `$services`.


Codestyle: missing spaces after commas.


Sorry I was long-winded, I wanted to get the point across.

... and what perfect timing: William Ma just created a pull  request for sureal code that defines new callbacks that allow WordNode and NumberNode  to behave as if they were variables.  This is because he wants to ... well, I did not try to review if his use was in fact the best or the easiest way ... but it seems legal.  He is NOT using that code with BindLink, and so this code would not be triggered.

(It is almost surely the case that there is some equivalent bindlink to what he is trying to do, but that bindlink is probably much more verbose than what he came up with. That's sort-of the problem with these model-theoretic, type-theoretic approaches: there are many ways to do the same thing, and some are more verbose than others.  I have not yet discovered a good way to tell apart good and bad usage.)


zxczc


@mathiasverraes site doesn't update after commit


.autocomplete() completely unbinds all keyup events .unbind('keyup'), therefore if both are enabled and instantsearch comes first, it's binded events are unbinded (gone); Moving AjaxSearch before InstantSearch fixes this.


I applied your edits. Thanks.


Thanks @cfinke ! With that patch  -

when having set visibility options for widgets and Infinite Scroll attempts to check if the sidebar(s) contain widgets by IDs, *widgets get displayed at least*, while they were just lost before (see #1474).

But pay attention please - to me it seems now:

*When having Widget Visibility enabled, a condition like*

~~~
is_active_sidebar( 'sidebar' )
~~~

*will just never return true !?*


Indeed, I just renamed the file.


TIL - yet another Android oddity. Thanks :)


thanks fixed in develop


All fixed.


It is the julia convention to name function which is not safe with prefix "unsafe".
Perhaps, it should be called "unsafe_vec", since it depends on the existence of the R object,
and "vec" will be a wrapper of unsafe_vec together with preserve and release procedures?


I agree. However, for implementation clarity (and due to some time constraints), let me go ahead with getting a stabilized model in and a test case going. We can then come back and rectify the duplicated functions.


géniale sete comite! mersi je vé pouvoire pacer ma picine! :DDD


r+


Sorry @JW-Vinay , could you roll this back and recommit with unix line endings?


all tests pass:
success: http://build.servo.org/builders/linux1/builds/480
success: http://build.servo.org/builders/linux2/builds/478
success: http://build.servo.org/builders/android/builds/515
success: http://build.servo.org/builders/mac1/builds/696
success: http://build.servo.org/builders/mac2/builds/697


Otherwise LGTM :+1:


General thought regarding the code. Checking for !(l3 & (1ul << 0)) [as a define ATTR_something_VALID or so] may be more straightforward for the reader and if we have something in PTE but not in bit[0] then this condition will pass and the page is not valid. So IMHO checking bit[0] will be more bug resistant don't you think?


All right, I'll leave the branch just in case we change our mind.


Looks good.


Hi Al,

Yes, I have a specific use case.
I'm currently using ASValueTrackingSlider for a reader. It allows the user
to navigate inside a book.
For user confort, the new page is loaded only when the user stop using the
UISlider (cancelTrackingWithEvent: and endTrackingWithTouch:withEvent:)

So I could use slider*Did*HidePopUpView:, but I didn't want to wait for the
animation to end before loading the new page.
That's why I needed slider*Will*HidePopUpView to load the new page when the
user stop sliding.

Maybe there is an other way to resolve my issue, and I missed something.

Regards,
Vincent

2015-01-08 19:51 GMT+01:00 Al Skipp <notifications@github.com>:

> Just wondering if you have a specific use case in mind for this addition?
> The hidePopUpViewAnimated is only ever called manually, therefore you
> shouldn't need to be notified before the animation takes place. You can
> just perform any actions you need to before calling hidePopUpViewAnimated.
>
> Please let me know if I've overlooked something here.
>
> Cheers,
> Al
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vince4/ASValueTrackingSlider/commit/00faf2dfeb2aa2a22253ae076d9c8fc128ac3316#commitcomment-9207112>
> .
>


I think we should leave the option in that we can call every function with _and_ without the argument visBool


I don't understand (because I always forget how types works). Can you elaborate?


@Subjunk I can build whenever your ready. Should'nt be any issues now i'm back home and able to build releases much easier


fast-forwarding master to auto = 7800d987


@DavidIvy003 : you're up!


r+ (belatedly)


Why the empty node here?


. . .  . . . @@ +0,0 -1,18 @@
           1 -filter


@MarkyMarkNo1 If you have visibility conditions set on all of your widgets, and none of those conditions can return true before the `template_redirect` call, and `is_active_sidebar()` is called before `template_redirect`, then no, I don't think it can return true. There would be no way to determine whether the sidebar should be active, so it's a bit of an undefined expectation.


Nope, it doesn't matter. The browser prefix exists until the w3c declares this as standard. Until then, non-prefixes do not even have to work. Try using "fullscreenchange" or "fullscreenElement" instead of the moz-prefixed version in firefox..

Try
     document.addEventListener("fullscreenchange", function(){ console.log("changed!");}, false);
Instead of
    document.addEventListener("mozfullscreenchange", function(){ console.log("changed!");}, false);

Once it is a standard, mozilla will simply ignore the prefix and use the non prefixed.


saw approval from mbrubeck
at https://github.com/pcwalton/servo/commit/53b74ae853194940ed1a19f9255a70869520d0af


merging pcwalton/servo/text-rendering = 53b74ae8 into auto


pcwalton/servo/text-rendering = 53b74ae8 merged ok, testing candidate = 1a021f66


separate ticket to clean this up not to use absolute paths?


Why are all the binops lowercase and the rest uppercase?


Those are fair reasons. My initial reaction was that other services don't generally do it and we end up with this information in logs, but I don't see any harm when I think more deeply about it.


saw approval from Gankro
at https://github.com/squidpickles/rust/commit/b88ae323d1297b54a119eed0925aa2a156e8a578


merging 2 batched pull requests into batch


status: {"merge_sha": "3149a6d7b3ddeb6284e2153e594f01bc7271534b", "rollup_pulls": [[20735, "b88ae323d1297b54a119eed0925aa2a156e8a578"], [20766, "018730b39aa64404f757ca0350e0cb131384eaf2"]]}


Testing rollup candidate = 3149a6d

**Successful merges:**

- #20735 Spelling fix compatibel -> compatible (squidpickles/master = b88ae323d1297b54a119eed0925aa2a156e8a578)
- #20766 doc: Fix unresolved import in Bitv example (tsurai/patch-1 = 018730b39aa64404f757ca0350e0cb131384eaf2)


oops, sorry to include the scratch file!


Try using Argparser.argparser to read from stdin


. . . . . . . @@ +0,0 -1,11 @@
           1-*~


Let's move this comment to its one line! And say something about why we are doing this!


@seif I just replaced this dependency with nunit as you suggested.


Oh bugger. I misunderstood Github's markdown syntax. Feel free to delete this - the intention is fairly obvious.


Would this be easier to read if you write it with a heredoc?

```
pp = <<- MANIFEST
class { 'elasticsearch': config => { ... }
  manage_repo => true,
  ...
}
elasticsearch::instance { 'es-01': ...
}
```


You don't, in response to the title.


looks good


@bors: retry


@oncletom This test if for scenario where selector is "null" (see tweak to Imager.js above too).


Yeah vi0 is fixed to zero, just like vf00 is fixed to 0.0 0.0 0.0 1.0


@mach6 Done.


Shouldn't the 2nd run be `apply_manifest(pp, :catch_changes => true)` ?

From http://puppet-a-day.com/blog/2014/04/22/modern-testing-of-modules/
>Now we do the actual test. apply_manifest() takes a manifest and various options about what the outcome should be. On our first run we’re looking to “catch any failures” and then on our second run we’re looking to “catch any changes”. This allows us to be confident we’re not changing the state of the machine on multiple runs if everything is set correctly the first time.


Here I splitted the multiline regex in a oneline


```
document.addEventListener("fullscreenchange", function(){ console.log("changed!");}, false);
document.addEventListener("mozfullscreenchange", function(){ console.log("changed!");}, false);
```

Neither logs anything when I toggle fullscreen?


@kimegede @kittylion

First install
```
ln -s ~/.dotfiles/sublime/js.sublime-builde ~/Library/Application\ Support/Sublime\ Text\ 3/Packages/User/
```
Then in sublime text go to `Tools` -> `Build Systems` and make sure that `js` is the selected build system. You can now do `Cmd` + `b` when editing a javascript file and it will execute the file and show the output in a console.


as is made evident by this line, your text editor is using tabs. find the setting in you're prefered text editor/ide that turns this off, then refactor this file by replacing all tab characters with an appropriate number of spaces for the language you are using. In C++, that number is 4 I believe.


you can use Stud::Try here, if you wish to have a faster run.

Stud's try will try a call a few times and retry on exception with exponential backoff (max 2 seconds per sleep, by default)

```ruby
Stud.try(10.times) do
  should be_listening
end
```

Thoughts?


TIL &Proc.new, very useful. Required some reading :)


recommend always using 'do ... end' for non-inline blocks.

```ruby
# This is ok:
it { something }
# This is OK, too
it do
  something
  something
end
```


concatenate with
var="foo"
var+="bar"
does'nt work on linux, solved with :
var2="foo"
var3="bar"
var=$var2$var3


Oh! Sorry, my mistake!


saw approval from Gankro
at https://github.com/squidpickles/rust/commit/b88ae323d1297b54a119eed0925aa2a156e8a578


merging 2 batched pull requests into batch


status: {"merge_sha": "6354d60ede5e2a7e60fa46f85243efd8dbe89711", "rollup_pulls": [[20735, "b88ae323d1297b54a119eed0925aa2a156e8a578"], [20766, "018730b39aa64404f757ca0350e0cb131384eaf2"]]}


Testing rollup candidate = 6354d60

**Successful merges:**

- #20735 Spelling fix compatibel -> compatible (squidpickles/master = b88ae323d1297b54a119eed0925aa2a156e8a578)
- #20766 doc: Fix unresolved import in Bitv example (tsurai/patch-1 = 018730b39aa64404f757ca0350e0cb131384eaf2)


Thanks again!

Only a small issue: I think there's a typo in line 481 of dictation_format.py...


I wholeheartedly agree with your comment in MainMenuActivity.java , line 495. There ultimately must be a better place to reliably initialize the miscellaneous fragments. I just added DashFavoritesFragment there as an emergency change over the Christmas break, to point out that there needs to be a proper initialization for non-compass devices, where update Compass is never triggered. Hence I placed it there, where other stuff was also initialized (one of which, DashAudioVideoNotesFragment, you just removed.) Did not want to interfere too much with Denis' code, because I think it is under heavy development. But yes, lets please find a more standard place to initialize the fragments!  :-)


Yes please :-)


This should probably not go on master :)


These should be moved to the test and cobertura targets


Thanks for the legwork to make a single source of bounds truth for both client and server.


Это же `log_alpha` и `log_beta`, правда?


handleIdleTime is called only when the operator is not consuming and events and emitting them. This is why I was processing the blocks from the queue head in the port process.


Download package


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1094 of 1094 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)


:D

On Thursday, January 8, 2015, Tyler Renelle <notifications@github.com>
wrote:

> feat(presskit): add presskit page cc @lemoness
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/HabitRPG/habitrpg/commit/cba232ef83fb15c3fd74ed8eb535b5dc3d317bad
>


Mozilla?


84205966-DefaultHVACConstruct (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2163 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-98.85740402193784%25-yellow.svg)


@mfrancois now that I look in more detail the regexp, shouldn't we escape the dot `.` like this `/config\.xml$/`?


cbecc-materials-bugs (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2067 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-94.46983546617916%25-yellow.svg)


Thank you. I see, it returns `true` in fact, once I have *at least one widget without visibility options* set.


It's generated by train_gb.cpp


Nice work!!!!


There hasn't been a new release of lein-cucumber since the updated project.clj file. Are you planning on updating the plugin anytime soon?


>  I am not convinced that better compatibility with defects is desirable

I strongly disagree with this point. Just because we find indicators of parts of the mod not working as the author intended them to, doesn't mean we should refuse to run it. That's like removing software from a repository just because you found some bugs in it. Everything has bugs.
After all this is a video game and not a space travel program.

Since the discussion isn't going anywhere, I will shut up for now. But if we get a volunteer for implementing the code converter, or if I decide that I want to work on it myself, then we can continue from here.


cbecc-materials-bugs (macumber) - i386-Windows-7-VisualStudio-12: OK (2174 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.3601462522852%25-yellow.svg)


missing "t" in existing


Small thing - can we store `screenTypes[i]` in a var so we don't have to re-access the array again?


Hm. Is this commit the cause, or did it stop updating earlier?


Oh I will add that.  That's a good suggestion.  I see what you mean.

Date: Thu, 8 Jan 2015 09:46:05 -0800
From: notifications@github.com
To: SpongeAPI@noreply.github.com
CC: sibomots@outlook.com
Subject: Re: [SpongeAPI] Added back getInterval() method (119ad29)

@sibomots I'm not sure whether you got me right.

No i did not meant the creation of the task.

In the creation process I totally agree with you that the interval shouldn't be Optional, when creating a repeating task.

I was referring to the Task.getInterval() method.

A repeating task is executed every  ticks.

A one time task is not repeated -> it has no interval (absence / null != 0). You represented it as 0, but it could have been -1 as well, because it does not make any difference at all. It's a magic value, that indicates the absence of the interval. The 0 could also be "all the time" (execute it immediately again when it finished the previous execution). In contrast to this method we are currently using Optional in (almost) the entire API  to indicate whether something is/can be absent.

The Scheduler has a method getScheduledTasks() for all the returned tasks you don't know whether the are repeating and IMO checking the existence of an interval with Optional.isPresent() is more straight forward than <1.

If you don't change it, please add some javadocs, to the method that state, that getInterval() == 0 states the absence of any interval / the task will not be repeated.

—
Reply to this email directly or view it on GitHub.

 		 	   		  =


Is that a custom format? You should have used something common, like `1h20m30s`.


Ukulele!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!


The 10 steps in driver.py are more up-to-date.

On Thu, Jan 8, 2015 at 3:43 AM, merlinzone <notifications@github.com> wrote:

> Thouis, I have read your work.
> But which workflow should I choose? can you give me some guide?
>
> 1.the 7 steps workflow in README.TXT
> ClassifyMembranes/classify_image
> Segment/segment.py
> Control/dice_block.py
> WindowFusion/window_fusion_cpx.py
> PairwiseMatching/pairwise_match.py
> Relabelabeling/concatenate_joins.py
> Relabeling/extract_label_plane.py
>
>    1. the 10 steps in the Control/driver.py ClassifySegment_Image
>    segment_image.py Block dice_block.py FusedBlock window_fusion_cpx.py
>    CleanBlock clean_block.py PairwiseMatching pairwise_match_labels.sh
>    JoinConcatenation concatenate_joins.sh GlobalRemap create_global_map.sh
>    RemapBlock remap_block.sh ExtractLabelPlane extract_label_plane.sh
>    ExtractOverlayPlane extract_overlay_plane.sh
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Rhoana/rhoana/commit/57dedc066afcedcdd5048a81cb7646fba9b335f3#commitcomment-9198176>
> .
>


I ran the test without this if first of course.


develop (Myoldmopar) - 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)


If you need logging messages you should user console.log() and watch for them in the developers console.

alert() is blocking your flow with prompts.


Do not use !global on a global variable.

Ref: https://github.com/nex3/sass-guidelines/commit/c6113377bb35c5ed57dba6056be9e378b309a7c7#diff-d680e8a854a7cbad6d490c445cba2ebaR1669.


Why was this line removed? Ruby is required by `build_compass`.


Add `@param`.


You can use an empty file for the settings.

On Thu, Jan 8, 2015 at 4:18 AM, merlinzone <notifications@github.com> wrote:

> where could I find the settings_file? can you sent my email(
> merlin_liu@163.com)
> 390 settings_file = sys.argv[1]
> 391 os.environ['CONNECTOME_SETTINGS'] = settings_file
> 392 execfile(settings_file)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Rhoana/rhoana/commit/c3c4bf39c1ecee58da4a16c210f53fc44ff2b955#commitcomment-9198573>
> .
>


Please correctly indent your code to the right when you implement the callback.


Add SassDoc comment.


Yes, it's possible and I actually thought about it - you would need to traverse the entire tree until the first SubqueryLeafNode if any. The only reason I went with the current solution is the performance implications. But, i guess, in case of the create view it is not that critical.

Would it be better a method on the JointNode itself (bool hasSubquery()) that the AbstractParsedStmt?

Do I need to create another pull request with the changes? or will this one be updated automatically once I commit changes to my branch?


The transition should be performed once you finished all the ops that you need to do (clean-up session, unload models)


Leave a `@return` statement outside of any conditional statement, or a custom `@error` directive.


@dkjer this should be `_ = tmp`


missing newline at EOF


I recommend storing the scope of this in a variable named _this. You'll get to see this practice among many developers.

var _this = this;


missing semicolon


semicolon


typo: should be `compatibility`


Also, can we correct the comment here - thanks!


Jacob can you post a log using this build


very nice!


more detail is needed in this area as it is crucial to survival


Read: http://stackoverflow.com/questions/729692/why-should-files-end-with-a-newline


Use keyword?


Edits for this commit: http://dpaste.com/2GES8S7

I am not sure if the organization of this info is the best. How to implement a custom backend seems like it belongs more in something like "ref/templates/api" than in a topic guide (which I think of as targeted at beginners). I guess it's short and easy to skip though. We can see what feedback we get if you think the organization is fine.

Is a miminum version of Jinja2 something we should document. Trying to run the tests with 2.6 results in `TypeError: __init__() got an unexpected keyword argument 'keep_trailing_newline'` but that seems like a test specific option, so maybe it's actually compatible in practice.

Will you add the docs for `django.template.Engine` before commit so we don't have broken reSt links?

Nice work as always.


Т.е., отдельный timestamp было решено не вводить? Мы подразумеваем, что все события отмечаются сразу же, нигде накоплены быть не могут?


Ship it!


Well then. Bone and Scorching removal I agree with, I was expecting Colored sand to be removed, that fits even less than torches, I thought torches were alright + they were already being used although I could see an argument for them not quite fitting, warning signs will bring user protest, they were already being liked and used. But as I say, I need to talk to you properly on IRC. I will wait as long as possible today but it is getting late here in the UK, and I need to sleep for exams.


`than the %s` -> `than the repository %s` or `than %s`


Add `@type`.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/481
success: http://build.servo.org/builders/linux2/builds/479
success: http://build.servo.org/builders/android/builds/516
success: http://build.servo.org/builders/mac1/builds/697
success: http://build.servo.org/builders/mac2/builds/698


What?


Add a warning.


событие остается в базе с timestamp создания. там есть еще один по дефолту, но он не используется. не совсем понял вопрос.


Eine Runde neue Schlüsselpaare für die Gates bitte! :wink:


aqui se crearan los aportes por cada uno de los usuarios


Add SassDoc comment.


Looks good.


r+


shouldn't the url be urldecoded here?


I used the code from the newest abtsys branch but without userland specific parts.


fast-forwarding master to auto = 1a021f66


As we discussed, I'm going to hold off on this fow now.


Why go with this approach instead of say:
```python
  ctx_factory = ssl.DefaultOpenSSLContextFactory(
      ns.ssl_key,
      ns.ssl_cert,
  )
  twisted.internet.reactor.listenSSL(ns.port, factory, ctx_factory)
```
?


Fixed copyrights.


First you need to check if the client made a request with the isAuthenticated flag set to true.

```javascript
else if (req.query.isAuthenticated === 'true') {
  // This means that the client is asking the server if the user who made this request is authenticated or not.
  if (req.isAuthenticated()) {
    return res.send({users: [req.user]});
  } else {
    return res.send({users: []});
  }
}
```


Might want the name of this to be more scoped (e.g. findSelectAll or findAll)


cbecc-materials-bugs (macumber) - 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)


Used PCB_LR value instead of hard-coded "30"


Removed wrong kdb_trap() from EL0


It's good that get_iterable is being passed the request object.  In 2.3.11, attempts to modify the queryset based on user identity are incomplete, meaning we can't really do it.


It needs to be done on final close or now and then not each write. If you do it each write you'll kill CF cards very rapidly as they'll keep pushing the bits to the media.


Awesome! People will love this :-)


Drawable does not exist.


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


merging glennw/servo/cef-resources = cf047d6c into auto


glennw/servo/cef-resources = cf047d6c merged ok, testing candidate = 968d1c2a


zxombie:
```
You should read #address-cells and #size-cells here using OF_getencprop.
It's misnamed as it's not fdt specific and is broken on little-endian systems when
either of the two properties are missing.
```


This should be the vm_or_template factory.


Last time it took longer, but it worked.


when i turn four i too will know the power of adding the extra dash


User error. Pushed!
-s

On Thu, Jan 8, 2015 at 11:48 AM, Ben Dean <notifications@github.com> wrote:

> @someara <https://github.com/someara>, is there a reason this changeset
> and the version bump to 1.3.0 (changeset 682f33e
> <https://github.com/opscode-cookbooks/lvm/commit/682f33e4a8924b8f30d5dc92177c9b37d97b9dae>)
> aren't on the master branch?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opscode-cookbooks/lvm/commit/6bcd348d767e67b1b04cce844046a59814760484#commitcomment-9207928>
> .
>


Or contraseña?  I only ask because you did this everywhere else, but maybe here it makes sense because it's a technical term.


this should not be needed... the WorkQueue itself has a lock and only calls the methods (except _process) while holding it


zxombie:
```
From the device trees I've seen the only children of the GICv3 device are ITS.
I would prefer the parent doesn't restrict which children it allows as if, for example,
a future revision of the GICv3 added some other child type we would need to then
add support here which could easily be missed.
```


Maybe we should also think about switching this domain to a redirect to meetup. It's duplicate. What do you think?


zxombie:
```
This code looks familiar, it would be nice if there was a central function to merge this,
ofwbus_setup_dinfo and simplebus_setup_dinfo.
```


zxombie:
```
Looks good
```


s/clustAuthInfo/configClusterInfo ?


I'm thinking of "ref/templates/api" as "DTL's API". I'd rather not mix bits of "Multiple Template Engines" there and I'm short on content to fill an entire page about "Multiple Template Engines".

The alternative is to keep only the bits about "Multiple Template Engines" and move them to a different location. But then I don't know what "topics/templates" should contain.

I'm happy to reorganize things as long as I don't mix DTL and MTE. Implementing both in `django.template` and having multiple `Template` classes is confusing enough.

----

There's no minimum version of Jinja (AFAIK) as long as you're only using supported options. Tests use the new-ish `keep_trailing_newline` option for consistency with the DTL, allowing to reuse the same tests.

----

Yes, I will add a commit with docs for `django.template.Engine` before merging this PR.


Remove the .xml...it implies Microsoft and you are going generic.


Not needed - already done by PKI_Malloc


expect syntax? like in what you replaced?


Is it necessary to confirm that start_time or end_time is nil (and not set as an empty string for instance)?


zxombie:
```
There is no need to prefix everything with arm_. We know it's arm as it's in a GICv3 driver.
```


I think just call this "status methods" without the methods listed.  That way if someone adds or changes something below it doesn't have to be changed in two places, and also the it lines should show the method being called anyway.


>break;
:^)))))))))


zxombie:
```
How similar is this file to sys/arm/arm/busdma_machdep.c? Could we just use the same file on both?
```


Eventually all the other vm factories should derive from this one, but that can be done in a separate PR.


What happened to `actions`? Is there a simpler way?


I have done the changes.


Let's convert the logout request to a POST request to /api/users/logout. This way we can move it in the users.js and delete this file.

Also, it's a better practice to implement logout as a POST request.

http://stackoverflow.com/questions/3521290/logout-get-or-post


some tests failed:
failure: http://build.servo.org/builders/android/builds/517


I have referred to the following answer: http://math.stackexchange.com/questions/108393/is-the-composition-of-n-convex-functions-itself-a-convex-function

Do you have a counter-example in mind?
Or if you agree with the result maybe my formulation is confusing?


We should make sure this is not `master` when being installed, as it might break future calls to `pip install -U ros_buildfarm`

I'm not sure about the correct solution.


Yeah, we can move the content from the site to meetup pages. We can't add
events in the past on meetup, unless they added that recently.

___________________________________

    Mathias Verraes
    Software Consultant
    Value Object Comm.V <http://value-object.com>
    Email <mathias@verraes.net> - Blog <http://verraes.net> - Twitter
<http://twitter.com/mathiasverraes> - LinkedIn
<https://www.linkedin.com/in/mathiasverraes>
___________________________________

On 8 January 2015 at 22:27, Tom Van Looy <notifications@github.com> wrote:

> Maybe we should also think about switching this domain to a redirect to
> meetup. It's duplicate. What do you think?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mathiasverraes/domaindriven.be/commit/a955ccc61617b98d8aef3062c87c64db8eba4b17#commitcomment-9209489>
> .
>


> s/clustAuthInfo/configClusterInfo

fixed.


В моей реализации, например, можно было зарепортить событие "три часа назад, в 15:00, пропал интернет", условно.
Впрочем, будем решать проблемы по мере их возникновения.


Alright, thanks


Хорошо что комитишь. Можно же еще лучше :)


Still buggy, tabs are not working, context errors.


saw approval from jdm
at https://github.com/Manishearth/servo/commit/9d2466bff509628ea182165bbe1bca301b0eae83


merging Manishearth/servo/android-warn = 9d2466bf into auto


Manishearth/servo/android-warn = 9d2466bf merged ok, testing candidate = e0d270b9


That's a good point, I'll add that to the test.


This won't run the eunit tests. Need the `-DTEST` to trigger the `-ifdef(TEST).`


Стоп-стоп. Перестали мы логить включение света одной железкой, но начали другой, в базе удалили первую — и кабздец всем событиям?


I would defer that decision.


CREATE TABLE `chat` (
  `id` INTEGER UNSIGNED NOT NULL AUTO_INCREMENT,
  `from` VARCHAR(255) NOT NULL DEFAULT '',
  `to` VARCHAR(255) NOT NULL DEFAULT '',
  `message` TEXT NOT NULL,
  `sent` DATETIME NOT NULL DEFAULT '0000-00-00 00:00:00',
  `recd` INTEGER UNSIGNED NOT NULL DEFAULT 0,
  PRIMARY KEY (`id`),
 INDEX `to` (`to`),
 INDEX `from` (`from`)
)
ENGINE = InnoDB;


Which parts are you talking about? I think you're confusing the helpers with something else...


Obviously you're going to get 200 spectators if you global declare. I got 1200 people in the Other Metas room with a global declare. I disagree with this format having a ladder. All it does is put the final nail in the coffin of the Gen 1 OU ladder.


There's a `return` missing in what should be `return BLAS.gemm!...`. So currently we're multiplying such matrices twice, once by a fast path and then again by a slow path.

An easy mistake to make. Having something like codespeed working again would catch such things easily.


Fixed in master


Randbat players don't prefer the OU formar and otherwise. This has helped reviving the old gens room and might help it from now on.

Right now: 10 ladder battles (while a tournament is on), more than all non-OU tiers. Almost together.


@eregon @nirvdrum there's now a couple of subclasses of `DispatchHeadNode`, `CallDispatchHeadNode` and `DoesRespondDispatchHeadNode` - create these using `DispatchHeadNodeFactory`. `CallDispatchHeadNode` has methods on it that return a particular type and will raise `TypeError` if it doesn't get it. I wrote those for `Math` which passes all the specs so they hopefully do the right thing in most cases. This will help us to get rid of all the random casts we often have after calls to dispatch.


Code_Cleanups (Myoldmopar) - 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-68.87626262626263%25-red.svg)


+1
The torches and warning signs are awesome!


some tests failed:
failure: http://build.servo.org/builders/android/builds/518


develop (macumber) - Win64-Windows-7-VisualStudio-12: OK (2166 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-98.99451553930531%25-yellow.svg)


Yes, this is because it is a new format. Say that in a week.


@tibbe This is my understanding of our current dependency policy. Do you concur? Or do we need to be more conservative about `cabal-install`'s dependencies, too?


Oh my bad. That's a bit confusing. Why have a separate function at all?


Thanks, updated.


Kathryn, I would like to take a different approach to these buttons if that's okay. The downside to using .btn-group-justified is that if we have more than one row of buttons—say 8—they will all be squashed together in one row or we'll need to manually group them into different .btn-group-justified groups.

Here is an example of how I'd like to implement this that I think will give us the flexibility we want:

http://codepen.io/lknarf/pen/dPNRdZ

![image](https://cloud.githubusercontent.com/assets/1809908/5671341/055ec284-9755-11e4-8218-0cdaac087724.png)


r+


@Mick43 Do you have an updated version of Spigot?


man kann in den Readme dateien in den ordnern lesen wie man das zeug abzulegen hat, und ich hab ihm 2 wochen zeitgelassen das zeug zu verschieben oder umzubenennen.. das muss reichen. er hats nichmal kommentiert, also warscheinlich auch nicht gelesen ;)


I don't know if we have a policy w.r.t. building with the libraries that shipped with an old GHC. It makes sense I think, as it's really hard to upgrade any libraries without having Cabal the library installed.

@dcoutts ?


saw approval from Manishearth
at https://github.com/Ms2ger/servo/commit/5f31da82bdbebe34ae1b721a60df07e4e069b1f0


merging Ms2ger/servo/if_let = 5f31da82 into auto


Ms2ger/servo/if_let = 5f31da82 merged ok, testing candidate = 4cd9eb12


awesome thanks!  is there a new binary reflecting the changes or should I compile myself?


er war nicht traurig. aber kora hat geheult.


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1094 of 1094 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)


sie hat ja keine blauen augen.


@chrisseaton I could have implemented this is Ruby as `to_a[1..-1]`.  That looked like it'd be doing a lot more and the Java here was fairly compact.  But I can change it to pure Ruby if you'd like.


Ok, CEGUI_DIR didn't work, but CEGUI_HOME did!!


I think that we don't need add another type of extension to create a thing like that. Will be more complex, and we are creating divisions. We can modify the code to allow load the applet.js file inside a desklet forder or load a desklet.js file inside a applet folder. As example, please see the Drives Manager implementation http://cinnamon-spices.linuxmint.com/desklets/view/12  Some thing like that could create the same effect, really the idea will be merge the desklet an the applet behavior on only one extension type on a composed extension. This will reduce the types of thing that we can create without lose the funcionalities.

Thats will be ok, but i think thats this is inconsequential to this issue. I considered a problem lose some functionalities, and now we lost at less the functionality to receive the keyboard events from a normal cinnamon mode. I can not see this as good thing, this will not only affected the sticky notes behavior, also will affect any other desklet that will can introduce an St.Entry.

So, we can enhanced cinnamon, allowing more complex types of extension, or the possibilities to load composed extensions... But on my point of view, here we have and we continue to have a problem.


Right.  I went with the justified group just to get something going, with the assumption it would get replaced eventually with a table or something once we get to adding larger groups.  I'll go ahead and close this PR if you'd rather work on it.


r=larsbergstrom


Fixed in 4b6aaa6ef5d749de1f369ebe30941940d8107a52


I have a few questions regarding your comments, so I'll go over them in the meeting with you on Friday.


This breaks the build http://hydra.nixos.org/build/18391578/nixlog/1 we do seem to need `optparseApplicative_0_10_0`.

I could not figure out how to force cabal2nix to do the right thing.


This feels so dirty... :shower:


Can this be made toggleable? When I'm clicking around an area I just visited I'm much more likely to remember what portals I flipped than what portals I captured. Especially on mobile it helps a lot to not have to go into every portal to check if I capped it and instead just have to open the WAY fewer portals I flipped and set them back to uncaptured.


For lines 69, 79, and 89 is there an expectation for these selections?  How do we know the values are submitted correctly or even submitted?  Thanks!


dope code, bruh


# Data Science Coursera


toRaw() method has been deprecrecated in Xcode 6.1


No @ovnicraft , en la 8.0 ya no se necesita, ni los otros dos típicos cr y  uid


Here are the latest continuous releses, https://ci.appveyor.com/project/hadouken/hadouken/branch/develop/artifacts.


r+


Commit title has a leading `coverity/:`.


dataWithContentsOfFile() function has been deprecated


/play trololo


:poop:


Bouuuuyaaaaaaaaaaaaaa


can't we use the default 'uri' module here? In ruby 2.2 they changed a lot of the URI RFC behaviour


completely fixes #21


Good idea.


Fix was for #626


I think that one should remove `:` after the name `Matthew Hutton`. It's sure a typing error.


Woah, finally!


New location


@halida It is effectively set to true by default if you import the relevant integration **before** bootstrap-sass, as per documentation.


was this code path not tested?


Fixed :).


Maybe, dunno. I switched to Addressable in the CMSScanner because I was annoyed by something in URI :x


Oh yeah it means Can't excute at all. So they can't get any import sources on libbacktrace. And I can't understand which can --with-host-libstdcxx='-Wl,-lstdc++,-lm' do. I want to change as libcxx but it doesn't work it says no stdio.h no such file or directory when I change to libcxx.


Is it not expected that the selections were successful if this test passes? Would the feature test not break if the selection was not a success? With regards to your second question (How do we know the values are submitted correctly or even submitted?) there currently is no way to know if it was correctly submitted since there is no functionality that exists to view if an emotion was positive or negative. This functionality will be added in other existing stories.


Hm, the latest elm-make builds from HEAD locally. Maybe your haskell-ng changes fix this?


develop (macumber) - 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)


I'd say close this and I'll work on it later in the sprint.


Merged into 3.0.x in 905265ea0ebd0f0166de3f8ace63bcc995cb1cee

Thanks


Nothing to add


And what was the purpose of that prefix by the way?


maybe we should make a boundary match here so "asdver=1.1" will not match? Example: http://rubular.com/r/WoWe3OUitz


it just fell back to the slower generic version


Change file names


Thanks. This commit is back ported so could you also back port you fix?


Done.


See https://github.com/JuliaLang/julia/commit/210102850be6c35fe7ef947c870333b55d370d82


yea, sure :) feel free to edit the pattern ;)


Please add a comment as to why this is necessary.


Should update the description to document the support for multiple files/directories.


This role needs a guard for `developing_and_testing` because the instances we build AMIs on should (hopefully) have enough RAM to build the Mapnik stuff.


Ok.  By the way, this PR doesn't do anything with the justified groups, it just styles the buttons themselves.  `@include button-variant` seems pretty neat for a way to set styling on bootstrap buttons while keeping hover and active state behavior consistent.


Maybe store the string ``"Are you trying to build a nonlinear problem? Make sure you use @addNLConstraint/@setNLObjective."`` in one place as a constant?


@xes No I'm sorry I didn't receive a message it seems. If you can put the samples on your favorite filesharing service and paste the URL's in a reply on this message you can be sure I will be able to find it. I did not yet find a description of the protocol for LD3091, that would also be of great help.


I used the FQN for this Customer type since I have the Customer area and I had to avod the name conflict.


Ah, I see... however, looking at that example code, I find he standard to be tested first. ;-)


Cool. I wasn't aware that arguments could be supplied multiple times.


@mbertheau Should there be a `dashboard/partials/pagination.html` or should we change the link?

@jlmadurga Yes, the dashboard should use Bootstrap 3 for v1.1 so we should avoid the problems.


Everything is fine, but part_builder->uncompressed_size is already guint64.


@dougbu Please review this. Let me know if you have more concerns on this.


This is an odd place to ask that question, but...

Eclipse with the M2E plugin automatically downloads the javadoc attachment for you from the Maven repository. I'm guessing NetBeans does similarly, though I didn't test it. So you shouldn't have to manually attach anything.

Please direct future questions like this to the [imagej-devel mailing list or IRC](http://imagej.net/Help).


Good point. worth trying out at a later point when i go over all the testing stuff again.


Good point, need to validate why i do this and not the catch_changes.


I'll remove this duplicate line before checking in.


Thanks for the catch, will address.


could do that as well yeah. will implement at a later time.


It should have been a single line, stupid me.


will test it first on some blogs :)


I'm guessing this is so you can predict output for unit testing?


Got it. I didn't realize that btn-group had been included in an earlier PR.

In that case, +1. Let's put this in.


The proper way to build with a nonstandard version is to add an override for the `optparseApplicative` argument in `haskell-packages.nix`.


@andreasnoack this is why I prefer allowing commits to stay on master for at least several days instead of backporting immediately most of the time


If that works, can't think of a reason. Copied this off an official [tutorial](http://twistedmatrix.com/documents/current/core/howto/ssl.html).


I don't get it...


Looks like your regex is working :)
`[+] WordPress version 3.8.5 identified from stylesheets numbers`


Hehe, yes, I saw it too but it really makes no difference :)
don't worry, it is a standard soon and then it doesn't matter anymore anyways ;)


What data would we have on tea times that had zero sign ups?

+10000 on having that stat


Looks good.


We don't doc it well at all.

----- Original Message -----
> Cool. I wasn't aware that arguments could be supplied multiple times.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/smarterclayton/kubernetes/commit/0194b9707ffd0617e19ba34c8ebb1142d9a9e012#commitcomment-9210032


Remove this comment.


<translation>Die Exportsyntax lautet: --export-map [Format] <TMX-Datei> <Zieldatei></translation>


Same here, we don't need it anymore.


@karianna In the meantime, you may use the "needs: merging" label if you would like to mark PRs ready for merging.


<translation>Dateiendung nicht eindeutig. Exportformat konnte nicht festgestellt werden.</translation>


develop (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2066 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-94.42413162705667%25-yellow.svg)


<translation>Exportiert in %1</translation>


Check the alt-text


<translation>Staffelachse</translation>


this suddenly broke installs with old npm versions on 0.10. i do not have the ability to update npm on servers and i'm not directly using this dependency, either :(


<translation>Staffelindex</translation>


can you change it to `>= 3.1.0 < 4` and publish an update, pretty pease?


file_browser_wdg.py line 125

comment should be : # replace // and \\ with /


<translation>Ebenen-Datenformat ändern</translation>


I'm not sure about that. The common idiom is to fallback to *everything* via multigraph when enhancing main plugins.

That said, I admit I didn't really understand what you meant :-)


dropped 100KHASH on the 980, but fixing it


Isn't this substitution wrong?


Fair point. I'll refrain from immediate backporting.


Ah cool - thanks!


d'oh thanks


Also this one is either not correct or not consistent with the title of the patch.


To not duplicate the implementation of `carp_instructions_binop(...)`. Take a look at it, it should clarify the reasoning.


backported this in 2d894e5d20d0df6c420abf3aa89b1bd22b6b25d4


Ok not so good: test my blog :D
ruby wpscan.rb https://www.firefart.at

`[+] WordPress version 1.2.1 identified from stylesheets numbers``

It detects the jquery version as wordpress version because i disabled all version leakages


Indentation is inconsistent.


backported in 2d894e5d20d0df6c420abf3aa89b1bd22b6b25d4 - just in time for 0.3.5, we were about to tag


There's also another place this should be done, but this is good. Thanks, and sorry for the lag!


Naming is inconsistent, should be on_exit.


Here the headline is tested if it is 64.


Here we test if the slugline is chopped to 24


This is the set up for the test: Here slugline and headline are longer than 24 and 64 respectively...


Why is this any better than a macro? Also, isn't `inline` just a suggestion to the compiler?


отступы


This is ok, please do a separate pull request so that I can merge it. Thanks!


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


Ok, I got on nerve but I finished by doing a make install and I'm back on track... I guess I have no choice but to use a standard path...
Sorry, for the fireburn. Modules suchas this one are overly complicated and are the kind of things I'm usually fighting against. :)


This looks ok. Please use the same indentation as the rest of the code.


Please elaborate on the typo on line 481 of dication_format.py?


Please make a separate pull request so that I can merge.


Done.


ResultsSchema (sanyalj) - 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)


It's better because it offers compile-time checking that the operation really is a binary operation (through the number of arguments to the function). Also, yes, inline is mostly ignored in gcc (I don't know about clang) but it declares well the intent of not introducing function call overhead here. Your macro was great for simplicity but I personally like to avoid macros if possible due to problems they *may* introduce (I used to love macros though, not sure what happened since).


Please squash this with the previous patch.


You need to update the places this function is called too...


There's a comma at the end which is invalid json. Caused an error, but I've fixed it :)


all tests pass:
success: http://build.servo.org/builders/linux1/builds/483
success: http://build.servo.org/builders/linux2/builds/481
success: http://build.servo.org/builders/android/builds/519
success: http://build.servo.org/builders/mac1/builds/700
success: http://build.servo.org/builders/mac2/builds/700


Ah, ok


spot on!


Mispelled "canola" !


This commit deletes the 2 files in `dist/` and the intermediate file `src/gl/shader_sources.js`. Please re-make and restore these.


有点奇怪，为什么宏定义在防止重复定义中无效呢？？难道只能建立.c???


fast-forwarding master to auto = 4cd9eb12


Personally I preferred the normal buttons at the bottom instead of icons in each row.
Now the treeview looks cluttered. Other opinions?
Thanks for fixing the bug though. :+1:


Great job, thanks!


ohhh just the versioning broke?


Minor, maybe we should make an rpm macro for the imagefactory version dependency so we can change it in just one place.  Regardless, :+1:


Whoops just realized the import is in the wrong place. Updating!


Having the default policy set to 'on-failure' would be preferred to support standard container stop/start workflows around an deploy.


I upload 2.3.12 to PyPi with this commit.


done


Fixed


I get the same behavior out of @collinss applet with or without this patch (sometimes it works, until it doesn't, and you can't break focus without selecting a different cinnamon element)

Your applet is definitely broken though with this patch.


saw approval from larsbergstrom
at https://github.com/glennw/servo/commit/09d97bbf408b8ee4eada728e81573bdf72816b5c


merging glennw/servo/cef-resources = 09d97bbf into auto


glennw/servo/cef-resources = 09d97bbf merged ok, testing candidate = 1699864f


And by Abstrated, I mean Abstracted!


why?


This should be addressed in the latest version: https://github.com/ros-infrastructure/ros_buildfarm/commit/cce744c318689efa89701f81555068b0cb66ebf6#diff-dd8353c286b898eaa8419cfc9966de8cR3


Could you extract the Key somewhere. Or better just MultipartKeyBuilder, that returns byte[] in build(). Thoughts


Thx works now as expected :)
`[i] WordPress version can not be detected`


velocity != velocityZ


By non-deterministic, do you mean incorrect?


:bomb:


rockin' :guitar:


I discovered the problem with this check also while updating my big library (I do tests only on a small one because of time)

Could you change this instead to
if ($genres === false) {

Actually it looks like there is an error in the PHP documentation :-(


Against your blog I get the 4.1 from fingerprinting ^^


isn't PATH Windows only - maybe a more generalized error message might be better?


k


does it make sense that we concatenate all errors, while we reset result.docs for each batch?


Two identical lines?


It does yeah, but it just looks ugly in strace :smile:


moottoreiden testausta


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1810](http://teamcity.monogame.net/viewLog.html?buildId=4246&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 293, ignored: 6 Build time: 00:16:28


Maybe do a `s/Node.js/io.js/g` in this file?


Yes same for me :)
Are you sure preg_split will return false? I cannot found any information about that on PHP documentation or related web site.


`grep -v -c "^$" project.list` should do the same.


Very nice!


:+1: Great addition to the docs!


not Windows-specific, *nix also have the same concept.


Sure, I don't see why not.


good catch, thanks !


rax retry tests


Bunch of tags:
![screen shot 2015-01-08 at 4 32 52 pm](https://cloud.githubusercontent.com/assets/1305122/5672281/caccacd0-9754-11e4-9a72-bca6c68f73b8.png)


HAH. No this has to remain the same, unchanged. It's intentional. ;)


maybe repositories here too?


теперь в Избранном помечаются удаленные и архивные треды.
Надо для остальных борд добавить селекторы архивных тредов.


Small w?


Yes. works also.


```
regex1 = /stable tag:\s*(.+)/i
regex2 = /tested up to:\s*(.+)/i
regex3 = /requires at least:\s*(.+)/i
```


cc @dengemann I think this is the last of the old-version changes, should fix the recent Jenkins errors


@codeinthehole I changed the link in f23ab84bc. @jlmadurga thanks for spotting this :)

@jlmadurga checkout/layout.html has been fixed to extend `layout.html` instead of `base.html` in 7234c1e6d :)


I do not quite understand the need for `toSeq` here


Sorry! My bad! Didn't know Python's unicode string syntax...


Are you sure you set the **environment variable** (NOT CMake variable) right? The script is looking in CEGUI_DIR, CEGUI_HOME and CEGUI_ROOT the same way it's looking in /usr/.


extra white space


Esa imagen tan fea con windows.


Sorry, I missed that discussion.

Examples of editing `creature_model_info` for creatures having more than 2 models (like 2 different model, each declined in male/female model):
https://github.com/classicdb/database/commit/1baa79b82558a45e11f7097c2c048435dcaba89e
https://github.com/classicdb/database/commit/c7717c02ba7c1a7ce95a23804c060641aaeb0e1c
https://github.com/classicdb/database/commit/9298eb61cf046744057c653465d84d7e41ae75fe
https://github.com/classicdb/database/commit/6d5916fe1bed15849e0ca7d97e8797c40c587211


Yes I tested it in a PHP shell, it is the same behavior like documented in preg_match:
php > var_dump(preg_split('/|', 'asbsdfbasd'));
bool(false)

After a second reading it looks like a misinterpreted a tip: If matching fails, an array with a single element containing the input string will be returned.


Also, for the readme detection, I would just scan the source of the page ;)


I think it would be better to have a method on AbstractParsedStmt.  The reason is, I don't think that DDLCompiler currently has any dependencies on anything in the parseinfo package.  It would be good to avoid creating another inter-package dependency if we can help it.

If you're concerned about performance maybe combine the two checks (checking for subqueries and checking for just one source table) into one.  Then all the AbstractParsedStmt method would have to do is check m_joinTree for anything other than an instance of TableLeafNode, yes?  The error message might need to get a little less specific, but I think that's ok.

Alternatively, you *could* make the make the AbstractParsedStmt method traverse the join tree.  I wouldn't expect that we ever do enough joins to that this would be costly?

You can just add another commit to your branch, push it up to GitHub, and the pull request should get updated automatically.


And also its updating song nr. 3661 with this fix :-)
Before it crashed on nr. 12.


cool :+1:


I think the `|| []` is redundant


Looks good.  Will merge and test on midas-dev


Using _.findWhere might be nicer, and then if none found, `ret = messages[0].content`


@chrisseaton This looks a bit ugly.  If you have thoughts on how to make it cleaner, please feel free to share.


the standard name is hasDualClass()


Hm ... derp !


Alright, the issue still comes for when moderators spoiler images:
Post an image and then spoil it as a mod/admin and get the results
**With Spoilers off**
* Image doesn't get spoiled and toggling it back and forth doesn't change anything

**With Spoilers on**
* Image gets initially spoiled, but toggling spoilers unspoils the image

For both, refreshing "fixes" the issue.


I'll remove it.


nix


cbecc-materials-bugs (macumber) - 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.26873857404021%25-yellow.svg)


Doesn't look like there was any review, but this is definitely problematic until we have support for 64 bit ints in labrad data itself


Hi,

I just found this because I noticed that my `grep` alias no longer works. Is there a way to accomplish your goal without interfering with the command/alias `grep` ? I can get around this by moving my alias below `source $ZSH/oh-my-zsh.sh`, but that's messy. Why is this solution better than the previous one (aside from ceasing to use the depreceated `GREP_COLOR`)?


폰


our scripts in asset branch are gplv3 for standalone startup, although we should discuss further the licensing of those as for myself i would best see the "moddable content" with a more free license.


Can we get the `=` aligned?


Indeed after testing you're right. preg_split PHP documentation requires a serious update ;)
I will comment the change.


Thanks. I'll apply it when I get a chance. Can I ask you for one more favour?
I just noticed that MSG_047 `"Multiple Partitions"` has never been translated. Can I please ask you to translate it as well?


I have Spigot 1.8-R0.1


Yes, It was pulled out in a subsequent patch.


https://github.com/cleanflight/cleanflight/issues/70 ;)

-=dave


so this will always give English "en" message? should that be language instead as that is the user preferred language?


Approved :+1:


Yeah, that'll be the day...


64-bit ints came up in another context and @ejeffrey did something for a temporary workaround--I think it was probably just raising a more descriptive exception. Evan, do you remember where this was?


log.c includes pilight.h
pilight.h includes devices.h before maxpulsestreamlength is defined
devices.h includes protocol.h
protocol.h does not include pilight.h again, therefore the MAXPULSESTREAMLENGTH definition is not available.
I think MAXPULSESTREAMLENGTH  needs to be moved up to L21

You need to define MAXPULSESTREAMLENGTH before devices.h is included, as devices.h includes protocol.h


What about this https://github.com/laravel/laravel/tree/develop/app/Http/Controllers?


Yeah that would be more efficient... ok.


@JeromeMartinez: are you sure about `MEDIAINFO_TEXT_YES`? Does it get disabled by another define we use? Also, can you confirm we need all the other defines?

Thanks!


You have a commit that says "Don't assert on Android" for common/os_posix.cpp yet do so here. Don't know if it's a problem, just  pointing it out.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/484
success: http://build.servo.org/builders/linux2/builds/482
success: http://build.servo.org/builders/android/builds/520
success: http://build.servo.org/builders/mac1/builds/701
success: http://build.servo.org/builders/mac2/builds/701


Hm. That looks pretty easy. I'll add it to my weekend TODO list. :)


I'll see what's up tomorrow.


heer -> here


all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3076
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3069
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3070
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3061
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3061
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3064
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3057
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3052
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2729
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2727
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1565
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1557


fast-forwarding master to auto = 6354d60e


haha, okay - changed to 8G [is that more reasonable?]


> Are you sure you set the environment variable (NOT CMake variable) right? The script is looking in CEGUI_DIR, CEGUI_HOME and CEGUI_ROOT the same way it's looking in /usr/.

Yep, I was doing:
export CEGUI_DIR=/home/me/cegui-0.8.3
cmake .

I dropped the cache and the CMakeFiles/ folder each time and and tried with all DIR, HOME and ROOT.
And still no luck finding CEGUI version. I supposed it is meant to be used with an installed version only?


done.


These unsafe casts should not be necessary. Can you remove them and check whether there is any measurable performance impact? CompilerDirectives.unsafeCast will become unsupported in the future and should be avoided.


fast-forwarding master to auto = 1699864f


Bleh! ;)

List<Object> messages = Arrays.asList(editTextToSend.getText().toString(), 1, 1L, 1.0f, 1.0, (short) 1);
Object random = messages.get(rand.nextInt(messages.length));
EventBus.getDefault().post(random, MainMobileActivity.this);


@mtwebster Yeah, that would certainly be helpful, though if you could figure out a way to make the entry widget work in Normal mode, that would probably solve the problem as well.


No objections here, always learning new words everyday. :)


terrainState is always constrained to be no less than 0.1m below the flight vehicle after every state update.


Ditto of a prev comment:
> You have a commit that says "Don't assert on Android" for common/os_posix.cpp yet do so here. Don't know if it's a problem, just pointing it out


Done in 186736e


saw approval from jdm
at https://github.com/mttr/servo/commit/020a767849e9979649909a1c7c3052524fd7bae2


merging mttr/servo/warnings = 020a7678 into auto


mttr/servo/warnings = 020a7678 merged ok, testing candidate = 07931376


Bleh! ;-)

messages.size() not messages.length ;-)


Confirmed working! Good to see heavy arty finally tracked!  We just need it to erase after 5 min of showing on the map.


@saadel hhhh


@JoshuaBrockschmidt - after this quick bandaid patch, everything looking good to merge into master?


Already updated. I had `Object[]` before ;)


That `.max().min()` trick is neat.


Hi , I don't know why you create a buildLayout(LoggerContext ) method.
The buildLayout(context, TimeZone.getDefault()) in my code is to call AbstractAppenderFactory's buildLayout :
https://github.com/dropwizard/dropwizard/blob/master/dropwizard-logging/src/main/java/io/dropwizard/logging/AbstractAppenderFactory.java

not create a custom layout.

It seems you didn't 'register' the layout to Dropwizard. I think the  code may not work.


I'm sleeping a bit here. I also did that commit locally already but again forgot to push it...


Consider overriding the `requireCamelCaseOrUpperCaseIdentifiers` in your .jscsrc to the value: `ignoreProperties` to avoid this disable.


So picky!!! of course.


That's a lot of type parameters. Could the documentation perhaps describe what they are?

Maybe after the rustup we can associated-type-ify them.


@updateing this needs any sepolicy? for work


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


For some reason I'm actually not getting the Python tracebacks after each routing attempt, just sporadically. And even then the traceback looked to happen in the turn-by-turn navigation code that should trigger once a route is found. So at least from the outside it looks the routing is indeed being triggered and that we are maybe getting some results back. :) I'll hopefully have some time to take a closer look what actually happens "under the hood" during the weekend. :)


Still true for forget to remove comment?


It cannot clarify which problem is db selection, as @marcosnils said.
Could we change JedisException to JedisDataException and parse message to check it is error from selecting DB?


refs #613


Why are you re-doing all the braces?


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1094 of 1094 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)


:D

On Thu, Jan 8, 2015 at 5:06 PM, Chris Klosowski <notifications@github.com>
wrote:

> So picky!!! of course.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/easydigitaldownloads/Easy-Digital-Downloads/commit/0d47e283762abccbdcaeb02818bb4e6290a3fee2#commitcomment-9210937>
> .
>


It means that the destructor closes over a lifetime-bounded type. This is basically a workaround for rust-lang/rust#8861.


To simplify even further, instead of checking for `@name_generator` on each `name` call, you could simply check it during initialization and if it's nil, and if it is set it to `generate_characters + generate_numbers`, that way could simply call `@name_generator` in `name`


Why is this list maintained in this repo, by the way? Especially since it's ghc-specific, can't we just put overrides in the `configuration` files in nixpkgs?


I would prefer to reverse this as well


Hi! I've tried to use your package via *bpwer install jquery-locationpicker*, but it didn't worked. Can you please suggest right way to use this package?


Can you use `bitflags!` for this?


Thanks, I had meetings and couldn't get to this quickly. Was this released as 0.3.4 or anything? There's a julia-users thread about some puzzling julia/numpy linalg comparisons.


I think you meant awesome. :+1:


Fixed and pushed to remote


I would prefer to reverse this as well. Similar below.


pDispparams->rgvarg[0].vt != VT_I4 is less Yoda, IMHO


Well, whatever the sweet spot may be, you want it fast, but not look washed
out.  High quality is a must or its just any other ROM....

On Jan 8, 2015 12:29 PM, "Joe Maples" <notifications@github.com> wrote:
>
> I've used it for quite a few builds and I personally haven't noticed a
difference except when I placed them side by side, but that could also be
the fact that my GS3 is 720p and my OnePlus One is 1080. While modern
phones have enough storage, processing power, and ram to load apps quite
quickly, 70% compression makes it load much much faster.
>
> —
> Reply to this email directly or view it on GitHub.


Gotcha~
In the end, major improvement, thanks for the hard work!


Yay!


rax retry tests


I think I'd prefer just using the `_` prefix to silence the warning instead of `let _ = ...`


r+, thanks!


Can you use the `while let Some(...)` trick here?


I thought we said to not expose KeyBuilder... Hm... This looks not good here. Let's put these methods into your class for now:

``` java

byte[] getMultipartKey(String parts...) {
   byte[] result = new byte[Bytes.SIZE_OF_INT * parts.length + lengthOf(parts)];
   // fill in all parts into result
  return result;
}

```


This would look better reversed too


Can you add a comment describing why we use setTimeout instead of transitionend?


Amlogic-based projects don't use irserver, but maybe for hassle-free reuse of backups on other devices, it is worth to rename the config file.


수정하였습니다.


Hi @AndydeCleyre, please look at #3341 and #3347 to see discussion in regards to using alias or function. In your case, my initially proposed approach (PR #3347) would suit you better.


This solves #12


Has this been removed accidentally? (because you've mentioned it in https://github.com/yourlabs/django-autocomplete-light/pull/341#issuecomment-69263320)


Can we also delete the corresponding non-webkit keyframes?:

```css
 @keyframes WinJS-selectionMode-shiftRight     { from { transform: none; } to { transform: translateX(40px); } }
 @keyframes WinJS-selectionMode-shiftRight-rtl { from { transform: none; } to { transform: translateX(-40px); } }
 @keyframes WinJS-selectionMode-shiftLeft      { from { transform: translateX(40px); } to { transform: none; } }
 @keyframes WinJS-selectionMode-shiftLeft-rtl  { from { transform: translateX(40px); } to { transform: none; } }
```


84205966-DefaultHVACConstruct (kbenne) - i386-Windows-7-VisualStudio-12: OK (2168 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.08592321755027%25-yellow.svg)