Good catch! This could be quite a deep design issue for permission policies, and by extension viewsets: for swappable models like Image, any class that needs to know the model at initialisation time, cannot itself be constructed during app startup. The immediate fix would be to write a `get_permission_policy` function, but that's really just passing on the burden of dealing with lazy evaluation to the next layer - when we come to port wagtailimages to viewsets, we'll have the same problem (and won't be able to postpone it any further by writing a `get_viewset` function, because we have to register urlconfs at startup). Also, the `get_permission_policy` route might not play well with the ability to override permission policy in settings - we probably need to decide (at least partially) how that mechanism is going to work.

In other words - *something* in the stack will need to support something like `model_callback=get_image_model` as a parameter to its constructor / class definition, in place of `model=Image`. I think viewsets (and other generic-CBV mechanisms) will have to do this in any case - which probably means that permission policies *don't* have to (the viewset would lazily construct the permission policy instead). That still doesn't answer the question of how to override the permission policy in configuration settings, though - a naive approach like

    WAGTAILIMAGES_PERMISSION_POLICY = BlanketPermissionPolicy(get_image_model())

would fail with this same error. And we can't really avoid it with

    WAGTAILIMAGES_PERMISSION_POLICY = BlanketPermissionPolicy

because different permission policy classes have different constructors. I guess this is where the 'backend' pattern (a la Wagtailsearch) comes in - it seems awfully over-engineered for this task though.


Yeah, one-liner looks more clever, until you have to quickly understand what it does later on.
Possibly related: https://twitter.com/SaraSoueidan/status/686596710819315712 (the thread, not just one tweet).


Now I realize I named that method very badly. It was supposed to be `.get_seed_nodes_private_ips`.

I will rename it and create get_seed_nodes()


Here and in the following fields: add `final`


You can skip this annotation


Why initializing here?


this change doesn't hurt IMO, and makes the implementation easier (no need to find an `IParameterSymbol`'s identifier node)


yap, i'm working on the predictor now, this one is just some stuff I did this morning, thought it may help with the statistic view :)


Probably go with “maybe not” ;)


[Uploading 虚拟桌面.docx…](
[Uploading 虚拟桌面论文.pdf…]()
)


With these being so long, would it be more clear to break up the lines and use block style formatting over flow style? For example:

```yaml
- role: elasticsearch
  es_instance_name: node1
  es_heap_size: 1g
  es_config:
    network.host: _eth0_
    discovery.zen.ping.unicast.hosts: ansible-test:9300
    http.port: 9200
    transport.tcp.port: 9300
    node.data: false
    node.master: true
    bootstrap.mlockall: true
```


I'll add these docstring also something to comment:
```
""" Initialize the Page class which will contain selectors.
    From this base Page class all other *Page classes are inherited.
    All other Pages using their own graphical items and that is why the `init_gtk` parameter is needed.

    :param bool init_gtk: enable/disable initialization of graphical
                          components here.
```

Basically problem here is `UnknownPage` and `CreateNewPage` duplicated the `Page` variables because they didn't called `Page` constructor... I think that is really bad idea and I didn't want to add `selected_members` to all children there.


It's fine. Whatever works for you.

I don't find it intuitive to read, but that could have just been because your functions aren't named meaningful names for me (I wasn't sure what they did until I read *all* of the code for them).

It's nice when code is self-documenting.


win7之后系统开始支持在cmd中使用斜杠作为路径分隔符，可以考虑去掉这段代码，尽可能的保持代码精简。


`removeParent` as it currently stands is a bit cumbersome.  Since there is no getter for parents, the only way to know the contents of any parent is to look at `doc.parent.regions` (or whatever placetype), so removing by index would be far more useful, for example:

`doc.removeParent(0)`

Would remove the first name, id, and abbreviation.  This would eliminate the need for callers to iterate the entire list accumulating the required parameters for the current `removeParent`.  This change would be made in concert with my comments on `addParent` above.


General comments:

1. Create a new branch on this repo: https://github.com/blunderboy/stackexchange-sdk
2. Make your commits on this branch (It will be your feature branch)
3. Once the review is done, we merge the feature branch to master branch.


'url': 'http://198.47.104.134/m1.music.126.net/%s/%s.%s' %


Variable name bad could be more descriptive, and having a capital on the A and B of userResponcea and  userResponceb would make it more easily read


@TomokoKuroki It's because you're not casting to a valid address, the program doesn't understand you're writing to a virtual address.

Change,
```c++
(INT)((INT)+0x7A97F0) + 0x55A8) = 308406
```

To,
```c++
*(INT*)(*(DWORD*)0x7A97F0 + 0x55A8) = 308406
```

To make your life easier, use the [this](http://pastebin.com/5qbXeb8q) memory manager. Ripped from my Maple trainer which can be found [here](http://www.gamekiller.net/threads/gms-v155-derp-trainer-source.3232734/).


Should this field be final?


No, we won't be using jQuery anymore. Lets use native xhr within a method.


return here


I think, in general, it would be nice to have the meat of this code in a class or classes in the lib directory (with corresponding tests) that gets called from the rake task.


Domain should not be forced when initializing the app. Consider StackEye - works with multiple domains.


SpongePowered/Mixin#93 is now fixed so you can add this back in if you want @Aaron1011


std::vector<std::vector<QPointF>> use in polypack2d.h
I do not want to use it on the QT dependence
in the future, take away QPoint
When I finish making a global refactor


Does Lwt really affect whether the **runtime** "records" backtraces? It seems like this should be rephrased, because I think these flags affect whether backtraces are simulated by Lwt across `bind`, etc. (by inserting `backtrace_bind` instead). This reads as if linking Lwt completely disables backtraces.

It may also be a good idea to note somewhere that the regular `bind` doesn't propagate (splice? compose?) backtraces (AFAIK), and since `backtrace_bind` isn't documented, this makes using the syntax extension important. One place where this should be noted is the documentation of `Lwt.async_exception_hook`, which refers to printing backtraces – which won't be "correct" if writing code with `>>=` and not using a syntax extension, and thus it will be surprising and disappointing. Not to mention difficult to fix, since that is the only mention of the word "backtrace" in the docs for module Lwt.

I also think the section on camlp4 immediately before this section "Backtrace support" should be exchanged for a section on ppx, or at least a section on ppx should be added.


need to handle error (null pointer) returned by curl_slist_append.


I don't have a problem with the functionality, I have a problem with the way it's written, the names that are used, and the fact that you're passing stuff around in a 3-tuple format which has nothing to do with your application. You've tied your pieces together VERY tightly.

You're using data structures to pass your data, rather than functions. This is brittle.

We try to keep things as abstract and separate as possible to maximize code reuse and to make code clean and highly understandable.


Not at all. Since I don't plan on making more changes to this, I'll delete my fork once you merge this in.


Well, the MessageIdComputer (I do not like the name much either, but I have not found a better one, generator could mean that it is a new one each time, while it is not, it is computed based on the message and stay stable) is not thread safe, therefore it could not be a static, and it is used only in the constructor here, so no purpose to keep it for later.


Вобще любые попытки прикапывания модели в onSaveInstanceState ничем хорошим у меня не заканчивались. Оно там полгода пролежит и юзер ее запустит. Уже и нипроекта нету того давно на бекенде, а из instantState оно ее достанет и отрисуеть :(:(:(


Generally, I don't think it's a good idea to repeat the purpose of a class in the JavaDocs of another.  If the behavior of the dependent class were to change, someone would have to know that this JavaDoc needs to change as well.  In essence, for the same reasons that we don't duplicate code, we shouldn't duplicate documentation.  If we decide to consolidate the strategies from many to one, then this class will be compile-time broken and we'll have to update both this and the documentation for it.

I'll add an @link or @see for the dependent classes.


What is this good for?
Preparing the shape in this way means that no optimizations are going to be applied to it.
Initializing will nil means that it probably will not specialize to primitives. And it means that an incompatible assignment will probably cause a shape transition. And most likely, fields initialized with nil will also never be considered write-only-once field and then benefit from subsequent optimizations.


Eamon, when you committed a new AvsAn-simple.min.js.gz, you didn't update AvsAn-simple.min.js
as well. Are they the same, or was that an oversight? I'm embedding the js directly in a script, but the file from the gz appears to have an encoding problem, as I'm seeing `-` displayed as `â€“`, so I'm using the older js file.


i usually just use common sense when deciding whether something can be a one liner or not.


Let us store all predefined query params separately.

Loop over all predefined query params and if it exists in queryParams object then store the 'key=value' pair in an array.
Finally, merge the array by using join operator.
Last, use queryParam string after appending '?' to the url.

Your code will add '?' multiple times and thus making the URL invalid.


Is this needed? We usually do not set the default value.


LOL. How do you REMEMBER all of that, and would you remember it all after not having touched it for 2 months? If not, it's bad.


Replace SW with SE where SE stands for StackExchange.


Should only be done if no alg is specified.


Right, but then it is `$I->` instead of `$this->`, which I prefer. I don't like God objects ;)


shouldn't be there a `checkIsLatest()` method that does exactly the `if` and `then` of what you got here?


haha


Is the string field needed? Since the field corresponds to the actual token's name you could use `name()`:
```java
DnsRecordType type = ...;
type.name();
```


Since this is a private method and not supposed to be exposed as API. Let us constrain these methods like this:

    SE._internal.getUrlForAllUserDetails

If a method is a part of SDK, it should be exposed directly.


Thanks. I have integrated your changes. See pbatard/libwdi@043e7588f5e2632d276ebb28ef099f4bbf83305d.


Also here, is the default value needed?


We could probably get rid of these comments


It's just used for the interface declaration.


Cool commit :+1:


bei mehreren Autoren nach RDA die Zählung in Pica-Kategorie 3000 und 3010... statt 3000, 3001...


Does this mean patterns with spaces in them won't work?


It would means that whole API need to be change to manipulate a Wrapper of the MimeMessage class instead of a simple MimeMessage, and it should not be an override of the getMessageID(), since this is a different purpose. My proposal here is mainly inline with keeping the API as stable as possible. Only the return values, and argument values of some method has change a little, and this should not impact normal usage of the API.


True :D


Please don't put magic constants into code - move to a DEFINE, and since these (potentially) vary by client, put the defines into include files that are client-specific.


Module 6


It is wrong to assume response.items is an array. We should wrap this inside try-catch.


Would be nice to have capitalisation of SQL operators in the SQL statements


Vladimir -

Thank for the book suggestion I ordered it today.

On Sun, Jan 17, 2016 at 12:16 PM, Vladimir Romanov <notifications@github.com
> wrote:

> The book I used to learn C/C++ is C++ How to Program (5th Edition) by
> Deitel & Deitel. You can find it on Amazon (http://amzn.to/1ZF11yJ). It
> will walk you through each concept step by step.
>
> As far as sensors go, look for capacitive sensors; those are typically
> used in industrial environments as they don't corrode. I would also browse
> through the digi-key catalog to see what they offer.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/VRomanov89/Arduino-Soil-HumidityProject/commit/a59ad1c6dc6074300fe5dcbc7339924e97d5dad6#commitcomment-15496473>
> .
>


Great! :-)


not used


You can even pass the Joomla version, which defaults to 'Joomla3'.


Nit: no newline after `<p>`. I also think this paragraph could be avoided. Also consider using javadoc's `@see` to reference the docs.


I was thinking this was more towards people that didn't understand the point of noproxy. But you'res might work as well.


@tomokinakamaru Why did you remove the prefix matching support? This was a big appeal for this data structure.


Это чего это вдруг? Откуда все эти изменения в web.config?


I see.


log4j not used


Likewise, does this work for display names with spaces?


Require


This can be a little difficult to interpret if the reader doesn't quickly realize it means that `>>` is piece of syntax added by the preprocessor instead of a function. I think it should just say that. Something along the lines of `Note that [>>] is not an OCaml value, it is a piece of syntax added by the ppx rewriter – i.e., you cannot refer to [(>>)].`


What's going on here? I'd recommend
a) using FALSE rather than 0 in general,
b) commenting out code you don't want to execute, and
c) moving magic constant 72 into a server specific include


:cactus:


Настроить e-mail с помощью события?


You :heart:


This change again introduces the use of DualLocation. It would be much better just to rely on TypedLocation here. Otherwise, this is not compatible with GraalVM.


/me sees @Minecrell shudder.


There's a lot of repetition in the following clauses. Would be nice to start with a data structure of states and colours, and iterate through that to reduce the duplication.


Почему consume? Mailer же отправляет письма


Consider using Guava's [checkArgument](http://docs.guava-libraries.googlecode.com/git/javadoc/com/google/common/base/Preconditions.html#checkArgument(boolean,%20java.lang.Object))


Could we define the colours as appropriately-named constants to make the code clearer?


You aren't following code style convention here, which is to put else on previous line.


Можно добавить аннотацию с @IntDef, раз тут не диапазон, а перечисление.


i don't think we use these last 2... this is old


nice line!


`'~> 2.6.2'`?


it is better to use expression pk is None or sk is None.
this helps to avoid of creating temporary tuple and uses operator 'is' instead of operator 'equal'


Yahoooooooooo :D


Это что за херь??? Во первых на моей трубке такого пути нет. Во вторых он есть у соей жены но ты туда ничего просто так не запишешь без приседаний. http://forums.androidcentral.com/verizon-samsung-galaxy-s4/384648-kitkat-issue-can-t-write-external-sd-card.html


you can use @Test(expect = XXXException.class)


Please ditto


What's going on with :::: ? Seems like an inelegant work-around.


Nit: we usually write todos as: `todo(@mziccard)` so that we know who to blame:)


Please ditto.


This process of cumulative exclusion is odd enough that I think it might be worth adding a comment to explain what's going on - ie. that shorter and shorter bars are being overlaid on each other.


It appears as entry point in API here https://github.com/PowerlineApp/powerline-server/blob/develop/backend/src/Civix/ApiBundle/Controller/SecureController.php#L411


Never worked on shared projected before. Do I just do the code and post it here?


One more question before I merge to master.  The ReactiveKafkaConsumer constructor (in master, before I began my work) obtained the topic from *properties.topic*.  Should I continue to proceed with that pattern or should I make topic its own parameter.  I don't want to clutter the API but at the same time I want the correct parameter combinations.


lol!


Is this something that is implemented for other sites too? If so, putting civicrm_value_visit_day_support_11 into include would be good.


The IL for these won't be ever inlined today. The JIT helpers are treated as special functions with no IL by RyuJIT. It is artifact of how the CoreCLR runtime is implemented - all JIT helpers are in assembly or C++, so it was reasonable to treat them that way.

Inline expansion of JIT helpers is done in RyuJIT by manually building the right IR. The JIT often exploits additional low level tricks to do so e.g. the overflow checks could be done using CPU flags in some cases if the inline expansion of these conversion is ever implemented.


Migrated as method from API https://github.com/PowerlineApp/powerline-server/blob/develop/backend/src/Civix/ApiBundle/Controller/SecureController.php#L456


This is the request graph x axis label I think when it should be the user one.


Could we make 22 an included DEFINE


The `ellipse()` function has a hacky workaround; the code should be fixed to come up with a reasonable number of segments.

You should also remove the implementations in `processingDemo.pde`.


Oke,

I used tuples because otherwise I could not make this work.
Text.PrettyPrint.Boxes works only on lists. So one way or the other I had to put the things I want to display in a list.

We could look how to make it better.
I found this code somewhere on the net.


Тут будет AOOBE, если dayIndex == 7


@frodrigo Ok, je viens de faire la modification pour avoir la meme function sur les deux classes css.


I think the 'set xlabel' bit at the beginning here is a typo.


22 should be included DEFINE


Используя не-support фрагменты у тебя постоянно что-то будет отваливаться на трубках с относительно старыми API level


*sad face*


This is a premature commit , please ignore it.


Should civicrm_value_visit_day_support_11 be in an included DEFINE?


А как аннотация относится к массиву? Типа каждый элемент должен быть от 1 до 7? Это чекается средой?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1191](http://teamcity.monogame.net/viewLog.html?buildId=13739&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 560, ignored: 6 Build time: 00:29:03


This was in some other commits too - should the 1 be in an included DEFINE? And should ?? it be TRUE rather than 1?


@frodrigo https://github.com/ngonzalez/mapotempo-web/compare/dev...ngonzalez:tomtom-update-clear-messages


Class names are written in UpperCamelCase.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11850](https://teamcity.mesosphere.io/viewLog.html?buildId=107137&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:08:38


Does the following clear it up?

the client MAY require additional validity requirements on the SCT feedback connection, such as an unrevoked certificate or accompaniment by a valid SCT.


Good question. Personally I'd prefer the API to remain unchanged and put the topic in properties, unless you see a good reason to move it to the consumer constructor?


68 should be included DEFINE, and we should comment out code to disable it rather than use 0 or even FALSE &&


@douglaslondrina .... I am using cocoapods version `0.38.2` and `0.38.0` (in two separate projects).


@bgoodri This commit and the previous add this correction in the model block and generated quantities blocks for continuous, count, binomial, and bernoulli. Do you know if lm.stan and polr.stan need a similar correction or do those Stan files avoid this issue?


Constant names use CONSTANT_CASE: all uppercase letters


Un test, ne pas accepter


Неа, тут индексами служат Calendar.*, там от 1 до 7 значения.


I proposed an alternative solution in #627. @justinedelson Can you comment on that one?


s/running//
s/teared/torn/


it's OneToOne


Угу, если явно другие значения задать - подстветит красненьким.


I'm not sure what you mean "regardless of age".  I still have to define a sample algorithm for sending the SCTs in general, which will take into account 'age' of the SCTs...

I don't know if it's well defined, probably not.


5420-OSCM-Docs (mitchute) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Actually christ yeah, why are there so many near the surface? Maybe then should only gen below a certain level, that could sort it.


To be clear, deleting this gets rid of a the build folder for good, as that's not supposed to be checked in at all.


I will leave it in properties.  If we want to allow specifying multiple topics in properties it could be done in a comma-delimited string, similar to the zookeeper hosts or bootstrap server parameters.


private static final?


The way I understand is:
Some creatures were added, not changed in any "harmful" way. So it's safe to upgrade a 42.04 game with these raws, but a 42.05 save will not work with 42.04 - which is a moot point I think.
The situation was different with 03-›04, because creatures were moved around AND renamed (eg: tigerman -› tiger_man) instantly breaking any saves.
I'll change min_version to 42.04, tell me if you disagree.


`void throwpe();` is not needed, because you rejected the third test.


>What do you mean? This attack is about servers filling up client storage, is it not?

You're right, so we should leave it as MUST NOT (or a SHOULD NOT with a note about the attack), but in general I need to overhaul any references to flushing attacks once I've described the attack and mitigation.

Going back to your first comment here, I guess I'm not sure what you mean.


I'm curious which tests was failing.

This shared order_purchased code gets run for all Stripe, moneris, paypall, cheque, etc.

The `current_cart` will not be what you think it is in the case of paypal, because the purchase is not run under the `current_user`.  (It's run under a postback from paypal).  Which is why this was (recently) changed to call `Cart.destroy_all`


Это какой-то ад. Фрагмент не должен ничего удалять/добавлять в собственный фрагментменеджер


I see. Still it's a strange choice for a public function in a CSS world, but I guess it's unrelated to this PR, and better leave it as it is.


Тут точно двойной отступ сверху?


how do you play this sick blob on maths piece of paper


[Uploading ios-homework.zip…]()


You could use `.last` instead of `.pop` to avoid mutating the `payload`.


my guess is that it was done in this file by defining one of the two preprocessor definitions. And it's true that if the compiler do not support c++11, there will be a compilation error at `using Hash_Map = ...`.
We need to fix that (I guess getting back to `struct Hash_Map : std::map...`, as typedef cannot be used)


много переносов


Kann weg?


I'll check


Строка длинновата. До конца дочитать не осилил :-1:


Yeah i think that comes from a other branch i was working on


because "upload_files need to be a list of tuples of (fieldname, filename, filecontent, mimetype) or (fieldname, filename, filecontent) or (fieldname, filename)"


I really would like to keep the `GetLongPathName()` call first as it is probably faster than creating a full handle and then reverse-engineering the full path from said handle.

In other words, I really want `GetFinalPathNameByHandle()` to be the fallback, not `GetLongPathName()`.

However, for that to work, I would need to know what error `GetLastError()` returns when `GetLongPathName()` fails due to the lacking *List Directory* permission. Would you know that?


That French spacing is _not_ "blindingly replicated" nor a typo, as I do prefer it in IT programming (as many other does, just search for it) in order to achieve a more fluent reading (used in Italian too: it's just a uncommon notation).
But I admit it's just _my_ personal opinion, so I'm not going to quibble over it: I'll push a mainstream spacing version.


тут NPE будешь ловить


I want to mutate the payload as the next [line of code](https://github.com/alphagov/specialist-publisher/commit/9657c4a0d495dd8cfbb550701baa7467f81197b1#diff-da766db961b1ebe7e7e8de051dac4096R120) takes all the `change_history` without the most recent change note.


тут то же


011 Peters T - The Woman Who Married the Bear - Text.txt

Test est


=neon is == like neon-vfpv3 .. but gcc doesn't support that flag .. it's missing.
Cortex A9 only support up to vfpv3  .. so using here neon-vfpv4 is completely useless.

if we want to increase performance, we should try to build kernel and rom as much as possible in -mhard-float... or if mhard isn't possible then at least =softfp .. this is far better than just the default -msoft-float.  - This is what i'm trying all the time.


@gschoenberger:
It seems that it didn't affect users of the 840 EVO [until the EXT0DB6Q](https://bugs.launchpad.net/ubuntu/+source/fstrim/+bug/1449005) (which fixed the performance regressions when reading old data). It doesn't mean that queued TRIM was working before, I guess that older firmwares just didn't advertise the queued TRIM availability.

@Daytron, @brejoc:
That is a different bug, reported by [Algolia](https://blog.algolia.com/when-solid-state-drives-are-not-that-solid/) and [affecting systems with RAID 0 or RAID 10 configurations](http://techreport.com/news/28724/samsung-docs-detail-linux-trim-bug-and-fix).
It was fixed with a kernel [patch](https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=f3f5da624e0a891c34d8cd513c57f1d9b0c7dadc) developed by Samsung.


check


The 1.0 version of the library has been released. Might make sense to see if it works.


@grainier Shall we provide class level comments here..


AlertDialog исчезнет при повороте экрана. Думаешь это ОК?


Sehr gute Idee!


I don't understand why this code is removed


It's done, see: https://github.com/ssllabs/ssllabs-scan/issues/295
When you're parsing ClientHellos, don't forget to take out the original SNI information and replace it with the hostname that you're testing.


Do not check-in ~ files.


Oh, "Mach-O Runtime Architecture" also didn't work.
I can't find it on developer.apple.com, but found here: http://math-atlas.sourceforge.net/devel/assembly/MachORuntime.pdf


The same type but no the same file name.
yes need add Q and for jpeg


Do not make this change.


in


I think you even can use `parttition` instead of `stable_partition` if the relative order of elemnets doesn't matter


What exactly is this? I changed it without realizing that it's pointing at the `assets/js/` dir rather than /assets/theme/js/` so I'm guessing that regex pattern matches... `/assets/**/js/*.js`?? Am I missing something?

I ended up reverting mine until I could figure out what the regex pattern is actually for.


Remove this commit (have to commit) entirely. This is changing files that should not be checked in at all.


Есть activity_fragment_and_toolbar с таким же содержимым.


Этого быть не должно. Что-то неправильно


geschieht ... Punkt


Remove this commit entirely. Do not checkin additional .gitignore files. Edit the top .gitignore file.

Use wildcards instead of individual file ignores. This pattern is already handled.


@grainier Please provide class level comment heres and other classes as well..


`tm_tags_dedup()` does a similar thing, maybe it'd be worth reusing?


Нужно адаптировать к разным темам.


copy and pasta from juttle-client-library.


Die Tatsache, dass


MainActivity.TAG_WORKER - плохо. Не надо фрагмент гвоздями прибивать к какой-то активити


@Zidane how come? This should be a good thing!


@grainier Shall we add java doc comments for methods.


I'll look into how to mark files for he browser.


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


Тулбар тут будет кастомный.


А чего не в onCreate? Просто интересно :)


This is untested, but yea. Currently messing about with other ways of solving the same problem.


good catch that should be `"all"`


Do not make these changes.


Gotcha. Thanks for the quick reply!


ooops!


В любом случае RelativeLayout стоит заменить на LinearLayout.


@grainier please add java doc comments to methods here and other classes as well


ooops!!


这块代码比较长，有没有可能精简一下？比如，在类Unix系统下直接调用open进行打开？

现在手上没有桌面版的ubuntu，没法验证这个想法，摊手。

另外一点就是文件路径里可能会有空格，『%s』前后应该加上引号，这样会更安全一点：）


We should not change the default datasource configuration here. For now, we will only use a PostgreSQL database if it is configured as a datasource through the Lucee Admin.


schreiben


default is never but I have learned to dislike counting on defaults to work all the time.


braucht


`while (true)`'s are used in a few places in our code and I don't think there's a good reason to go changing that code ?


Ну нельзя так делать. Логически неверно + андроид раскорячивает если так делать.


Эм, зачем?


will elminate.


GitHub doesn't like your ó. Encoding?


I thought I had... my bad.


От мержа осталось, выпилю


Should probably be a minor bump (ie. `0.4.0`), because you are adding functionality.


@grainier Shall we add proper access modifiers here for clarity


@lilacelephant , во избежания багов в дальнейшем при использовании класса в коллекциях нужно сделать Override методов equals && hashcode , equals должен проверять логическую эквивалентность объектов .
http://stackoverflow.com/questions/2738886/what-is-a-best-practice-of-writing-hash-function-in-java - здесь второй ответ - это как правильно писать hash code (попробуй поискать в eclipse , там должна быть генерация этого метода).
Вот здесь еще хорошо написано про equals && hashcode - http://www.ibm.com/developerworks/library/j-jtp05273/


And if we'd have to deal with it properly we need to deal with it with cmake, because there are some nasty cases to deal with, like certain versions having `unordered_map` inside a transitional namespace like `std::tr1::unordered_map` and so on. In case i can contribute as i have some code for that already done.


Fixes #47


Видимо, надо будет вынести атрибут и добавить цвет для ночной темы


Possible that header <> null and req_headers == null


cc @edolstra


странное название


pasiba


thanks


Это лишнее. Тем более tools: в стилях не работает.


Please use a better name for the intake motors. And why is there four? Good suffixes might be 'left', 'right', 'top', 'bottom', etc. We'll need to be able to locate motors by name.


We should also do something with these lines when `false`:
https://github.com/abiquo/abiquo-cookbook/blob/master/templates/default/abiquo.conf.erb#L49-L50


On a hunch I've tried to disable `boot.cleanTmpDir` and revert this patch. That was it, this is just another symptom. I'll do more testing when proper fix for `boot.cleanTmpDir` lands, let's not revert this until we are sure.


yep, that's true and affects others macros like `SOL_EXP_CHECK`


The more I think about this, the more I think we should use:

```
"ix_#objMeta.tableName#_version_number"
"ix_#objMeta.tableName#_version_author"
"ix_#objMeta.tableName#_record_id"
```
```

This allows the index names to be more informative when looking at from other code (and is how the table indexes work too).


@Mumfrey: We're still having an issue related to anon inner classes in production. Would you be able to come on irc?


Ночной стиль?


We can try a compilation of a tiny C++ program under cmake in order to see what the compiler can support. And so turn OPENMVG_STD_UNORDERED_MAP to 0 or 1 accordingly.

My own experiment was that there was no special gain when unordered map was used... but perhaps I didn't made sufficient experiments.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11852](https://teamcity.mesosphere.io/viewLog.html?buildId=107139&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:03:53


Modified getOrganizationMemberships with Iterable
@matthewtckr - please go through the updated pull request


Ок, мб


Okay, but this won't help when such a `typedef`'d enum appears inside something else, e.g. a namespace.  Maybe using different icons would somewhat help in this later situation, as at least they won't look totally duplicates?


@MrTortoise Thanks a lot for this!  I agree with the switch to `System.Collections.Immutable`.  Could you comment on why you added all of these to `packages.config` as well?

- `System.Collections`
- `System.Diagnostics.Debug`
- `System.Globalization`
- `System.Linq`
- `System.Resources.ResourceManager`
- `System.Runtime`
- `System.Runtime.Extensions`
- `System.Threading`

I don't see why those are listed in `packages.config` but maybe I am missing something. :)


Ah, indeed, thanks!


The purpose of this line is to be able to do escape entities like `mytable.mycolumn`. I think the result here should be `"mytable"."mycolumn"`.


А тут могет быть null?


@crivano algum motivo pra não simplesmente apagar as linhas ao invés de comentar? Estamos versionando código, então se por algum motivo precisar voltar, basta ver o histórico :)

Deixar código comentado pode confundir quem for ler, além de deixar o arquivo com linhas inúteis.

Alguns links sobre:
http://elegantcode.com/2010/06/06/the-best-code-you-will-ever-write/
http://nedbatchelder.com/text/deleting-code.html


Simply a class name amendment to make the code easily readable


Тогда тут могет быть NPE :)


Das passt so, von mir aus ists abgeschloßen


Added `@import 'spree/backend/themes/blue_steel/globals/_variables_override';` to my `spree/backend/globals/variables_override`. Couldn't see the changes, does the import order of variables and variables_override need to be switched in spree_admin?

```
@import 'spree/backend/globals/variables_override';
@import 'spree/backend/globals/variables';
```


Need public function construct..


Yes, I'm fixing it.

2016-01-18 12:43 GMT+01:00 Stefan Kohlbrecher <notifications@github.com>:

> Shouldn't this be 'DistVoxel& distVoxel = ...' for efficiency? Currently,
> a copy of the voxel is created.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/schnaubelt/OpenChisel/commit/e1046d04806cf20fcbf2dabdb9b5d54013ff98e6#commitcomment-15504939>
> .
>


а "selected" локализовывать никто не собирается? Никогда?


Pretty sure this is ignored.


С этим null что-то не так


src to this XSendEvent-system: http://www.doctort.org/adam/nerd-notes/x11-fake-keypress-event.html


@ChalkPE you've removed the public function on either the file,  re-write it in RedstoneOre.php and work


Isn't setZoom -> setView -> _resetView always calling _limitZoom internally?


The code can take whatever it likes, but that doesn't mean the code you wrapper it with has to behave any particular way.

It's fine, though. Whatever works for now is ok.


Yes, although for readability I would prefer if the .stan files said

if (has_intercept == 1) {
  // stuff
}
else {
  # include "eta_no_intercept.txt"
}

and then eta_no_intercept.txt would not need braces.

On Mon, Jan 18, 2016 at 11:14 AM, Jonah Gabry <notifications@github.com>
wrote:

> correct eta if no intercept
>
> @bgoodri Does this look good to you for correcting the issue due to
> centering X when the model has no intercept?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/stan-dev/rstanarm/commit/4cca945477e8f0e3a4b42ccbc9574a93af0952a9
>


check


Because, this code block is duplicated. Net-tools package install with package command at the beginning of the recipe.


Same as


Что это???


@tomruttle better?


Does this do the job then?

    (defvar *cif-table* (load-time-value (make-hash-table :test 'equal))
      "A hash table of foreign functions and pointers to the foreign cif (Call InterFace) structure for that function.")


Никаких вьюъ в моделе быть ни должно


This breaks refreshing of the anonymous tag name when the `typedef` changes.  Fixable with i.e.
```diff
diff --git a/src/symbols.c b/src/symbols.c
index b76779d..776e65c 100644
--- a/src/symbols.c
+++ b/src/symbols.c
@@ -1448,7 +1448,7 @@ static void update_tree_tags(GeanyDocument *doc, GList **tags)
                 if (parent_name && ! g_hash_table_lookup(parents_table, parent_name))
                     parent_name = NULL;

-                if (!tm_tags_equal(tag, found))
+                if (!tm_tags_equal(tag, found) || tm_tag_is_anon(tag))
                 {
                     const gchar *name;
                     gchar *tooltip;
```


your hand shimmered


5420-OSCM-Docs (mitchute) - x86_64-MacOS-10.9-clang: OK (2050 of 2050 tests passed, 0 test warnings)

![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)


excessive var


Hi,

here, the size of the previous record should be subtracted!

Best,
Lukas


The polr.stan file never has an intercept and estimates J - 1 cutpoints, so
it should be fine. And it looks like lm.stan does the shift regardless of
the value has_intercept.

On Mon, Jan 18, 2016 at 11:25 AM, Jonah Gabry <notifications@github.com>
wrote:

> @bgoodri <https://github.com/bgoodri> This commit and the previous add
> this correction in the model block and generated quantities blocks for
> continuous, count, binomial, and bernoulli. Do you know if lm.stan and
> polr.stan need a similar correction or do those Stan files avoid this issue?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/rstanarm/commit/48256ef33aad3fef01cf06d5c4a47c2d575eab1c#commitcomment-15511486>
> .
>


Ha, you're right. Can you try removing this call to `_limitZoom` and see if everything still works OK?


Please always use curly brackets to define blocks, even for single lined blocks.


Thx - did you reread my concerns? That's what I pointed out before.


How does this affect the index page?


nope, this afternoon I got a very bad connection and sometimes no connection at all. cmake was just crashing on me.
working offline is legitimate I think and I couldn't work with these crashes.
Moreover
 * it was also crashing with Feel++ tarballs and in my case  brew installation  of Feel++
 * hpddm, paralution jsonlab quickstart nlopt are not mandatory for Feel++, there is no reason why they should crash Feel++ I think

what do you think ?


Webpack - you do it wrong!

Wait PR request in next few month.


C++11 is now required in openMVG (with cereal and many c++11 specific syntax we started to use everywhere) that's why I changed to "using".
So I suppose that we need to compare map/unordered_map again and choose the best one in term of performance to remove the ifdef.


Are you using capp_lint? It catches these things.


Fixed in PR #659


Actually, users can just double-click the .sketchplugin file and it will be installed.
It'll even do version check when adding it that way.


Is `result` always going to be an array?


Oke, Then I will commit it.
Next challenge refractor the io stuff out and after that make a report with all the transactions of a account in tabular form.
Maybe on the way we find a way to make this work as you want it, Work with functions instead of data-structures.

Good plan ?


BTW, yeah, before this patch it'd a bit odd looking, but I don't think it's a real issue either, so not a blocker for me either way.


to say it a bit differently, all the codes above are included in Feel++ but simply being offline forbids you to do a cmake. I don't think that's a nice/wanted behavior, no ?


mContext - в моделе это не оч хорошо. Но если уж очень хочется - бери от него getApplicationContext() тут а не надейся, что это сделает кто-то кто ее будет использовать


LGTM


Oh, I hadn't twigged that, sorry!  There's probably a way to avoid mutating the payload that's expressive (`change_history[0..-2]` on the next line sprints to mind, but it's not expressive), but maybe it's not that important - my objections are fairly purist in nature.


whoops


We'll need a redirect for this and we'll need to update the smoke tests.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11854](https://teamcity.mesosphere.io/viewLog.html?buildId=107154&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:08:28


This shouldn't cause segfault no? It didn't on my machine.


It looks this change fixes the interactive dismissal problem, but now photosViewControllerDidDismiss is not called when I press the X button


C++ code on Linux is frequently `.cc` too


Yep, you are right ... userspace camera drivers indeed read info from app_info, that's why I updated app_info in my kernel  ... but there is some issues with camera , sometimes mm-qcamera-daemon crasch ... and cannot have working on the same time , photo camera & video rec !  I used updated Camera Hal from cm-12.1 , and I tested also with camera_wrapper ( camera.vendor.msm8916 ) - but I can not find a way to make these two things happen at the same time - I have camera working on photo without video rec. or video rec. working but camera photo not working ...


Can you explain the protocol numbers? They don't seem to follow the client hello format. (Iirc)


@bufferoverflow
The message says

    # 	cd .../test
    #	python -m SimpleHTTPServer 8001

So it would result in trying to browse ./test/test and 404.

Maybe it's better ?

    # 	cd .../test && python -m SimpleHTTPServer 8001


This needs to be deleted.


А этому ифу за что не досталось скобочек? { } ;)


Thanks for all of your suggestions, guys! Will push an update addressing some of them soon.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11853](https://teamcity.mesosphere.io/viewLog.html?buildId=107149&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:09:59


space


@kittnz are you sure these should have movetype 1 and not spawndist 0? They're called "miners", which seems to imply they are standing still and mining until pulled.


if this was wrong, I guess I noticed it in some other places also. It is slow to load in the browser, ie why telling here. Sorry.


any chance to merge that PR ?


Looks good thank you for this.  Only question I have is does adding an Audio Listener automatically have any performance side effects?  Should this be a configurable setting?


Hello @aparajita ! No, I just didn't know about it...


had a closer look at the args enforcement is hitting us quite a bit where we have functions declared with empty body's to be implemented by "sub classes", see here from doing a quick run with the args set to all or even after-used:
```
/home/rlgomes/jut/workspace/juttle/lib/views/view-mgr.js
  73:50  error  "reject" is defined but never used  no-unused-vars

/home/rlgomes/jut/workspace/juttle/lib/views/view.js
  32:20  error  "time" is defined but never used  no-unused-vars
  35:20  error  "time" is defined but never used  no-unused-vars
  39:23  error  "data" is defined but never used  no-unused-vars
...

```/home/rlgomes/jut/workspace/juttle/lib/runtime/procs/base.js
  138:31  error  "from" is defined but never used  no-unused-vars
  147:34  error  "from" is defined but never used  no-unused-vars
  150:34  error  "from" is defined but never used  no-unused-vars
  153:27  error  "from" is defined but never used  no-unused-vars

/home/rlgomes/jut/workspace/juttle/lib/runtime/procs/batch.js
  90:26  error  "from" is defined but never used  no-unused-vars
  97:28  error  "from" is defined but never used  no-unused-vars

/home/rlgomes/jut/workspace/juttle/lib/runtime/procs/fanin.js
  49:26  error  "options" is defined but never used  no-unused-vars

/home/rlgomes/jut/workspace/juttle/lib/runtime/procs/head.js
  36:26  error  "from" is defined but never used  no-unused-vars
  41:19  error  "from" is defined but never used  no-unused-vars

/home/rlgomes/jut/workspace/juttle/lib/runtime/procs/join.js
  723:73  error  "item" is defined but never used  no-unused-vars

/home/rlgomes/jut/workspace/juttle/lib/runtime/procs/periodic_funcs.js
   15:26  error  "options" is defined but never used  no-unused-vars
   43:26  error  "from" is defined but never used     no-unused-vars
   47:30  error  "points" is defined but never used   no-unused-vars
  184:29  error  "epoch" is defined but never used    no-unused-vars

/home/rlgomes/jut/workspace/juttle/lib/runtime/procs/procs.js
  78:35  error  "params" is defined but never used  no-unused-vars
  85:53  error  "reject" is defined but never used  no-unused-vars

/home/rlgomes/jut/workspace/juttle/lib/runtime/procs/reduce.js
  210:35  error  "params" is defined but never used  no-unused-vars
  246:19  error  "from" is defined but never used    no-unused-vars
  274:35  error  "params" is defined but never used  no-unused-vars
  320:27  error  "epoch" is defined but never used   no-unused-vars
  326:25  error  "time" is defined but never used    no-unused-vars
  330:28  error  "from" is defined but never used    no-unused-vars
```

In the new PR I'll put up I'll simply keep the args: none to be fixed in a future PR.


Very nice optimization strategy.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11856](https://teamcity.mesosphere.io/viewLog.html?buildId=107185&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:03:47


## 做了以下修正
- 把 pagepling 的語法弄正確
- 並且加上 afterrender (在 pagepling 執行完之後就做....) 事件內加上把 pages overflow 設 auto 就可以滑動了
- 再把妳改掉的語法再加回來 = =


This is interesting
http://supercomputingblog.com/windows/ordered-map-vs-unordered-map-a-performance-study/


I re-read the code because I still don't understand your comments :)


Я так понимаю тут у нас на UI-треде картинки в base64 туда-сюда перекодируются???


What is this (slightly annoyed and questioning tone)
If you want to do an intellij IDE, then make a branch and do your development there and merge the code changes back in, but not the ide stuff


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11857](https://teamcity.mesosphere.io/viewLog.html?buildId=107220&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 216 Build time: 00:04:17


Tiny thing, but can we rename the parameters `panDelta` an `zoomDelta`?


good


I wanted to use the library name rather the package name, but I don't mind either way.


apparently unordered maps have faster access and insertion time and 'The only downside [of unordered maps] is that you can’t directly iterate over the elements in alphabetical order. Of course, that’s only a downside if you need that feature'.


yes


Thanks! I would expect browsers to inline code as well, so there should be no impact form splitting out the `hasAttribute` code into a separate function.


выглядит очень не логично, когда метода с названием notify** Переставляет состояние


@Menda thanks for the review, but I'm not getting which part you are suggesting to delete :)


Why reformat the whole POM? Makes it hard to merge and see what changed. Also, we always use tabs for indentation in both XML and Java source files.


Basically I asked, why you have overwritten msg.timestamp with m_audioClock and why you did not keep: audioframe.pts as it was valid before.


force_text instead of force_test


Now you do.


Leave a line note


No, `ffcall-body-libffi` should expand into something like

```lisp
(let* ((cache (load-time-value 'prepared-function-cache nil))
       (prepared-function (or (cdr cache) (prepare-function ...)))
   ...)
```


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11851](https://teamcity.mesosphere.io/viewLog.html?buildId=107138&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:08:24


5420-OSCM-Docs (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2088 of 2088 tests passed, 0 test warnings)

![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)


Ah! I didn't even think of that.


Можно спрятать в вёрстке.


I want to debounce, but what I really should do is to debounce right :)


I think this is the good solution,
http://stackoverflow.com/questions/973541/how-to-set-sqlite3-to-be-case-insensitive-when-string-comparing


5420-OSCM-Docs (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (956 of 956 tests passed, 0 test warnings)

![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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-26.85%25-green.svg)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11855](https://teamcity.mesosphere.io/viewLog.html?buildId=107155&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:10:13


Please open a ticket and be more specific in your questions :)


Судя по этому интерфейсу модель которая называется SIgnInModel занимается в том числе публикацией чего-то-там и выгрузкой картинок. Что есть конечно плохо. Либо разбить на 3 модели, либо переобозвать в DoAllWorkGodObjectModel. Во втором случае со временем без сомнения код этого класса будет весит мегабайты и никто в него залезть не отважится. Даже ты :)


came with the sniff so i think they should yes


We normally don't use namespaces


Should drop the generic `<T>` as well.


Ну вот как ты думаешь - гугл это апи подепрекейтил исключительно для того чтобы заставить тебя либу ручками подложить, или еще какие-то причины были?


members are normally prefixed with `m`

    C* mConn;
    P* mPool;


http://www.wowhead.com/npc=18369/corki & http://www.wowhead.com/npc=18445/corki
npc in the world are not confused?


No. If `many === 'one' // one or anything else other than 'many'`
than it will return 1 value. For `many==='many'` it will return array of posts


Please linebreak (I think astyle even does this meanwhile, no?)


This line https://github.com/tes/seguir/commit/1870afa1a7f81a31763efd3dab0c6bd79930624b#diff-c464dc9095a6f91e6862451557182f27L29


nullptr


@kwin per my comments in your PR, I don't think that's the right implementation. I'll test out a related concept, but if the exception I alluded to above (which was in the 5.6.1 era) still occurs, I don't think anything related to optional dependencies will be workable.


Is this the fix for issue #28?


лалалал


лололо


Yes. The header is not defined in the C++98 standard, so compilers are not required to support it. It exists in C++11, though, and all it does is define __bool_true_false_are_defined, which is rather pointless.


it's a way to make sure the symbol does not go outside of the module (kind of "module static" for C++)


@xavi-mat -- it looks like the current .ª for ordinals comes from you originally. I'm wondering why the period? This currently gives us 7.ª  as opposed to 7ª  -- why? I have never seen that period in Spanish.


It did on mine.


@t-oster: can we make java error messages show up on the gui somehow?


Pushed commit to develop branch.  Thank you @jamesfry


a "," is not a valid float separator here


All of the code is the same, that is preventing uploading of unnecessary IDE files. It will not affect anything.


I like the idea to make connections scoped.

What's the advantage over

    QScopedPointer<QgsPostgresConn>conn( QgsPostgresConn::connectDb( uri.connectionInfo( false ), false ) );


Fine then.


Hey @pat270

the pull looks very good. My only concern is that we are including css related to staging in the control menu file. Could we move some of this css to the staging file? (publish-btn, staging-ready... and even the control-menu-level-2 since that html is generated by the staging bar, right?)

Other than that, it looks perfect! I don't have any ticket for this, but feel free to open a subtask of the big lexicon story or any other.

thanks a lot!


oh, I'm new to pull requests, in fact that's my first one. I've just arranged to test in locally :)


提交的注释“现实”改为“显示”


Hmm, http://tc39.github.io/ecma262/#sec-array.prototype.copywithin suggests it should be `delete o[to]`. Can you please file an issue?

Everything is better than a loop when it comes to expressing intent.


@bechte, I am addressing this topic in #124.


This is stupid comment


Module as in .so/.dll?


відвідати сайт


we have to call `->unref()` (if using the to-be-deprecated API) or `pool->releaseConnection` (if using the pool api) on the connection object.


É interessante nesse caso usar o bourbon: ```@include size(100% 75%)```


Atenção para a ordem alfabética ;)


this is not working not selecting the selectedOne


Tenho usado o ```transition``` do bourbon também ;)


That looks good to me. Thanks @jwahlin!


Оно там еще в нескольких местах по проекту захардкожено (((


> maybe use fontTools.misc.py23 instead?

Sounds good.


@Sulfer For any further inquiries on the matter, contact me at john@neetgroup.net.


shouldn't that happen anyway in the destructor?


Ok, I will remove it :D


Yeah, agreed. Too ugly and not consistent with what we've done so far :)


Это зачем??? О_о


fr


تنماتنلالا


It does NOT happen, in the current implementation. The destructor is meant to be called by "unref()", when count goes down to 0. The pooler's `release` method calls `unref` which eventually destroys the connection.


Yeah I know how to do it, but I couldn't edit. No permissions or something, wasn't added to a branch or whatever ;)
Yeah probably set the gen to a lower level.
Are you coming on?


I like this idea very much, but I would like a couple changes.

* `*fetch-assoc*` is not a good name for what it is - better would be `*fetch-scheme-functions*` perhaps
* New functions and variables should have docstrings
* The default http fetch function should be named http-fetch, and made external


@ibelyaevjcvi, Instead of the internal adama-dev.tacc.utexas.edu URL, you could use the official external facing URL https://community.araport.org/docs/adapters/index.html.

Thank you!


It's amazing how much information is in the Rails guides that you can miss unless you're looking for it exactly.
http://guides.rubyonrails.org/active_record_validations.html#conditional-validation


I think that this logic can be simplified:

```C

if (ret == LUA_YIELD) {
    // do something special, like reschedule a TS_LUA_EVENT_COROUTINE_CONT?
    TSMutexUnlock(main_ctx->mutexp);
   goto done:
}

if (ret != 0) {
     TSError("[ts_lua] lua_resume failed: %s", lua_tostring(L, -1));
}

lua_pop(L, lua_gettop(L));
TSMutexUnlock(main_ctx->mutexp);
ts_lua_destroy_async_ctx(actx);

done:
return 0;
```

Don't you have to reschedule an event if the Lua function yields?


I guess we should even drop the refcount from those connections, but we can't until we expose an API that allows "sharing" them.


@sbrunner @ger-benjamin Here, the theme node `name` property is used to create the layer. Shouldn't it be the `layers` node property, and the `name` would be the label ?

Please confirm, don't change the code i'm refactoring.
I need to understand what `layers` and `name` node properties are.


FYI, I think `->pointerfree` is not just a fast path, I think you can get incorrect result for mutable type members otherwise.


@raphaelpaiva vc tem toda razão. :)


А что помешало остальные restAPI - запросы построить по этой замечательной схеме и не городить сервисы/ресиверы?


The RHEL doc said that oddjob was required for all versions of RHEL. Only `oddjob-mkhomedir` was specific to RHEL 7.


yes


I hope @dmsherazi merges this soon so that I stop getting notifications with people asking why a non-functioning plugin has errors. kthanksbye!


Решение через `push` не очень, если где то нужно будет обратиться к пункту меню для его изменения, искать пункт по `id` в массиве не ок


In order to pass through html classes, etc, I had to tweak the builder a little.


I added the "m" prefix and the "nullptr" usage. The pre-commit hook does not add the newline you mentioned


sure, here is the PR. https://github.com/elastic/elasticsearch/pull/16062


Torven brought something up, and I just tested and confirmed he was correct. Resurrection Effects should not block all strength buffs, only ones where the strength effect is in the spell's first slot (since that is where Resurrection Effects has the strength debuff.) Two good exmaples of spells that should not be blocked are Yaulp II and Yaulp III, as they have the strength effect in their third slot. I've confirmed the client allows Rezz Effects and Yaulp II/III plus several others in the same situation to stack (spell icon, strength stat increased.) But, it blocked any buff where Strength is in the first slot (no spell icon, no change in strength stat.)

So the correction is simply instead of using IsEffectInSpell() you'd use something like:
spells[spellid2].effectid[0] == SE_STR


@Nuku : Is the mis-named file of Hadiya for Hayato maybe still in your compiler figures folder? I did push the correct Hayato file into the hub shortly after the first images, but in the current game version, it's still that mistaken identity image.


ok


I've changed all of the ones that I could find with grep.


There are probably many ways to write this functionality, I just chose one that came to mind.


> The pre-commit hook does not add the newline you mentioned

Do it anyway ;)


Thank you. I will change.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11858](https://teamcity.mesosphere.io/viewLog.html?buildId=107229&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:03:59


Windows 10 seems to like constantly undoing my directory links, which is
fouling that up.

On Mon, Jan 18, 2016 at 9:28 AM, Wahn128 <notifications@github.com> wrote:

> @Nuku <https://github.com/Nuku> : Is the mis-named file of Hadiya for
> Hayato maybe still in your compiler figures folder? I did push the correct
> Hayato file into the hub shortly after the first images, but in the current
> game version, it's still that mistaken identity image.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Nuku/Flexible-Survival/commit/8a176f1ed32f2df0a0f2a9b6773fddb83630985a#commitcomment-15512979>
> .
>


why are pswpin and pswpout multiplied with a factor of 4?

are pgpgin and pgpgout at least as important?


Ok. But it doesn't appear required on centos, fedora or amazon linux?


NICE! :+1:


Shouldn't this be
    #if !defined (_MSC_VER) or ...


Hi, I appreciate the patch, but I just had a thought that in the past couple of days I have been building with `riscv64-unknown-elf-gcc -m32 -march=RV32I` It might be better to use these flags during software build rather than building a new compiler. What do you think?


`openssl-perl` exists, it's just not in a repo that gets configured by default. If we don't need the package any more, maybe we should just remove it altogether?


5420-OSCM-Docs (mitchute) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


@ErezSela doesn't this bugs the order full flow?


di file migration kan undah ada yg institution, teh?


I was thinking about that, but I think it's too risky without reliable automated tests.


as in translation unit


TeamCity pull requests :: netty [Build 1050](https://garage.netty.io/teamcity/viewLog.html?buildId=4856&buildTypeId=pulls_netty&guest=1) is now running


To what I understood,  the std::map are generally implemented using redblack trees or avl and the unordered map as hash tables.
If we have loads of elements, more than the number of bins or if the elements we want to store in the map are not well distributed, meaning many of them are mapped to the same bin, it is better to use a std::map. Otherwise unordered_map seems to be a better solution. Anyway it's better to test in real conditions :)


I don't think this will work because I'm compiling all three targets from the same platform, so I need to be able to choose between them.


`::` is useless in `ng-click` directive expression.


Yeah, will do.


Good to know!!  Thanks for mentioning!!


The reason I have a problem with this phrasing is because if I run a bunch of code before or after `Lwt_main.run`, as long as I've enabled backtraces for the whole program, they will work in that code, regardless of Lwt. As an Lwt newbie, this sentence would (and briefly did) make me worried that Lwt is doing something crazy to the runtime that is affecting exception backtraces globally. I would recommend something along the lines of

> If an exception is raised inside an Lwt thread, the backtrace provided by OCaml will not be very useful. It will end inside the Lwt scheduler instead of continuing into the code that started the thread. To avoid this, and get good backtraces from Lwt, use one of the syntax extensions:
>
> With {{{Pa_lwt}}}, you need to pass... [SNIP]
>
> With {{{Ppx_lwt}}}, fixed backtraces are enabled by default. This has a small performance impact, so you can disable them by passing... [SNIP]
>
> As always, to get backtraces from an OCaml program, you need to either declare the environment variable {{{OCAMLRUNPARAM=b}}} or call {{{Printexc.record_backtrace true}}} at the start of your program.


@mshazly here


[BODY:QUADRUPED_NECK:TAIL:2EYES:NOSE:2LUNGS:HEART:GUTS:ORGANS:THROAT:NECK:SPINE:BRAIN:SKULL:MOUTH:GENERIC_TEETH:TONGUE:RIBCAGE:5TOES_FQ_REG:5TOES_RQ_REG]
Lizard, giant lizard and lizardman got new bodypart - GENERIC_TEETH.
Their bodyparts array is now longer. If you spawn giant lizard in 42.05 arena with 42.04 raws, punch it in the last body part - fifth toe, save game, update raws to 42.05 and try to load - game will crash - IndexOutOfRangeException. Not 100% chance to break save, but bigger than 0. Especially if you play as lizardman adventurer.


linebroken !

About the "what's different from"... this could probably be done with std::unique_ptr and passing a custom destructor relying on the singleton. But I don't know if we want to rely on that singleton to remain a singleton forever.


From http://unix.derkeiler.com/Mailing-Lists/FreeBSD/net/2012-04/msg00171.html and some further research (Darwin examples corroborated with FreeBSD doc), it looks like the following would be the compatibility equivalent for FreeBSD-derived kernels, and presumably others:

    s.setsockopt(socket.SOL_SOCKET, socket.IP_RECVIF, listen_interface, len(listen_interface))

This could be handled using a lambda in the try-catch block previously described, though the `IP_RECVIF` constant may not be defined in Python, so there would need to be another test for that. I'll worry about refactoring for BSD compatibility post-commit if you implement what was previously suggested; it won't change the interface you've suggested in any way, so this comment is mostly for my notes when it's merge-time.

For finding the appropriate interface, it looks like getifaddrs(3) is consistent enough between BSD and Linux to allow for matching of the server-IP to an interface on the host, so I'd like to prototype a function for that before adding a possibly-redundant option to the config-level. It's a problem solved by the netifaces package, which is broadly available, but that targets Python 2.6+ (and adds a dependency for a non-optional feature), and I know of at least one environment that can't move on from 2.4 yet.

I do know why it didn't work as expected, though: I added the broadcast handler after the Server IP option way back in 2009 during one of the initial sprints and had since forgotten that there were actually two sockets providing the service. The nature of the network topology then prevented me from ever experiencing the issue you're facing, and the host providing responses ran both of its interfaces over a bonded link, so emission problems never came up, either; since then, everything I've tested has been single-interface, an environment where noise on the other side didn't matter, or a purpose-built VM.


@rpavlik  This was removed in the subsequent commit.

It might be nice to allow for a zero count, though, to make it easier to programmatically generate the device descriptors.


This is probably not the correct image name, or is this supposed to go upstream?


I don't think this is a valid way to do timeout.  Join doesn't guarantee how long its going to sleep.  https://docs.oracle.com/javase/tutorial/essential/concurrency/join.html


Is that really what you wanted to do, or was it more like
```
total_shares = 0
```
Because as far as I see, this will cause the total_shares to increase, while the miner shares are getting reset, which means the rewards per miner will just keep decreasing and you will have KST left over after payout to the miners.


I guess just not `CORE_EXPORT`ing it would have mostly the same effect but not sure.


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


> istream_iterator also uses `post_increment` to override the default behavior - it needs to return a copy of the "old" iterator value like a `Forward` iterator would.

Why? What makes `istreambuf_iterator` special among `Input` iterators?


I would probably allow it to read a bunch of files from stdin, or accept the file-to-be-read as an additional parameter instead. Therefore, the git example could become something like this:

    git ls-tree -r --name-only master | finja -i -l -

(This assumes -l to specify the "file-list" parameter)


@rpavlik  This was removed in the subsequent commit.

I wish JSON was permissive about trailing commas. It makes it easier to programmatically generate it.

Incidentally, I did try to validate the JSON before pushing it, but the validation failed due to a bug in the schema. See https://github.com/OSVR/OSVR-JSON-Schemas/issues/14.


In any phrasing, I think it's better not to mention debug versions of operators. That would make me start looking for them, but I think they are deliberately undocumented. Perhaps Lwt needs to make the operators debug by default, so we can get nice backtraces even without preprocessing. But that's a whole separate issue.


This looks good to me. Just wondering if the attribute name `OfferIntPtrOverloadGenerator` still needs the `IntPtr` in it.


No, the proposed approach is definitely better than this kind of workaround!


Corrigido @marciovsena  :+1:


5420-OSCM-Docs (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1507 of 1507 tests passed, 0 test warnings)

![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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.69%25-green.svg)


> `istreambuf_iterator` has a constructor signature that accepts the return type of its post-increment, corresponding to the cursor constructor up on line 134.

Why can't this be handled generically: a generic post-increment proxy that stores the cached value and is implicitly convertible back to the iterator? I'm pretty sure even `boost::iterator_facade` manages that. Why can't we?


The issue is true, but there is an easier solution not involving new flags at class level


Out of curiosity, what was the issue with the original commit?


This does make it much easier to get from module Lwt to the backtrace documentation, but I still recommend adding a note to `Lwt.async_exception_hook` as well. It will make life easier for people whose first encounter with a strange Lwt backtrace is through `Lwt.async`.


Sounds good! TO be clear, this is an experiment for now. I'm suggesting this as a direction for sbt, but I don't make that call :-) (it does sound like we have some consensus forming around this)


nvm, writing the test for it wasn't that difficult. Fixed in d24d4fbff2c263046374262fcd28037024962ec3, thanks!


This match is literally just the identity function. :smile:  You should probably just remove it (it's in the prover9 tactic as well).


A quick search suggests it's supported on all three, and the old approach (using `pam_mkhomedir`) is deprecated. A RHEL security update was the thing that actually broke the old approach -- maybe the other platforms just haven't caught up yet.


5420-OSCM-Docs (mitchute) - x86_64-MacOS-10.9-clang: OK (2050 of 2050 tests passed, 0 test warnings)

![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)


This was a derp moment on my part. Of course it wouldn't notify, the build didn't break.


:clap:


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11860](https://teamcity.mesosphere.io/viewLog.html?buildId=107254&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:04:38


`snapZoom` vs `zoomSnap`.

Are you sure you want to send `this._map.options.zoomSnap` as `_animateZoom` /  `noUpdate`?


-g3 was removed in this commit: 339a429dd2560a8f098fb4932dc18fbc9493bda0


lol, wanted to write dot. I meant the "Arch. Filter"


OK. Maybe just fix the comment to indicate that the package doesn't appear to be necessary (rather than saying that it doesn't exist).


Problem, Christopher King should go first


Well, now it's multiple (well, it was multiple before, but it was two camlp4 extensions before, now there are direct alternatives). So, maybe, "offers a choice between Ppx and Camlp4 syntax extensions, which increase..."


I'm not sure how to best approach these short tactics which are basically just abbreviations.  Having them as separate classes is consistent, but a lot of boilerplate.  Another option would just be to have them as defs in the package object, like so:
```scala
def forget(label: String) = WeakeningLeftTactic(label) orElse WeakeningRightTactic(label)
```

Sorry if this all seems a bit nit-picky, but I would just like to keep the code size down so that it is easier to maintain in the future.


Will this work on a non mysql database?


add_custom_checks (lefticus) - i386-Windows-7-VisualStudio-14: Build Failed

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


add_custom_checks (lefticus) - Win64-Windows-7-VisualStudio-14: Build Failed

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


ASND/ARCV cycles = (start_cycles - end_cycles) / 2; //just the average!


I'm trying this version of the Chrome extension out via `gulp dev` but I don't see either `background.js` or `background-chrome.js` running? So the GitHub CSP is still intact and it blocks the iframe loading.


Would be nice to have things that fall under pkgs/os-specific/linux to automatically do that :smile:


It would definitely be nicer to fail gracefully. We should eventually fix those.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11859](https://teamcity.mesosphere.io/viewLog.html?buildId=107253&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:07:44


Now this one seems wrong. I'll have to look into it more, but I wouldn't change the logic here. The example can confirm it as well, try scrolling on the right side, it's super slow.


Nice licence bro


Ok, let's see....

2016-01-18 18:34 GMT+01:00 Adrián <notifications@github.com>:

> @mbarrenechea keep an eye here for YOU-KNOW-WHY and lets hope this helps 🏥
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/Vizzuality/gfw/commit/908f8a6b3e5839f3b081fc6b4a4f18382334c079


Qemu output for micro_booter test:
------------------------------------------------------------------------------------------
phani@phani-thinkpad:~/transfer$ ./qemu.sh micro_boot.sh
System memory info from multiboot (end 0xc11c9000):
	Modules:
	- 0: [011ca000, 012a4000) : 40800000-40801010 @ virtual address 40800000, _start = 40801010.
	Memory regions:
	- 0 (Available): [00000000, 0009fc00)
	- 1 (Reserved ): [0009fc00, 000a0000)
	- 2 (Reserved ): [000f0000, 00100000)
	- 3 (Available): [00100000, 07ffe000)
	- 4 (Reserved ): [07ffe000, 08000000)
	- 5 (Reserved ): [fffc0000, 100000000)
	Amount of wasted memory due to layout is 10 MB + 0x36000 B
Initializing virtual memory
	Setting up initial page directory.
Found good RSDP @ c00f0990
Found good HPET @ c1bffb89
Initiliazing HPET @ c1bffb89
-- Checksum is OK
Addr: 00000000fed00000
hpet: fed00000
Set HPET @ c1d00000
Setting up the booter component.
	Capability table and page-table created.
	Creating 1 booter VM PTEs for PGD @ 0xc010e000 from [c11ca000,c12a4000) to [40800000,408da000).
	Mapping in booter VM.
	Creating 8 user-typed memory PTEs for PGD @ 0xc010e000 from [c2000000,c4000000) to [80000000,82000000).
	Mapping in user-typed memory.
	Creating 1 untyped memory PTEs for PGD @ 0xc010e000 from [c12b4000,c15ca000) to [60000000,60316000).
	Mapping in untyped memory.
	Created boot component structure from page-table and capability-table.
	Creating initial threads, tcaps, and rcv end-points in boot-component.
	Boot component initialization complete.
Enabling timer @ c1d00000
Upcall into boot component at ip 0x40801010
------------------[ Kernel boot complete ]------------------
p
Micro Booter started.
---------------------------
switchto 36
	New thread 2 with argument 0, capid 0
switchto 37
	New thread 3 with argument 1, capid 1
switchto 38
	New thread 4 with argument 2, capid 2
switchto 39
	New thread 5 with argument 3, capid 3
switchto 40
	New thread 6 with argument 4, capid 4
test done
---------------------------
pppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppAverage THD SWTCH (Total: 1345202058 / Iterations: 1000000 ): 1345
---------------------------
---------------------------
Starting timer test.
.p.p.p.p.p.p.p.p.p.pTimer test completed.
Success.
---------------------------
---------------------------
Page allocation: victory
---------------------------
---------------------------
Creating threads, and async end-points.
--> sending
Asynchronous event thread handler.
<-- rcving...
<-- pending 1, thdid 0, rcving 0, cycles 0
<-- rcving...
--> Back in the asnder.
--> sending
<-- pending 1, thdid 0, rcving 0, cycles 0
<-- rcving...
--> Back in the asnder.
--> receiving to get notifications
--> pending 0, thdid 10, rcving 1, cycles 0
Async end-point test successful.
Test done.
---------------------------
pppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppAverage ASND/ARCV (Total: 1614312243 / Iterations: 1000000 ): 1614
---------------------------
---------------------------
Return from invocation: 0
Test done.
---------------------------
ppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppppAverage SINV (Total: 10332070254 / Iterations: 1000000 ): 10332
Average SRET (Total: 7887547771 / Iterations: 1000000 ): 7887
---------------------------

Micro Booter done.
BUG @ micro_booter.c:469.
Page Fault in thd 1 (not-present write-fault user-mode  ) @ 0x0, ip 0x40800f9c
Shutting down...
QEMU 2.0.0 monitor - type 'help' for more information
(qemu) quit
phani@phani-thinkpad:~/transfer$


Series returned by `getSeriesArg()` must be treated as read-only, as other functions might want the original data.  This patch modifies series[0].  `r` needs to be a copy of series[0] instead.


Why not dual license?
gpl and mit?


Did you tried to switch the order?

On 18. Jan. 2016, 17:49 +0100, adaddeo<notifications@github.com>, wrote:
>
> Added@import 'spree/backend/themes/blue_steel/globals/_variables_override';to myspree/backend/globals/variables_override. Couldn't see the changes, does the import order of variables and variables_override need to be switched in spree_admin?
>
> @import 'spree/backend/globals/variables_override'; @import 'spree/backend/globals/variables';
>
> —
> Reply to this email directly orview it on GitHub(https://github.com/solidusio/solidus/commit/ea22dffd8a02e356bcc52f141897a6f513af6365#commitcomment-15512115).
>
>
>


Ah, of course! Will use it instead.


@hyperknot It's supposed to be 4x slower - the zoom delta is 0.25 instead of the normal 1.


I removed this file in commit a1e61c674c8f393c1afeee2a3595a371a770d2e1


Where does that version number come from?


Well, it's mostly one syntax extension with two implementations. The two versions are very similar.


This section currently gives a lot of a attention inline to Camlp4, while I'm under the impression that ppx is what the OCaml "community" is emphasizing. Since the ppx extension docs give a decent treatment, I don't think there is a need to give a correspondence table or anything else like that on this page, as done for Camlp4. However, I'd either say that the ppx extension is newer, or is using the "new" ppx facility of OCaml, or something to that effect, to avoid the impression that ppx may be an afterthought.

Is the Lwt project emphasizing the ppx extension? If so, that should be mentioned so that people aren't misled into using Camlp4 for new code.


Pour ne voir que les ajouts ou suppressions effectuées --> <-- à chaque mise à jour


sure


_has**Own**Property? :)


Ok, I'll trust you on that :)


\*Shouldn't it be enough to check whether I have a DoD object?

I doubt that handling actions via variables is the rails way:)


@hpi1 next try ;)


@vcunat so the only regression you're pointing out is in `gobjectIntrospection`? I'm fine with that breaking for now. Not entirely sure why it's broken right now though. Looks like it works fine when you have sandbox turned off and XCode installed (because it silently reaches out and grabs XCode stuff) but otherwise breaks if either of those things isn't true.


You're right, thanks for catching that. Not really sure what happened on the mismerge.


capinv.c: mainly cleaning up + removed assert as current->state can be preempted from interrupts.

timer.c: as per HPET specification, to set periodic timer, Tn_VAL_SET_CNF bit should be set and main counter should be cleared with 00000000h.


OK


Manque un espace me semble


IMO, to solve this properly, one step would be to make the evaluator respect `meta.platforms` and another to add `meta.patforms` to `mdadm`.


How does that look concretely? Does passing a `Ptr{Float64}` register as `jl_is_floattype == true`?


add_custom_checks (lefticus) - i386-Windows-7-VisualStudio-12: OK (382 of 383 tests passed, 0 test warnings)

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


Well, it seems `gtk2` and `gtk3` depend on it.


systeminfo*


Of course, it should be possible to build these without introspection on darwin for the time being...


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11862](https://teamcity.mesosphere.io/viewLog.html?buildId=107256&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 215 Build time: 00:03:53


For polr.stan what if the number of outcome categories is 2 and it does a bernoulli model?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11861](https://teamcity.mesosphere.io/viewLog.html?buildId=107255&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:03:58


aha, good call. i'll set up another test for that.


That's true, but someone reading the Lwt docs doesn't know that, especially if they are also new to Camlp4 and ppx. Since the syntax is different, the command line is different, and the implementations and packages are ultimately different, these are completely different to the user, even though there is a nice correspondence between the syntaxes they provide (form and function? :) ).


After the port to the new object model, this model of reflective shapes becomes somehow obsolete. I just made the minor changes to made it compile. I'm not currently using nor evaluating it. Forgot to put the comment, sorry...


I haven't been sure if we should deploy this file at all.
buildiso writes this file for sonar.
I just left it in there so the package as standalone provides some default.


No, what I mean is sth like

```
type Float64Wrapper
    x::Float64
end

immutable T
    field::Float64Wrapper
end
```


 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/acc4345b65cbe1a92772dc59957a8f66a2e7c262/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:683](../blob/acc4345b65cbe1a92772dc59957a8f66a2e7c262/include/chaiscript/language/chaiscript_parser.hpp#L683) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/acc4345b65cbe1a92772dc59957a8f66a2e7c262/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/acc4345b65cbe1a92772dc59957a8f66a2e7c262/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/acc4345b65cbe1a92772dc59957a8f66a2e7c262/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:0](../blob/acc4345b65cbe1a92772dc59957a8f66a2e7c262/src/main.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 13 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __information__: [src/main.cpp:1](../blob/acc4345b65cbe1a92772dc59957a8f66a2e7c262/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/acc4345b65cbe1a92772dc59957a8f66a2e7c262/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/acc4345b65cbe1a92772dc59957a8f66a2e7c262/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/acc4345b65cbe1a92772dc59957a8f66a2e7c262/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/acc4345b65cbe1a92772dc59957a8f66a2e7c262/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/acc4345b65cbe1a92772dc59957a8f66a2e7c262/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


On timer interrupt, context switches to main thread (thd id : 1), so need this flag to resume to async test! (Overlooked it for async unit test.. fixed it in a later commit.)


you may avoid extra branches on this method.

`if ( this === global ) {` is not a responsibility for this `QUnit.stop`. Let this be carried over the abstracted warning method, that's exactly why it's abstracted.

Also, It's also you're defining here the warning message on `warning = QUnit.warning.deprecated.stop;`, instead of it, just say the kind of message to QUnit.warning, like:

`QUnit.warning( "qunit-stop" );`, let QUnit.warning map `"qunit-stop"` to "QUnit.warning.deprecated.stop".

This way you move the complexity to where it should belong.


`--mkuhn` it is ;)


And `pointerfree` is for julia object pointer, not the `Ptr` type.


It still estimates a model with no intercept and 1 cutpoint, and then after
the fact it negates the draws of the cutpoint and calls the result the
intercept.

On Mon, Jan 18, 2016 at 12:54 PM, Jonah Gabry <notifications@github.com>
wrote:

> For polr.stan what if the number of outcome categories is 2 and it does a
> bernoulli model?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/rstanarm/commit/48256ef33aad3fef01cf06d5c4a47c2d575eab1c#commitcomment-15513455>
> .
>


Hmm, now it's no longer identical to the output of the CMD devices (as far as I know)


dude cherry picks code for changing 5 letters


@artburkart I'm not sure - you'd be better off asking in a new GitHub issue or somewhere like StackOverflow, since commit comments can easily be missed. I'm no longer working on this project, but the current devs can probably answer! :smile:


is it enough to replace this line by "r = *series[0]"


same mapping issue as `QUnit.stop`, the older form - `QUnit.warning( "asyncTest" );` - is great, it just need to be mapped by `QUnit.warning`.

It makes easier to trigger a warning on methods.


Mmmm...Perhaps. But this change is actually a rollback to the som original version of the file after i update the object model.


This is saying that we can have many acceptable forms?


That and you need to copy `r.Values` and `r.IsAbsent`.  That should be sufficient.


missing eof


If we are going to refactor this, we should probably just make a SendwithUs template instead of having the template here in the code. However, if we want to keep being lazy, we could use a .jade template here, or at least a CoffeeScript multiline interpreted string.


@sduckett Yes. We don't actively take advantage of it yet, but most states accept both the national and their own state-specific voter reg. form.


Ruby here, no '=>' - just `name: 'value'`


en effet, j'enlève


add_custom_checks (lefticus) - Win64-Windows-7-VisualStudio-12: OK (382 of 383 tests passed, 0 test warnings)

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


Fair enough. Let's use the plural. :)


Ok made that switch for readability in https://github.com/stan-dev/rstanarm/commit/cdc5c0913e0e81c68555f09e8cd3736f2af6755f


YYwWWx, YY = Year, WW = week and x = increasing letter. I found it didn't make sense to start from 1.0 because it's not really a "first release" considering it's a fork, plus I like that it carries release date info with it.


why extend `log` when you can just set `log.deprecated` directly?


Well, it was broken with **v0.10.0** so far, so I fixed it.


Mmmm not sure what you mean here. The get()?. Nevertheless, in the last version i think it is correct.


Done.


The current order is important. `variables ` is taking advantage of [SASS !default](https://robots.thoughtbot.com/sass-default) to allow `variables_override` to take priority.

The idea is that a library is usually `@import`ed instead of being `#= require`ed into your application, so you need a mechanism to set variables _before_ the library (as a whole) is included. Spree could certainly have reversed the order of those two files and dropped the need for `!default` entirely, since they've defined the hook themselves, but I didn't see any need to change it.


Ok cool


@lgeinert fixed, thanks


fix for #2


I don't think zoomDelta should be effecting scroll zooming, it is made for discreet zooming events (double click, zoom control, keyboard, clicking buttons, etc.), not continuous experiences, like scroll zooming.


Done


Actually I'm not sure why you would have `this.state` at this point? `getInitialState` is called when the component is mounted, does your issues stem from when the component is mounted multiple times?


sorry to be picky, but I'd really prefer ``if (mean - five_sigma) < datum < (mean + five_sigma): ...`` here.


When we did it that way at robot club, it was annoying because afterwards we'd have to type out all 50 or so email addresses, and people's handwriting was hard to read and we'd sometimes misspell the emails. How about suggesting using an Excel sheet instead? (Most clubs seem to do sign ups using a pencil and a clipboard anyway, but I can't think of a good reason of why that's better than using an Excel sheet.)


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/tests/output/ConsoleInteractionTest.py`, lines 298 to 301.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/tests/output/ConsoleInteractionTest.py
+++ b/coalib/tests/output/ConsoleInteractionTest.py
@@ -295,10 +295,10 @@
                                     file=STR_PROJECT_WIDE,
                                     line=5)],
                 {STR_PROJECT_WIDE: ["test line\n",
-                                "line 2\n",
-                                "line 3\n",
-                                "line 4\n",
-                                "line 5\n"]},
+                                    "line 2\n",
+                                    "line 3\n",
+                                    "line 4\n",
+                                    "line 5\n"]},
                 {},
                 color=False)
             self.assertEqual("""\n{}
```


1. can this be made asynchronously so that a request does not have to wait for a refresh token
2. Can we add a random seed to ensure that mappers on all machines do not timeout at the same time.


> Okay, but this won't help when such a typedef'd enum appears inside something else, e.g. a namespace. Maybe using different icons would somewhat help in this later situation, as at least they won't look totally duplicates?

Since I'll remove the renaming patch this shouldn't be a problem any more. Do you know where the icons from the sidebar come from? It would be nice to have a few more following a similar style.

> BTW, yeah, before this patch it'd a bit odd looking, but I don't think it's a real issue either, so not a blocker for me either way.

I'll post this patch separately.


:P


Would make more sense to call this *nsec, rather than *rr, IMO (due to the `assert()` below).


Or I'd rephrase my idea: ideally, scrollZoom should benefit from the smoothness of the fractional zoom levels, but wouldn't change the zooming speed slower or faster.


corrigé


I meant, we can perhaps remove it, we don't need it.

I need to check again if the file is written correctly.

https://github.com/manjaro/manjaro-tools/blob/devel/lib/util-iso-calamares.sh


I'll add that as a suggestion as well, though I've heard about troubles
with schools' outdoor internet. Thanks Andrew!

On Mon, Jan 18, 2016 at 10:02 AM, Andrew Downing <notifications@github.com>
wrote:

> When we did it that way at robot club, it was annoying because afterwards
> we'd have to type out all 50 or so email addresses, and people's
> handwriting was hard to read and we'd sometimes misspell the emails. How
> about suggesting using an Excel sheet instead? (Most clubs seem to do sign
> ups using a pencil and a clipboard anyway, but I can't think of a good
> reason of why that's better than using an Excel sheet.)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hackclub/hackclub/commit/f418eea0e904938e0765f73d58216710d936a1f7#commitcomment-15513644>
> .
>

--
Zach Latta
https://zachlatta.com <http://zachlatta.com>


el.namespaceURI ? el.namespaceURI + ':' : '')

is not correct you should look up the namespace prefix:
https://www.w3.org/TR/DOM-Level-3-Core/namespaces-algorithms.html#lookupNamespacePrefixAlgo


Alright, cool.


Updated to say 3.0.2 (which is what I have on the build machines)


hi

they came as part of the sytem.collections.immutable.

I am not sure why. When making this change set i reverted back to yours and
only performed the action of uninstalling bcl and installing the newer
package.

I *think* these come as part of the move away from the centralised core clr
approach into a more modular one to support things like windows 10 and
their open sourcyness.

Sorry I cannot be more specific but its not something i've needed to read
up about but however is something i am noticing in other areas of .net also.

On 18 January 2016 at 16:47, Timothy Shields <notifications@github.com>
wrote:

> @MrTortoise <https://github.com/MrTortoise> Thanks a lot for this! I
> agree with the switch to System.Collections.Immutable. Could you comment
> on why you added all of these to packages.config as well?
>
>    - System.Collections
>    - System.Diagnostics.Debug
>    - System.Globalization
>    - System.Linq
>    - System.Resources.ResourceManager
>    - System.Runtime
>    - System.Runtime.Extensions
>    - System.Threading
>
> I don't see why those are listed in packages.config but maybe I am
> missing something. :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MrTortoise/graphviz/commit/9aa0222c376070b31dfc48455fe639e0bf9f3bf6#commitcomment-15512078>
> .
>


he simple


Microsoft Excel or LibreOffice Calc don't require Internet access right?


Well, it's more of an accidental consequence of the fact that the ppx version has it's own page, while the camlp4 do not. Should be fixed.


TeamCity pull requests :: netty [Build 1051](https://garage.netty.io/teamcity/viewLog.html?buildId=4857&buildTypeId=pulls_netty&guest=1) is now running


Thanks, that's indeed better.


These were placeholders since we don't know the exact layout of the intake, and what they will be named. But to be changed on the once the "the final" bot is built.


It be great to release v1.03 so we know. Also I have another suggested class to add if you want: It allows you to just use 'getLogger()] and not have to say the class name each time. Makes it easy to cut/paste code.
 Should I send it to you?


Fixed.


Another test to be verified

SVG embedded inline in XHTML

     <svg:svg width="300px" height="200px">
         <circle xmlns="http://www.w3.org/2000/svg" cx="150" cy="100" r="50" fill="#ff0000" />
         <svg:rect width="100" height="100" />
     </svg:svg>


But should be noted to change. Using TODO feature maybe?


TeamCity pull requests :: netty [Build 1051](https://garage.netty.io/teamcity/viewLog.html?buildId=4857&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:55


The `<fun>Ptr` suffix for function typedefs is ugly.
IMO it is much better to typedef function
`typedef bool sensorIsDataReadyFunc(void)`
and then use it to declare pointer to that function:
`sensorIsDataReadyFunc *isDataReady;`
This way it is clear that isDataReady is pointer ..

Also it is quite good idea to use `gyro->isDataReady = &mpuIsDataReady;` when taking function address, even when '&' is only syntactic sugar ...


Please, use format. Please


Yeah, you're right. Not entirely sure why I thought that was a valid
counter argument. Internet actually shouldn't be a concern for Google
Sheets either (as long as you were able to create the document).

Anyways, just submitted #659 to add this.

On Mon, Jan 18, 2016 at 10:07 AM, Andrew Downing <notifications@github.com>
wrote:

> Microsoft Excel or LibreOffice Calc don't require Internet access right?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hackclub/hackclub/commit/f418eea0e904938e0765f73d58216710d936a1f7#commitcomment-15513728>
> .
>

--
Zach Latta
https://zachlatta.com <http://zachlatta.com>


This file is almost a copy of other files. Can we generalize it a little bit (less bug-prone)


Can we use here json.loads? :P


Yes, please. Use
```
// todo: make this change later
```


TeamCity pull requests :: netty [Build 1052](https://garage.netty.io/teamcity/viewLog.html?buildId=4858&buildTypeId=pulls_netty&guest=1) is now running


TeamCity pull requests :: netty [Build 1052](https://garage.netty.io/teamcity/viewLog.html?buildId=4858&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:05


Excellent – thanks! I will fix this shortly


    m <- NULL
    set <- function(y) {
        x <<- y
        m <<- NULL
    }

    get <- function() x
    setInverse <- function(solve) m <<- solve
    getInverse <- function() m
    list(set = set, get = get,
         setInverse = setInverse,
         getInverse = getInverse)


@yannis79
That was the name given in 1993. The UN site is stating this was provisional. Things have changed.
See https://en.wikipedia.org/wiki/Republic_of_Macedonia


Did you mean to erase the license check? Will that cause an expression error.


#628 provides an implementation similar to @kwin's but not creating new APIs and backwards compatible.

@kwin I think the situation I was describing with the exception was where there was no unresolved service dependency so SCR tried to instantiate it. In this case, there is a service dependency blocking the component from being instantiated, so there's no NoClassDefFoundException happening. Good catch. The classpath dependencies are kind of a PITA (it isn't just the component API, but the Promise and DTO APIs), but it doesn't seem to be blocking installation on 6.0