sure, no problem :)


This `^` ensures `test\\[1]` parsed as the property `test[1]`, not the index `1`


Specify that this is called automatically as a post build script by Jenkins.


1.7.6 -> 1.7.7


Also fixed the angular.toJson thingy but is this needed in the launch.js seeing as it's already in the background.js file which both the NewTab and BrowserAction use anyway?


How about making `getKeyEncoder()` and `getValueEncoder()` like this:

```js
function getKeyEncoder (options, op) {
  var type = op && op.keyEncoding || options.keyEncoding || 'utf8'
  return encodings[type] || type
}

function getValueEncoder (options, op) {
  var type = op && op.valueEncoding || options.valueEncoding || 'utf8'
  return encodings[type] || type
}
```

That would get rid of a lot of annoying parenthesis and still rows under 80 characters long.


Yeah at first I put std::endl, but then I thought there was probably a reason why it was preferred not to flush the output buffer in the original code, so I tried to preserve the logic.

Now, reading this, I wonder if that's the best implementation:
> If you are in a situation where you have to avoid buffering, you can use std::endl instead of ‘\n’. An example of the above situation is when you are using old style debugging using multiple cout statements between code lines  to see where exactly a crash occurs.

Don't we want to flush after "Computing turn...", so that we can better localise if a crash happens before this log output and "During this turn there were..."?


Yeah, I see #1. It's definitely doable via BinDeps.


> This not-initialized variable used several lines below caused me 15minutes of debugging. Please initialize your variables!

Update the CFLAGS to show uninitialised variables being used?


Each condition should have curly braces


> pi->envp can be null because we do not intialize this variable in name_acquired_handler() because it is not necessary as we are sure that the crashed application is not running.

problem_create_app_from_env() will return NULL if envp is NULL. So you don't need to change the code...


LGTM


Hi, did you encounter any issue I should be aware of?


Das passiert durch isohtmlspecialchars() unten automatisch.


I've fixed the bug that the host was not found if the domain name is not part of the hostname:

commit: c887f595efdfb88c24e9bd802d1a7623c82c62dd


TeamCity xBIM :: Xbim Essentials :: Release [Build 36](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1176&buildTypeId=XBIM_XbimEssentials_Release) is now running


The way to manipulate changing lists is usually to return the "GList *" from the function. So for example:
```GList * show_problem_list_notification (GList *problems_list)```


Why modified object looks like Object {ONE: "ONE", TWO: "TWO", THREE: "THREE"}? Upper-cased keys means "constants" in JavaScript. Normally we should modify values, but not keys. Use loop for working on it


TeamCity xBIM :: Xbim Essentials :: Release [Build 36](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1176&buildTypeId=XBIM_XbimEssentials_Release) outcome was **SUCCESS**
Summary: Running Build time: 00:00:35


Change to a class?


Class?


Just pushed again - it's now being set in `cider-interactive-source-tracking-eval`, which is the only caller of this function.


Hey Chris,
thanks for contacting me. In a couple of days i'll be buying your premium
plugin, just waiting for my company to be officially founded (in Italy it
takes AGES :-)).
Talk to you soon.

Alberto

2015-03-19 0:01 GMT+01:00 Chris Scott <notifications@github.com>:

> I see you're doing some work on my obsolete Android code.
>
> I've completely refactored Android and dispensed with doing HTTP in-plugin.
>
>    - it calls the javascript callback just like ios now.
>     - No more foreground-service.
>    - Battery efficiency is significantly improved (app can run for days
>    now and no longer turns on GPS until you move).
>    - It now uses the ActivityRecognition and FusedLocationProvider APIs
>    from google-play services.
>
> http://christocracy.github.io/cordova-background-geolocation/
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/geezmo/cordova-plugin-background-geolocation/commit/7f875c66374b78dd0906f8a676a8682c9512ba1a#commitcomment-10275670>
> .
>

--
Very funny, Scotty. Now beam down my clothes.


If you need an array from another array you can use iterators witch creates and returns new array


unten kommt ein urlencode() das das erledigt, im Redirect: sind keine HTML-Kodierungen notwendig


Oops! Fixed.


This won't work. It's required for all keys in the module data to be qualified with the module name. Reason being that nobody would find them otherwise. Only lookups with qualified keys will ever reach this data.


isn't this just `oGridRespondTo(S)`? Why do you need the `$from`


:) thanks


> Yeah at first I put std::endl, but then I thought there was probably a reason why it was preferred not to flush the output buffer in the original code, so I tried to preserve the logic.

Ah, so you did know. Sorry then for patronizing.

> Don't we want to flush after "Computing turn...", so that we can better localise if a crash happens before this log output and "During this turn there were..."?

It never was a problem before because someone always ended up running the beast with gdb or something to reproduce but I think you're right. Once we have more players, we'll have more feed-back and this could get handy.


Kami tidak menghapus JS API


das simmt. Wie krieg ich dies in Github smart übertragen?


@jzawadzki, zastanawiam się nad jednym. Czy symfony pozwala na stworzenie jakiejś abstrakcyjnej klasy np BaseManager  w ktorej umieszczę kod odpowiadający za dodawanie / kasowanie / edycje / liste userow , kategorii itd? bo w kontrolerach ten sam kod się praktycznie powtarza :cry:


Headline should go in the article_header block. Not the body.


> Ah, so you did know. Sorry then for patronizing. :S

Please continue to do so, there are advanced things I know and newbie things that I don't, so I'm glad to have some input from you :-)


Don't use IDs.


That's the way Braulio told me to did it... I can´t test it now but I search over the internet and I found this in Angular documentation for angular.injector function... The ng module must be explicitly added.


Setiap penambahan API, akan ada perubahan versi minor


> It never was a problem before because someone always ended up running the beast with gdb or something to reproduce but I think you're right. Once we have more players, we'll have more feed-back and this could get handy.

Ok, I'll use a std::endl then. I'll do that in a new commit so that we don't lose this conversation, and anyway the two commits have different purposes :-)


Please remove the comment.


Note that Phil Burk already have his pForth here:
https://github.com/philburk/pforth


"coalition-pie" rather than "coalition_pie" would be more consistent with codebase.


Reviewed changes, looks good to me.


@mpenet there's a behavioral change that I need to figure out.


@GalaticStryder Just noticed that too. Fixed it and pushing the fix :)


Line 530 (new) and 531 (old) are the same. Please keep the old one (line 531).


What's the meaning of the commented out lines? I guess it should work without.


I'm getting strange behaviour from this checkin...

By not #include'ing MyConfig.h, DEBUG is no longer #define'd in MyHw and MyHwATMega328, tripping up when you call debug().  My gateway reboots after calling gw.begin, because of the debug() call at the end (I presume it's the avr version of a segfault)...


Are sockets and pids really stored in /tmp/ on Redhat?


Part names seem to be littered all over the place. Needs refactoring so we don't have to change stuff in loads of places.


I'm not familiar with `assert_match()` used here but from the looks of it, no lookup will ever reach the module data since none of the keys in this example is prefixed with the module name.


Bad selector.


peerDependencies are super painful to work with, is there any way we can work around this limitation?


This gets unset over and over for each package returned by PackageSources. looks a bit redundant. why not:

```go
for _, src := range cfg.PackageSources() {
    addPackageSourceCmds(cfg, src)
}
cfg.UnsetAttr("package_sources")
```


if we merge the level-codec pull request those will be removed anyways so i wouldn't change them yet


Surely there's need to create a new object each time the filter runs.


I wish there was a way to tell `npm`: _"hey, please `dedupe` me if these modules aren't near me"_ but that's not how `peerDependencies` works :-/


Any reason for repeating the same call ? (261 and 262)


// TODO (aznashwan): more clear details here


Was there a `comment_params` method defined in this class?


same as above


@jensopetersen That is fixed here - https://github.com/eXist-db/exist/pull/535


In future, for this kind of DOM anchor/hook could we use the Origami recommended convention of a class name with the `--js` suffix. So in this case '#visualisation` => `.visualisation--js`.


I had a go with Eclipse. I could not get it to work with the Maven configuration either by importing as a Maven project or via direct GitHub checkout. So I ran ``mvn eclipse:eclipse`` from a command window (requires a standalone install of Maven and upate to PATH) in the directory containing the ``pom.xml`` to convert to an Eclipse project and then imported into the workspace. All worked.


HotFixPurchasedAirUnitTest (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1667 of 1669 tests passed)

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


Setiap perubahan di API C++ V8 akan ditangani oleh nan, dan akan ada perubahan versi minor juga. Sedangkan peningkatan versi mayor akan terjadi jika perubahan API terjadi di luar kemampuan nan.


Okay, I see. According to this link:
http://odetocode.com/blogs/scott/archive/2014/01/13/meet-the-angularjs-injector.aspx

You have to explicitly list the ng module if you are going to use any services from the core of Angular. Unlike the angular.module method, which assumes you have a dependency on the ng module and will silently add “ng” to your list of dependencies, the injector function makes no assumptions about dependent modules.


`console.log`


Additionally I propose to rename `core` to avoid misleading.


> Ok, I'll use a std::endl then. I'll do that in a new commit so that we don't lose this conversation, and anyway the two commits have different purposes :-)

Sure, np. :)


WHY?


Et en passant les requêtes directement dans le navigateur, il dit quoi le serveur Rest ?


`.main-visualisation--js` ?


Move this TODO inside the function. Comments outside of functions are docstrings.. Also:

// TODO (aznashwan): Implement this on centOS if necessary. For now, it implements the cloudcfg interface. (or something similar)


I should mention here that you need ``psycopg`` installed


Line 9 could be simplified to (no parens, no `.present?` call):
```ruby
if current_user
```


No construtor após sortearClubes vou colocar gerarRodadas, pra gerar a primeira rodada e sortear as partidas


This is my doing. Basically I'm not sure if the cfg passed is mutable by the receiving function.


买了一个梨


Reverted in ce03cd3661ee09127f42f33d8da7cdfa420d3a39


What about having it in tee_dispatch_open_session(): it is a generic function, and serves any architecture (not only TrustZone,


hi @jgcaap

is this work for CM12 ?


Why are you removing this method? Shouldn't it be kept for symmetry to `del_import`?


Remember to change `\"` by `'`


Device_g2m.mk should be com.cyanogenmod.keyhandler


:wizard:


Where is the NaN value coming from?


should we use a constant for links?


Use the association when its available. Look at the `Post` model to see its available associations.

One of `Post`'s associations is `has_many :comments`. This means that an instance of `Post` will have comments associated with it that are available by calling `.comments` on the `Post` instance.

This would mean line 11 here could be simplified to:

```ruby
  @comments = @post.comments
```

Similarly, `Post` has the `belongs_to :topic` association, meaning line 9 could be simplified to:

```ruby
@topic = @post.topic
```


初始化人员


Committed by accident?


CHECK


Excellent work


The [] setter is missing I think.


この変更で、マンガ投稿であっても画像が１つのみのものは、イラストと同じようにフォルダを作らず保存する。


"Logo grafis Web Platform Docs dan Web Platform" seems wrong. "docs" is basically a short form of "dokumentasi", isn't it?


Hi @skarbat, heads up:

Sorry I didn't notice this earlier, I only stumbled over the following when adding i18n to kano-wifi.

Using _ for throwaway values (line 114):

    _, _, _ = run_cmd('lxpanelctl restart')

can break gettext, where `_()` is commonly used as a global function and is being installed by `gettext.install()`. When using _ as a variable, this breaks the definition of _() for this scope.


@dclarizio @martinpovolny I need to review the @skateman's Patternfly PRs.


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1160 of 1162 tests passed)

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


lgtm


Apparently declaring scope in dependencyManagement is bad.


Keys should be added only if repositories was changed (to be idempotent).


No new line


We probably should reduce amount of mandatory packages and exclude vim, mc and curl from list.


Make lib Java 1.6 compatible


This should be done from php role.


@bjeuriss , just a couple of notes on that commit:

- I took out the `get_valid_DW_scheme()` call since the header should already have a DW_scheme by that stage, and we want to allow output even if the scheme doesn't match the image it's found in - i.e. I could use `mrinfo` on a derived FA image for instance and still allow it to dump information about the gradient table, even though I clearly couldn't use that information for processing the FA map itself. The `get_valid_DW_scheme()` function is meant to be used in applications where that gradient table is about to be used with the associated image - i.e. you're asserting that the image is a DW image.

- shells are now listed on the same line rather than one per line to allow multiple inputs

- I use a `"\n"` for newlines rather than the C++ standard `std::endl` since in my experience, allowing a mix of DOS & Unix newlines just messes things up at some point, especially in mixed environments. Besides, only the most retarded software on Windows (ahem, notepad) is unable to cope with Unix newlines...


Keys are usually offered by repositories, and you need to accept them. Check how this can be done non interactively.


Remove extra unneeded comments.


Hey,
    I studied the example and successfully connected MSP430 now for multiple device connection what configuration is needed for devicehive code?


I don't think we should have this additional method in `TraceProxy`. If there are no properties, well, let's have the user pass in an empty list. I see two disadvantages:
- An additional method in this contract. We usually want to minimize the number
- An optional argument, which is never great in a contract. Plus, it's not the last argument


asdfas


Yes. This should be a possibility. Remove this and leave a TODO if you don't plan to do this yet. Also, open a bug for this.


No


You did not resolve a merge conflict here


So, all these `flush` will have to go, as we discussed yesterday


kind of. We nee resiliency. If a yum install fails because of slow DNS, we need to retry.


Ah, sorry, this actually does this.


Good question - this is not necessary as `CommentsController#new` isn't used anywhere.

The place to put any authorization related code for a new comment is in `PostsController#show` as that's where a new, blank comment is initialized:
```ruby
class PostsController < ApplicationController
  ...
  def show
    @post = Post.find(params[:id])
    @topic = Topic.find(params[:topic_id])

    # Only initialize @comment if policy allows new comments. (Admittedly
    # there's not much to secure in this particular case):
    @comment = Comment.new if policy(@comment).new?

    @comments = Comment.where("post_id = ?", params[:id])
  end
  ...
end
```


just want to make sure that i understand this case correctly: are you talking about the case where `xio_connection_xmit_inl()` gets called twice but failed due to `EAGAIN`?


> What is the purpose of this extension? I read this as: You want to get callbacks whenever you published a transform. But I do not understand the purpose.

The purpose is to know when the robot state changes (to emit corresponding events). Usually one would subscribe to the ```tf``` topic and see if the joint states are changing to do that. But sometimes you don't need to do that. The general logic is the following: if I just commanded my arm to move and I trust my arm to be reliable I don't need to double-check with the sensors and I can just assume that my command got executed and the robot state changes. In our particular case it's more trivial: if I'm publishing transforms it means that my robot is moving, I don't need to subscribe to the ```tf``` topic to make sure of that. The publishing frequency needs to be taken into consideration here for certain reasons but I'm going to omit those details here.
If you think that this is not a general enough reason to put this code here, I can put it somewhere else. But then I'd appreciate it if ```send-transform``` where a generic function.

> I think the logic of with-transforms-changed-callbacks is flawed: Imagine the callbacks have been enabled before calling it. Then your macro will disable the callbacks upon finishing. Is that the desired behavior?

True, I messed up the exports. ```add-transforms-changed-callback``` and ```remove-...``` are supposed to be private.

> Making the topic latched is not correct. TF2 comes with an extra latched topic over which static transforms shall be published.

Ok, so how are you going to publish on a ```tf_static```? The point of latched topics, as we all know, is that all new subscribers get the last published package delivered to them. In that sense, it is the publisher who defines if the topic is latched and not the subscriber. In that sense, saying that "TF2 comes with an extra latched topic" is not exactly correct as ```buffer_server``` is only subscribing.


not sure what you mean. But on CentOS you only need yum -y install <package>. There is no --forceconfold if that's what you mean.


Good, this is the right spot for `authorize @comment`


not relevant on CentOS


Good, this policy is :+1:


Sorry for not being clear. What I meant in my previous commit, is that there should be no indent here.


XR_REG_ERROR_STATUS_BREAK is defined twice. Should one of them just be removed or is one of the just named incorrectly?


thx


Good, correct


Well done


Oh right! Thanks @hzulla ! in this particular case it's safe to remove them, i'll fix this. I'll keep an eye on those throwayas from now on!


make sure package names are correct. Do a yum search for these and use correct values here. Also make sure versions are correct. If now, we may need to ask for a 3rd party repo to be created.


83782874_LBNL_ThermostatOffsetFault (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1160 of 1162 tests passed)

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


Fixed it. Thanks!


Or alternative replacing
```
    private static void loadMtgInfoSetsMap() {
        final Properties prop = new Properties();
        try (final InputStream in = MtgJsonReader.class.getClassLoader().getResourceAsStream("mtginfoSetsMap.txt")) {
            prop.load(in);
        } catch (IOException ex) {
            throw new RuntimeException(ex);
        }
        for (final String jsonSetCode : prop.stringPropertyNames()) {
            final String key = jsonSetCode.toUpperCase().trim();
            final String mtgInfoSetCode = prop.getProperty(jsonSetCode);
            mtginfoSetsMap.put(key, mtgInfoSetCode);
        }
    }
```
with
```
    private static void loadMtgInfoSetsMap() {
            mtginfoSetsMap.put("TOR", "tr");
mtginfoSetsMap.put("TOR", "tr");
    }
```


79453838-AutomateSomeDocumentation (Myoldmopar) - x86_64-MacOS-10.9-clang: Build Failed

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


I looked at France translation also, they use "Web Platform Docs et le logo Web Platform font l’objet d’une licence Creative Commons Attribution, CC-BY 3.0 Unported."

If it's translated, that will be "Logo grafis Dokumentasi Platform Web dan Platform Web", which I think is inconsistent. Or do you have a substitute for "Plaform"?


ok, this should be implemented in packaging I think


Maybe we could add a test on the number of solution found? Otherwise, the test is useless


What's the rationale for this substitution?


this is related to package preferences which we might still need


Why not private variables?


LGTM


This patch fixed OpenLMI problems with python-blivet-1.1-1.fc23.noarch and LUKS.


Ça marche bien. Il affiche les données

2015-03-19 13:36 GMT+01:00 jcheron <notifications@github.com>:

> Et en passant les requêtes directement dans le navigateur, il dit quoi le
> serveur Rest ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Ka3l/my-open-beer-angular/commit/5843a04dd482c4cfc5e172bc23209eda91c9e065#commitcomment-10284194>
> .
>


Magic value "Pref", move to constant with significant name.


Magic value "LoggedIn", move to constant with significant name.


@sdt ah, perfect! I actually set it to false for testing to disable any readline functionality. thanks for the catch.


Use CamelCase and start variables with lowercase.


Most probably a policy change in Forge management - you should be privy to the details.


Not yet, we have to implement ELF parsing in druntime, not too hard though. I was just wondering why you removed this again.


Hi there, Ian - we've been chipping away today at the dockerfile, but as you would have seen, it's a big beast. Looks like you've got some logic there in your dockerfile which looks pretty useful. Because of the size (and the fact the current scripts are missing deps, etc) we're going to try an approach of splitting each of the servers (build and run) into their own images...provide a bit more flexibility. If you're interested, issue #26 has some more comments, etc. Luke.


UI strings should be always in strings.xml file.


Magic value "Email", "Haslo" etc., move to constant with significant name.


Actually, there is no need to return anything. We just need to clearly declare that show_problem_list_notification() and notify_problem_list() destroy their arguments.


あああああ


@kerbylav @mzhelskiy вот это как раз та ситуация когда не ``and`` нужен, а ``&&``


I think that this code could be placed in onStart() and code in onCreate() which do exactly the same can be removed. See http://developer.android.com/images/activity_lifecycle.png


Thanks for the reply.  I have only tested on one system so far, but your explanation is helpful.  I have no need to change this in @alvagante's source.  We have between 30-40 that run puppet but have no modules applied.  Since we're just diving into doing something with Puppet on Windows, I might just take the route of blocking the source permissions in my site.pp.


>поэтому стараюсь использовать операторы and/or

ты прошлый раз говорил с точностью до наоборот и я предлага как раз использовать and/or


CHECK


Fair point. I'll add that in. Just no object support atm - tricky to get that to work without doing an O(n^2) deep equality check or using slightly hacky approximations like sigmund


Your version of "STATUSBAR_INSOVR_TOOLTIP" looks better.


In activity lifecycle methods superclass method must be called as first method, not the last.


Yes, didn't notice those _ variable before, either.

There are some more usages of _ in kano_settings/, but none of them hurt i18n because there isn't any translation going on later within the same scope.

That only happend in kano-wifi, making it necessairy [to rename them there](https://github.com/hzulla/kano-settings/commit/22334d0b980ae3d165c4c0bde4bf2558c238c91a#diff-37a5e331a1c7fbff51b8692a9991f6eeL295). So if you trip over weird errors after adding gettext to your code, some old code using throwaway variables might be it.


Ser bra ut!


Why not a private variable?


The values in my test came from a long SPSA run. It is quite possible that the sugar author had also done an SPSA or similar run - since Sugar is a SF offshoot, it would not be surprising for similar values to emerge. Or he/she may have started from my values.

The bishop and knight weights are so small as to be ignorable, imho. The Q and R values are significant. A number of tests around this (not just the one I linked to) all either failed STC, or passed STC only to fail (as 0 Elo) at LTC. Basically the "badness" of blocking a passer with a heavy piece is detected concretely in search at LTC.

To answer your last question - I would always try the simple ideas first!


Magic string, see above comments.


> I was just wondering why you removed this again.

Oh, that's because it breaks the windows build and I don't have a `dependencies-posix` option


This revision is clean.


@gkalpak - you are right. This was probably leftover from a previous version of the PR. Feel free to remove and commit.


You should add empty constructor and newInstance method for all fragments. http://stackoverflow.com/questions/10450348/do-fragments-really-need-an-empty-constructor. And create them only with newInstance() method.


ConstructionsGridView (rpg777) - Win64-Windows-7-VisualStudio-12: OK (2170 of 2195 tests passed)

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


больше не буду


Why it is above super.onCreate()?


Always start method names with small letter.


Better use fragments from support library.


Web Platform is actually a name (https://www.webplatform.org/), therefore
there's no need to translate it.

It would be better it we just stick to something similar to mine, something
like "Dokumentasi dan logo grafis Web Platform..."

On Thu, 19 Mar 2015 20:00 Yolanda Septiana <notifications@github.com> wrote:

> I looked at France translation also, they use "Web Platform Docs et le
> logo Web Platform font l’objet d’une licence Creative Commons Attribution, CC-BY
> 3.0 Unported."
>
> If it's translated, that will be "Logo grafis Dokumentasi Platform Web dan
> Platform Web", which I think is inconsistent. Or do you have a substitute
> for "Plaform"?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yolapop/brackets/commit/4c6a0775dcee25e8e106644b3135f2a1479312ce#commitcomment-10284589>
> .
>


gif? pfttt no cli?


ok ı squashed the needed changes into single commit https://github.com/sirmordred/samsung-kernel-msm7x30/commit/6783ad61928d28a49c6adf3d7580ae3981057515


What if pref is null here?


I noticed it because I'm moving that file in another place inside, so it'll will complain about it ... without reason :D

Uomo avvisato ;)


We originally had communicate happening inside pdsh, but I moved it out so we could support having different clients decide if they wanted to block on the remote process.  I'm a little hesitant to remove that functionality, but perhaps in the end it's cleaner to just expect that anything we launch remotely needs to complete (via backgrounding it or whatever).  Certainly leaving too many connections open might not be ideal.  Thoughts?


Don't force activity to recreate this way, try to back to state before login.


the no. Not needed on CentOS


It is not useless as the test failed before the bug fix (weak propagation leading to unfixed variable).
Anyway, if you want to do even better, then go on! ;-)


Modul hinzugefügt


powiem o tym w sobotę trochę ale trochę do innych celów
na Twoje potrzeby starczy że napiszesz BaseController z kodem który jest współdzielony i dziedzicz po nim resztę:)


Para salvar la ShoppingList, deberiamos dejar esa responsabilidad al Provider:
```objective-c
[self.provider saveShoppingList:shoppingList]; // no el newDictionary
```
Dentro del provider deberia ir toda esta logica de crear el diccionario. El creator no deberia saber como se GUARDA el objeto ShoppingList. Al creator le da igual que sea en un PLIST, CoreData, etc. Por eso al provider le deberias pasar directamente el objeto ShoppingList y dentro del provider crear el diccionario y salvarlo en un plist.


@RobertoSanchezHerrera could you please review my code, please?


no crees aqui el ID del shoppinglist. Deberia ser el creator quien lo crea. Pero ni siquiera necesitas ID.


I only finished the last part from yesterday commit.


Esto esta bien


remember I told you about how to implement a dict


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


Congratulations! For now you may use several gateway instances to connect
to each MSP430.
Please mind the different device IDs for different MSP430 boards.

2015-03-19 15:51 GMT+03:00 thakorjay <notifications@github.com>:

> Hey,
> I studied the example and successfully connected MSP430 now for multiple
> device connection what configuration is needed for devicehive code?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/devicehive/devicehive-cpp/commit/3319387fab4cee5747676aa5bf61924a8ef893cc#commitcomment-10284438>
> .
>


@bloodbare pdb? :)


Como te he comentado antes, aqui no deberias CREAR un Chain. Deberias seleccionar uno desde un listado y pasarselo al metodo addChain:
En un comentario anterior te he comentado como deberia ser creado el Chain.


I would change `other.getAttributes().get(name)` to `other.attributes.get(name)` so a new UnmodifiableMap for attributes is not created once per loop iteration.


恩>_<

On Thu, Mar 19, 2015 at 9:18 PM, Yuxin Wu <notifications@github.com> wrote:

> remember I told you about how to implement a dict
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ymy513/leetcode/commit/387991db8a898b637daf4761c56f7dfb61b07b87#commitcomment-10284934>
> .
>


Waaooowwwwwwww...............................


I'm might be stupid but I don;t get why the skill.c change with if(), elseif elseif ain't integrated with the switch( ud->skill_id) of line 10474.


I have a few questions about this:
1) Why only these classes? There's plenty of other internal stuff in this module related to properties that is not supposed to be a public API.
2) Could this be handled on the other level, e.g. via a configuration parameter to Dokka which packages to skip? In our case that would be `kotlin.internal`, `kotlin.jvm.internal`, `kotlin.reflect.jvm.internal`. Otherwise I'm not sure whether I should write `@suppress` if I add new code to these packages or not.
3) `@suppress` is confusing to me and almost looks like the warnings are suppressed, not the documentation (`kotlin.suppress`). Why not `@suppressDocs` or similar?


Dois layout


Does linker report duplicated symbols?
Interesting why it never happens for static library.


No to czas na refactoring ( ͡° ͜ʖ ͡°)


Do you have any good names in mind?


This will make your apache config corrupt if you have commented out a Directory entry or remove password protection if you have Directory entries with password protection.

Not sure why we would want that.


Thanks. Well I thought exposing it as an object would be more "future proof" (extensible). Exposing the `Query` class would be useful as well... I will try again later.


nice catch!


I would add some case insensitive cases to this test, besides the ones already there:

    the(prgFromDB.dirtyAttributeNames()).shouldContain("First_Name");
    the(prgFromDB.dirtyAttributeNames()).shouldContain("LAST_NAME");


16: M102 unrecognized bullet/signature
17: M102 unrecognized bullet/signature
1: M111 unrecognized component name: WebSubmit
3: M190 line is too long (74 > 72)


Very nice , thanks for your effort work's perfect !
Keep up the good work!


Holy crap! How did that happen 0_o Thanks a lot. Wasn't aware of that.


86046200_CondenserToZone_CherryPick (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1123 of 1123 tests passed)

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


Also, please add a case insensitive test like this:

    Programmer prg = new Programmer();
    prg.set("FIRST_NAME", "John");
    prg.set("First_Name", "John");
    prg.set("first_name", "John");
    the(prg.dirtyAttributeNames().size()).shouldBeEqual(1);


@ioanpocol can you take a look at the changes in this PR?


once more line needed :)


I'm using the second option for now - doing my head in!

Thanks for having a look, glad it wasn't just me! All a bit beyond me.


89224666-FixNANProblems-Issue4557 (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


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


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1158 of 1160 tests passed)

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


checkinterval is in seconds


Why? It's not been released yet; have we done that warrants a minor bump?


@agorrod, ouch, sorry -- thanks for fixing that!


That sounds right.


No can do as attributes is private.
I will do
Map<String, Object> otherAttributes = other.getAttributes();
and iterate over otherAttributes map.


I didnt get exactly "Different gateway instances" . Will you elaborate ?
 On 19/03/2015 6:49 PM, "Sergey Polichnoy" <notifications@github.com> wrote:

> Congratulations! For now you may use several gateway instances to connect
> to each MSP430.
> Please mind the different device IDs for different MSP430 boards.
>
> 2015-03-19 15:51 GMT+03:00 thakorjay <notifications@github.com>:
>
> > Hey,
> > I studied the example and successfully connected MSP430 now for multiple
> > device connection what configuration is needed for devicehive code?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/devicehive/devicehive-cpp/commit/3319387fab4cee5747676aa5bf61924a8ef893cc#commitcomment-10284438
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/devicehive/devicehive-cpp/commit/3319387fab4cee5747676aa5bf61924a8ef893cc#commitcomment-10284952>
> .
>


oi queria amiga,,, fico feliz de ver vc programando como um anjinho!!!11! abs na familia


Looks good!  I think eventually we should follow pdsh's tradition to also make it so you can specify wildcards in the yaml for whatever labels you want, ie:

"/dev/sd[a-f]"

or

"/dev/disk/by-label/cbt-device-[0-6]"

This is a good step though!


You can! It is private but you're accessing it from the same class. Try!  :- )


Seems like this was enough


We have always bumped the number after a release.


>- pod install --project-directory=Tests

Is this needed?


Load time for "api/app/views/spree/api/products/show.v1.rabl":
10 products: 7996ms (Views: 7755.6ms | ActiveRecord: 232.3ms)
30 products: 21802ms (Views: 21198.6ms | ActiveRecord: 595.4ms)


Oh, yes :-) thank you :+1:


IDK if classes PerspectiveActivity and JSWorkbenchPerspectiveActivity are still used?

I only ask as they're contained in the IOC BeanManager lookups you've deleted above.

Possibly PerspectiveActivity is no longer used, but JSWorkbenchPerspectiveActivity remains for statically defined Perspective Definitions?


I see, is it OK if I release a patch release?


yes, it show where is the project for the pod is located. by default it is root.


- Colors changed a bit after I removed a forked repo I had contributed to. This removed an entry from my chart.
- This demonstrates that the color distribution is sensitive to other datapoints already on your chart.
  Experiment to find a good balance!


Also, I think almost all our releases have been warranting a minor, actually :p


also it's needed to install pod if you have ignored it


1) These are the only classes that have public visibility. All others are package-protected, and are excluded from the documentation because of that.
2) Yes, it could, and it's likely that it will be implemented later.
3) I think we've already settled on @suppress and I would prefer not to do any last minute changes to it.


From kumajaya/android_kernel_samsung_lt02, adapted for goya boards


I'm not sure about this. Issue 630553 has a problem with this feature because the mail contains dynamically-generated content. They are set in stone by this button and no longer dynamic. That being said, I don't think the feature is bad overall and should be removed entirely. I think the button should only be hidden if the mail has dynamic content, but of course it may not be possible to define such a condition. So here are the possibilites:
1. Hide the button only if the mail contain dynamically-generated content, if it is possible
2. Hide the button entirely, if the issue is so bad that we prefer to drop the feature entirely
3. Don't change anything, if the issue is not bad enough to drop the feature entirely


Sure go ahead.


MMAPS generator is not auto set to true. There are some errors while compiling with gcc. VS (Windows) compiling is fine.


Cool, 0.26.1 with support for web-fonts then, coming along now.


bumping @Discover- : do u have some time to update the spell.sql for alle spells in cata? would be really nice to allow the propper generation of the comments :)

currently it's like this:
Burning Blade Reaver - In Combat - Cast '<Spell not found!>'


:racehorse: "(\r|\n| )+" -> "^(\r|\n| )+"


Do you have a Windows machine ? Or do you want me to push it to nuget.


wow, very cool explanations :+1:


MISSING COPYRIGHT


Hmm... No I don't have a Windows machine...


Good catch. Easy to miss these.


JUST USE BRACES


No problem - please do commit any changes you make. When I have some free time I will look at trying to configure the project so it is IDE agnostic.


Ok you push. Right now then?


:question:
Esse método vai receber o Pedido e mudar o estado dele de a para b, é isso?


From kumajaya/android_kernel_samsung_lt02
Needed to build with Ubuntu/LinaroToolchain (default arm toolchain in Ubuntu/Linux Mint)


MISSING NEWLINE AT EOF


The word-wrap support has been implemented correctly in https://github.com/flyingsaucerproject/flyingsaucer/commit/9b0c5cbffa46e38b2f3d66e3c97d86518598ac45 and is still available.
On top of the duplication, this one here is wrong, because the CSS specs say 'An unbreakable "word" may be broken at an arbitrary point if there are no otherwise-acceptable break points in the line.' (http://www.w3.org/TR/css3-text/#overflow-wrap-property), but here is no attempt to make a "normal" break.
IMO this should be reverted.


Please remove square brackets to prevent markdown parser making a link.


OK

On Thu, Mar 19, 2015 at 9:42 AM, Henrik Feldt <notifications@github.com>
wrote:

> Ok you push. Right now then?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SuaveIO/suave/commit/b68acc9851f91f2ce9f20e2d9f7e2189841d5b8d#commitcomment-10285363>
> .
>

--
Ademar Gonzalez
+1-647-891-3606
http://ademar.name
https://github.com/ademar


Im Falle von 5,acg7 bzw statt "acg" einen anderen nicht numerischen Wert, wird durch format zu 5.00 geparst.
Das ist kein Fehler aber unschön. Ich suche derzeit nach einer Lösung


why is it reverted? it has not been tested or there are errors?


Looks good until we get this fixed upstream.  Created http://tracker.ceph.com/issues/11165


So V4 coming soon for lollipop? Nice :D I will try/adapt it on AOSP ;)


Thanks @anlutro, will amend/fix that on the next commit! :D


Changed testing Attractors for inputs instead. Tests full flow of adding Attractors.


> I'm might be stupid but I don;t
Sorry, just kidding.

Btw, thank you guys for your work. You really do awesome things.


Stimmt, vermutlich erst das falsche angepasst... Passe es gleich an.


These were not actually supposed to make it into the commit.  Will remove.


1: M111 unrecognized component name: BibDocFile


Problème de z-index


그리고 이것도 그냥 db.Model 로 만들어주세요


```  depends_on "homebrew/science/opencv" => "without-brewed-numpy"``` should work, however @ReDetection is reporting that it has no effect.


Any reason you want to make the change?  For very large clusters pdsh may be siginficantly faster for deploying the conf file as it can execute the transfers in parallel.  Not saying this isn't right, just not sure what the rationale is?


Increased debug level


> ты прошлый раз говорил с точностью до наоборот

вполне может быть )


pairing with @flmmartins


아 미안한데 앨범에 여러 아티스트들이있을수있으니까 artists가 맞을거같습니다 song도 마찬가지 ㅠㅠ ㅈㅅㅈㅅ


That looks a bit cleaner way to do things to me. Thanks!


여기에 polymorphic identity는 artists 가 되던가 해야합니다


W.I.P


Hi Scott,

When you are making big changes like this could you give a reason why this has to be. In the commit, in order for us to know why this change is necessary.


Ok, cool, will do - just working out what sets to convert and what to ignore.


Why BETA1 to DEV ?


thx


Hier gibt es noch einen Fehler. Wenn kein numerischer Wert eingetragen wird, steht ind er Ausgabe statt zum Beispiel 5,asdas7000 -> 5,00 erwartet wäre, dass einem der Test um die Ohren fliegt.

Here is an failure. If the String contains any not numeric chracter, the sysout won't work. Expected was a failure but the output is 5,00


Indeed, a more general version can be created that just returns the query result.
But you need to prepend with "room_" so I allowed myself to make something custom


[Deployment available online](https://s.brightspace.com/apps/ipa/dev/03c3604e150ae48243f956d00a87c1eb223d0b03/app.js)


Nice.


Debugging on lines 60, 61 can be removed


`unzip()` here will cause problems when purrr is loaded ;-)


`wait_for_ajax` shouldn't be needed


@amenophis the branch is always a dev version.


s


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


It was closed because an explanation was provided for your arguments (which were mostly invalid).


Minor thing, but whitespace seems off here if you don't mind fixing that.


Check `days_left` returns `7` when an `Item` is first created - this doesn't look like it will.


Done.


Confirmed, this works with v7 worlds with teleporters in it.


Or we should use the `createDistribution` method (which is available on the WorkspaceImpl).


why '|' for dividing? ;) :smile:


Or we should use the `createFeature` method available on WorkspaceImpl.


I guess it's just strange to me to see numbers in quotes. I'll change it back.


@mzhelskiy На мой взгляд привычка тут не совсем то слово... просто при использовании && и || результат всегда именно тот, который логично проистекает из функций. А при использовании and/or - могут быть нюансы, что очень плохо.


rest in pieces


  run first gateway instance:
./simple_gw --serial /dev/ttyUSB0 ... to connect first MSP430
   at the same time run an another gateway instance:
./simple_gw --serial /dev/ttyUSB1 ... to connect second MSP430

2015-03-19 16:34 GMT+03:00 thakorjay <notifications@github.com>:

> I didnt get exactly "Different gateway instances" . Will you elaborate ?
>
> On 19/03/2015 6:49 PM, "Sergey Polichnoy" <notifications@github.com>
> wrote:
>
> > Congratulations! For now you may use several gateway instances to connect
> > to each MSP430.
> > Please mind the different device IDs for different MSP430 boards.
> >
> > 2015-03-19 15:51 GMT+03:00 thakorjay <notifications@github.com>:
> >
> > > Hey,
> > > I studied the example and successfully connected MSP430 now for
> multiple
> > > device connection what configuration is needed for devicehive code?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/devicehive/devicehive-cpp/commit/3319387fab4cee5747676aa5bf61924a8ef893cc#commitcomment-10284438
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/devicehive/devicehive-cpp/commit/3319387fab4cee5747676aa5bf61924a8ef893cc#commitcomment-10284952
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/devicehive/devicehive-cpp/commit/3319387fab4cee5747676aa5bf61924a8ef893cc#commitcomment-10285194>
> .
>


@legaultpierre:
- W.D. Montgomery: Charles River Associates, USA
- E. Krause: Lawrence Berkeley National Laboratory, USA
- A. Pegov: Institute for Systes Studies, Russian Academy of Sciences, Russian Federation


Please position this alphabetically, after Nanograms.


Yeah, that's correct. If a specific package is specified, then it can be built. This is also true for there blacklist.


Sure.


89224666-FixNANProblems-Issue4557 (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (652 of 654 tests passed)

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


Ok.


> Why not call the env variable AGDA_BIN like in the Makefile?

Done (see https://github.com/agda/agda/commit/bec378b20b3f77e0dd6ac73f9573972f90a2d46c).


Korrekt. Hatte ich aber wohl später selber korrigiert, taucht jedenfalls nicht mehr auf:
https://github.com/meyway/wisy/blob/html5valid/core20/wisy-edit-renderer-class.inc.php
https://github.com/meyway/wisy/blob/html5validpluswiki/core20/wisy-edit-renderer-class.inc.php
https://github.com/meyway/wisy/blob/html5validpluswikiplusiso885915/core20/wisy-edit-renderer-class.inc.php


@legaultpierre:
- K. Goran-Maler: Beijer International Institute of Ecological Economics, Sweden
- H. Jacobson: University of Michigan, USA


Wait a minute, I see that `ValidFile` has also been removed. The CRAB3 code was relying on that..


i have problem about some skill WS_CARTTERMINATION and NJ_ISSEN
It cannot reduce by Thera frog card in pvp mode


remove features views


@legaultpierre:
- R. Richels: Electric Power Research Institute, USA
- R. Tol: Vrije Universiteit Amsterdam, Netherlands


这脚本你现在还能用啊。。闪讯升级后都失效了，除了个别高校和个别校区，所以项目就没维护过了。。MW300R v3失败具体是怎样的，运行脚本后路由器有拨号行为吗？人工复制成功率很低，动态加密的用户名5s就变了，而且前面有\n\r有些路由器的配置页面不支持转义。虽然脚本已经对水星和tp做较大的兼容了但毕竟有限，身边没有这款路由器我也不能确定是什么原因，可以尝试下这个http://pan.baidu.com/s/1eQ6Czmy


@traversaro :
I'm not sure of:
- should I provide a "disable limit check"? What do you think?
- coordination.. but maybe I've just had an idea.


@elfring Tested and it didn't break anything on my system, added here: https://github.com/Revolutionary-Games/Thrive/pull/305


sscsa


Some frameworks make it hard to Do It Wrong, others are designed to Just Work. Rx (IMHO) is designed so it's hard to Do It Wrong....although with Rx proper it's super easy to do wrong things.

This type of feature seems more inline with making something work as you would expect; I believe this is something I'd be happy to see in RxUI.

One question; when would you ever not want to Marshal bindings to the UI?


zsmalloc is arch-independent on stock sources, just need to tweak Kconfig's


I don't know :P. Thats just momentary :P


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.47](http://teamcity.monogame.net/viewLog.html?buildId=5794&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 460, ignored: 6 Build time: 00:08:42
### Failed tests
```
MonoGameTests.exe: MonoGame.Tests.Visual.SamplerStateTest.VisualTestComparisonFunction: <no details avaliable>

```


CHECK


bro, do you even Makefile!?


"Wie krieg ich dies in Github smart übertragen?"
Du meinst, wie bekommst Du einen einzelnen Commit übertragen ohne die anderen HTML5-Anpassungen?
Wenn es nur um diesen einzelnen geht, würde ich sagen, übernimm das händisch, ggf. mit dem Github-Editor online - oder ich erstelle noch mal eine Branch mit nur dieser Änderung, basierend auf dem aktuellen 5.73 - und Du übernimmst nur das vor allen anderen Änderungen?


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


This should fix the failing job
http://zusa.demo.cfpb.gov/view/OAH/job/OaH-build.cf.gov-API-tests/156/

Part 1: modified Jenkins job to point to  http://build.consumerfinance.gov/oah-api/rates/rate-checker
Part 2: modified assertion in step definition to reflect the updated error string that the API returns when a parameter is left out


@mzhelskiy  Ну вот я и говорю что выше нужно исправить на &&


twitter is blocked in my school ;-;

I have a proxy for it, but the mobile version only comes up. D:


print ("\nDo you think you are smart? No one can beat me. I'm Batman! But normally people get to call me Sheldon. Anyway, there is no way you can beat me here. But you decided to challenge so let's challenge. First, who are you?")
nm = input ("My name is ")
print ("\nOkay, %s. I see why you're here. You're thrilled for challenge and you are trying to beat Batman. That's impossible but I admire your courage. So shall we begin."%nm)
reply = input("Are you ready? ")
yes = "Hell yeah sir!"
if reply == yes:
    print ("\nGreat! Now that's good! So let's move on. I'll now give you game instructions. You may choose stone, paper, scissor or spock. I will choose one of these. Upon choice is made, one of us will win. This will certainly be me, but anyway. Rules are: scissor cuts paper. Paper covers stone. Stone crushes lizard. Lizard poisons Spock. Spock smashes scissor. Scissor decapitates lizard. Lizard eats paper. Paper disproves Spock. Spock vaporizes stone. Stone crushes scissors. Do not worry if you can't remember this all. I'll remember you as we go. Shall we begin...")
while reply != yes:
    print("\nThat's not good enough. I wanna see enthusiasm! Either you go home now or shout 'Hell yeah sir!' (case sensitive)")
    reply = input ("Are your really ready? ")
print("\nBEGIN!")
user = 0
sheldon = 0
ans = input("You may choose stone, paper, scissor, lizard or spock: ")
s = "scissor"
p = "paper"
r = "stone"
l = "lizard"
k = "spock"
import random
options = [s,p,r,l,k]
cans = random.choice(options)
while ans == cans:
    options = [s,p,r,l,k]
    cans = random.choice(options)
    ans = input("\nWe chose the same... Stop imitating me and play with honor! Choose once more! You may choose stone, paper, scissor or spock. ")
while sheldon!=user+3 and user!=sheldon+3:
    options = [s,p,r,l,k]
    cans = random.choice(options)
    if ans == s and cans == p:
        print("\nScissor cuts paper. You won this time, %s, but won't again!"%nm)
        user = user+1
        sheldon = 0
    if ans == s and cans == r:
        print("\nStone crushes scissor. Cry in deep agony, %s!"%nm)
        user= 0
        sheldon = sheldon+1
    if ans == s and cans == l:
        print("\nScissor decapitates lizard. I am going to lose my head because of you, %s!"%nm)
        user=user+1
        sheldon=0
    if ans == s and cans == k:
        print("\nSpock smashes scissor. HAA! Life long and prosper is not for %s!"%nm)
        user=0
        sheldon=sheldon+1
    if ans == p and cans == s:
        print("\nScissor cuts paper. And your score has just been cut!")
        user=0
        sheldon=sheldon+1
    if ans == p and cans == r:
        print("\nPaper covers stone. You're covering a shadow in my scoooore! Quit thaaat!")
        user=user+1
        sheldon=0
    if ans == p and cans == l:
        print("\nLizard eats paper and #%sisalooser"%nm)
        user=0
        sheldon=sheldon+1
    if ans == p and cans == k:
        print("\nOkay, paper disproves Spock and I am disproved. #%swins"%nm)
        user=user+1
        sheldon=0
    if ans == r and cans == s:
        print("\nIt was getting boring and you deserve this point. Let's make it more interesting now. We're just getting heated. Now we will play it serioysly. #give%sapoint"%nm)
        user=user+1
        sheldon=0
    if ans == r and cans == p:
        print("\nThis is how we play. Your score is back to zero and mine is +1 closer to win! Paper covers stones. Haaaaaa")
        user=0
        sheldon=sheldon+1
    if ans == r and cans == l:
        print("\nGood Lord! That was luck! Won't happen next time. Stone crushes lizard and you get one point")
        user=user+1
        sheldon=0
    if ans == r and cans == k:
        print("\nI SPOCKED YOU! Spock vaporizes stone!")
        user=0
        sheldon=sheldon+1
    if ans == l and cans == s:
        print("\nGuess what? Scissor decapitates lizard. You must have a head if you want to score anything! Zero head, zero point for you!!!!")
        user=0
        sheldon=sheldon+1
    if ans == l and cans == p:
        print("\nLizard eats paper. Gosh, you were hungry this time, uh? Take it easy and eat your lunch properly next time! No need to go eating everything around here! But you'll never eat my Amy!")
        user=user+1
        sheldon=0
    if ans == l and cans == r:
        print("\nStone crushes lizard and I am your crush! <3")
        user=0
        sheldon=sheldon+1
    if ans == l and cans == k:
        print("\nLizard poisions Spock. Spock was going to have a life long and prosper until you poisoned him early March 2015! YOU ARE A SERIAL KILLER! POLICE!!!! HELP!!!!!!!!!! I'll tell 911 %s killed Spock!"%nm)
        user=user+1
        sheldon=0
    if ans == k and cans == s:
        print("\nSpock smashes scissor and you smashed me. Fair. Spock is awesome!")
        user=user+1
        sheldon=0
    if ans == k and cans == p:
        print("\nPaper disproves Spock. You're disproved, my dear!")
        user=0
        sheldon=sheldon+1
    if ans == k and cans == r:
        print("\nSpock vaporizes stone and you vaporized me. Fair. Spock is awesome!")
        user=user+1
        sheldon=0
    if ans == k and cans == l:
        print("\nLizard poisions Spock. I poisioned you. Feel the fury of my fury.")
        user=0
        sheldon=sheldon+1
    if ans != s and ans != p and ans != r and ans != l and ans != k:
        print("\nNEIN NEIN NEIN NEIN NEIN! This option does not exist! Please choose between stone, paper, scissor, lizard and spock!")
    ans = input("Either of us shall win! Choose stone, paper, scissor, lizard or spock in your own honor! ")
if sheldon==user+3:
    print("\nYou lost the challenge! Game is over for you! A %s cannot beat a Batman!"%nm)
if user==sheldon+3:
    print("\nWHAT? YOU WON THE CHALLEGE!!! You're the new Batman! Game over to me! Congratulations, %s!"%nm)


Yes, I think it was lost in a rebase. It should not matter because it will always be out of sync with the true commit id. All scripts should run `scripts/dist/common/prepare_release.py` that sets it to the true commit id.


This formatting is still not in alloy's patterns, as I mentioned on the other pull request. The correct formatting for calling a function with multiple arguments of which one is an object would be:

    this.fire('create', {
      field: field
    });

Can you update this call, as well as other similar ones on this file?


17: M102 unrecognized bullet/signature
18: M102 unrecognized bullet/signature
1: M111 unrecognized component name: WebSubmit


не ключ, а урл


This request is no longer necessary since #148 has been merged. This pull request can be closed.


Congrats, well done and good job:).


Nice, simply fix :smile_cat: Thanks!


I added a new commit changing a little bit how I fix that. I can squash the two commit if you want


bel lavoro ma ci sono fin troppe linee di codice inutili, potrebbe creare problemi, farò dei test poi ti faccio sapere.


`=== null`
is actually the more precise variant, since otherwise a difference of numerical `0` would also evaluate to false.


This change speedup some tests. e.g. on my machine (linux 64 bit):
runtests.py (20 tests): 13.0s -> 8.2s
charmap.py: 13.5s -> 0.43s


他有没有文档提到“检测账号的账单”的?可能你用的那个sxplugin.so是重庆版的也说不定，江浙的不一样


I would require the opinion of @tde-banana-odoo on this, since he suggested to me to remove the button :)


Rafiel- Edited the wrong page, was meant to be working on the Bournemouth page and has just used the code made by me and deleted the images and the jquery. Will replace with the original so he can do the right page then will link them


я про это


This commit removes the base.scss file because it included redundant css that is already included in Foundation.


needless now


Я предлагал так:
```php
$iPostfix = 0;

do {
  $sUrl = $sSlug . ($iPostfix ? '-' . $iPostfix : '');
  $iPostfix++;
} while ($oTopic = $this->GetTopicBySlug($sUrl) && (is_null($iSkipTopicId) or $iSkipTopicId != $oTopic->getId()));

return $sUrl;
```
Здесь нужно не ```and```, а ``&&`` либо взять в скобки ``($oTopic = $this->GetTopicBySlug($sUrl))``


OK, thanks, fixed.


Ahh so that's why you have so long in between tweets.
On 19 Mar 2015 14:09, "xxFlare" <notifications@github.com> wrote:

> twitter is blocked in my school ;-;
>
> I have a proxy for it, but the mobile version only comes up. D:
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/xxFlare/SimpleAuth/commit/900bc41ebe47e7f54747e3146a2fd467a878bd06#commitcomment-10285860>
> .
>


Там было:

Поставить ```%title%``` перед ```%id%```, и **поиск топика будет идти по ид**, а для пс будет красивый урл:
```php
%year%/%month%/%day%/%title%-%id%.html
```


Cheers, but there may have been complications.


Я тебе выше писал - твой вариант НЕ рабочий
Работать он будет только со скобками (для &&), для and скобки можно не ставить.


And just found out that it looks like the json file uses NMS instead of the official NEM for Nemesis (Which magiccards uses 'ne' for)

Gonna have to check the json too :disappointed:


@legaultpierre:
- unfortunately, no institution, so mark "N/A"


Кто-то из нас путает приоритет операторов && и and


See #105


HotFixPurchasedAirUnitTest (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1667 of 1669 tests passed)

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


Looks good!


@legaultpierre:
- K.B. Bartlett: University of New Hampshire, USA
- J. Laine: University of Helsinki, Finland


The rule is: include them in markdown files, don't in API docs. The reason is that API docs don't get rendered in places like GitHub, but md files do.


Good work! I must say I'm very suprised this passed because of the huge values for the queen. These values are increased further by the "Weight[MOBILITY]".
Just noticed something strange. The last values for queen endgame mobility are 99, 100, 111, 112. I guess 99, 100, 101, 102 was intended?


а если поставить только ``%title%``, то поиск будет по урлу, что чуточку дольше чем по ид, поэтому предлагаю вставить и тайтл и ид


Догадываешься кто? )
У && приоритет выше чем у "=", а у and ниже


I am trying to revert my change back to its original state...


Sean...
Its complicated on android, due to some poor implementation on some devices.
On android, seems that EGL needs GLES2 for some reason, so we have to link GLES2, even though we are only using GLES1.1.

And it we want FBO's on GLES1.1, then we need to explicitly load the OES FBO extensions to be sure that we don't get the GLES2 methods by mistake.
Note also the method naming convention is different for the OES extensions, so there will be a different codepath anyway.

This might get cleaner if I move android to GLES2, but I want to minimize the scope of the changes at this point.  Shaders is a whole new area that I want to avoid until we get some more experience with the platform.

Right now,  I have three android devices for test, and they all respond differently to GL initialization.  I found it best to be brutally explicit.  I expect more breakage in the field....

Feel free to hack away with conditionals to get the embedded linux GLES stuff working.  We can simplify once we get everything working, no matter how ugly conditional it may become short-term.

I'll try to verify your pull for CMakeLists today.

I find that the Mali400 hardware performance (compressed tex) under GLES1.1 is not too bad, so it is worth the effort.

Dave


Te sobran unas comillas, no?


@legaultpierre:
- J. Innes: you can enter him as John Innes, Swiss Federal Institute for Forest, Snow and Landscape Research, Switzerland


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


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


Why not make this a HASH?


дальше пусть пользователь сам настраивает как ему нужно


... for_class_**NOT**_in_compilation_unit, you meant?


@hedgerwang This is exactly the solution that document suggests in the `Note` section


Поставь какой-нибудь коментарий, чтобы потом не забыть поменять stHero


badiya hai bhai


Omg, the YAKS!!!


If it's only to do that, you might as well use:
g_dbus_connection_call_sync()
for which we already have a connection, and won't leak a GDBusProxy.


@legaultpierre:
- R. Lal: is actually Rattan Lal, Ohio State University, USA


frist


This should have been left there.


Could we populate pi->command_line instead?


Congrats!
This values passed fast, so i hope this gives a good amount of elo.


hi @dorimanx

the OPTIMIZED will work for NExus5 ?


that is the nature of human existence, the myth of sisyphus and all of that :)


develop (macumber) - i386-Windows-7-VisualStudio-12: OK (2176 of 2195 tests passed)

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


@legaultpierre :
- Unfortunately no institution for this person; mark as "N/A"


OK, thanks


Do you mean you will update it again before release? The hash being one or two commits off, especially when these commits are "release stuff", is fine, but not being able to find it...


"stage1ID" can be a constant, so it would not need to be repeated in ``stage0/run.go``


I just wanted to use 4 lines of code I wrote. :crying_cat_face:


Tried to fulfill Issue #10.
Variable names may now contain every character except:
- dots and commas, question marks and exclamation marks (2abb222), because these characters are usually marking the end of a sentence (and such should end a variable, too).
- dollar sign `$`, because it marks a variable. This allows two variables to be placed next to each other, without any character in between.
- new line and tab.

And a variable name must contain at least one character.


1. Could you rename things a little bit ?

- "FloatOn" to "FloatTrigger"
- "FLOAT_ON_VALUE_PRESENT" to "SET_VALUE"
- "FLOAT_ON_FOCUS" to "FOCUS_CHANGE"
- "flw_floatOn" to "flw_floatTrigger"
- "value_present" to "setValue"
- "focus" to "focusChange"

2. Can you invert the logic to check the floatOn value ? I mean instead of:

```
if(floatLabelOn == FloatOn.FLOAT_ON_FOCUS) {
  // Your code to float/anchor the label here
}
```

Make it:

```
if (floatLabelOn != FloatOn.FLOAT_ON_FOCUS) return;
// Your code to float/anchor the label here
```

This style saves some indentation.


我又试了一下，好像想明白这个问题了。英文的单词之间本来就有空格，所以可以分辨。中文在前面加上空格之后，parse 到这里就会分辨出来需要斜体，所以后面有没有空格都可以识别了。

我还是把空格加上了，和其他地方保持一致吧，可以再搞个全书格式说明。


@bradwilson out of interest, why did you make this method public? It seems at odds with the patten used by all the runner types, which have similar virtuals as protected.


`mono tools/paket.exe update nuget suave` works fine here with Paket 0.28.8


@legaultpierre:
- S. Diaz: is actually Sandra Myrna Diaz, Universidad Nacional de Cordoba, Argentina


Do we want to bounds-check here like below? (isNaN, 0<hours<24, 0<minutes<60)


using 0.33.15.0 here


@scytacki Thanks, Please let us know if this(https://github.com/concord-consortium/lara/commit/22dda539dd91abf671005dbd1dda96202be1853e) works for you


get '/irfan/help'    => 'static_pages#help' what will happen here?


Just a heads up that this is 100% something we don't need to do; total cargo cult that not even Rust cares about. Fine to do it, though.


+1 excellent


Do we even need these headers on the files at all?


I chose `OutOfNNException`, because `test_empty_namenodes_autoclient` asserts `OutOfNNException` to be thrown when no configuration is present.

Another option, towards which I'm leaning right now, would be to return an empty list here. This would be very similar to how it has been by now (not entirely similar, because in case of `HADOOP_HOME` present, but no configs present, `Exception` is thrown). With an empty configs list returned here, the `AutoConfigClient` will throw `OutOfNNException` anyway.


Apache license seems to want them.


This one also should not be missing. I will end this therad here.


@DennisMenjivar Porque subio este commit a MASTER!! :angry: Se armo un solo macaneo. Con las ultimas cosas que habia que darle sincronizar.
Tengan mas cuidado al momento de subir cosas!


@legaultpierre, all of these authors should already be listed as authors in the AR3, WGI. To the extent they have an institutional affiliation, it should already be included in the database for this AR3. Not sure the easiest way to pull them up, but there should be an easier way to do this. Essentially we need to go just author by author in the data base and associate a new "participation" to each one of these people for AR3. We can discuss this one.


This thing only takes effect if they don't have that annotation property at all. You can still specify it as Empty if you really want to. Uea that's not in this specific code, its in a slightly later commit.

Also, this change was made atvyhe express request of lex, if you want it removed, talk to him.


je suis 100% dac


@adamretter , are you sure? I still get

19 Mar 2015 15:32:04,572 [main] INFO  (JettyStart.java [run]:141) - [Operating System : Mac OS X 10.10.2 x86_64]
19 Mar 2015 15:32:04,572 [main] INFO  (JettyStart.java [run]:146) - [jetty.home : /Applications/exist_dev/tools/jetty]
19 Mar 2015 15:32:04,573 [main] INFO  (JettyStart.java [run]:148) - [log4j.configurationFile : file:/Applications/exist_dev/log4j2.xml]

org.exist.EXistException: LOG
	at org.exist.storage.BrokerPool.initialize(BrokerPool.java:1060)
	at org.exist.storage.BrokerPool.<init>(BrokerPool.java:716)

on develop-f056806 at startup.

I don't worry about this (because for me it's a excuse to take a rest), but eXist won't start up at all. The new log4j2.xml file is there alright ….


Is this just an example file for people to install manually, or are you planning on adding an install rule for it to the makefiles?


Let's introduce a `dedupeDependencies` field ;-)


Я :disappointed:


Please fix this ASAP!


Hoooou yeah !


"CVE of the month" ahah


Sorry, not sure I understand. Can't you just create an ``NMS=ne`` mapping? The set code is only important in the sense that it is used to generate the image url. I don't think it has any relevance beyond that. Then again, I might be missing the point entirely!


Oh, wow. Yeah, that code was totally broken. Also, we no longer need to keep this O(1) since it only gets called when a string is created. Did the simpler thing and just hashed the whole string:

https://github.com/munificent/wren/commit/5e1ddb7db55fb9de1f3b51a1d092d1d0aaa2b811


Coding with Mitsa.


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


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


was 10 seconds the least you could use reliably?


The Apply button is not enabled if check_input() fails.


I would have to add a special handling to name_acquired_handler() where I create new problem_info and initialize its problem data in a for loop and I also want to get rid of "pi->" expressions which is related to porting from the direct access to the D-Bus API.

abrt-applet isn't critical application and I don't think we need to optimize it for speed. We should rather write readable and easily maintainable code.


Sorry ! replacing it now ... sorry for the confusion.


Yes, you are right but I want to replace all ```dd_*``` functions with D-Bus calls in the near future. Hmm, the leak, I forget but it we leak it at exit, so it isn't so bad.


```ruby
   def full_title(page_title=nil)
     base_title = "Ruby on Rails Tutorial Sample App"
     page_title ? "#{page_title} | #{base_title}" :  "#{base_title}"
   end
```


This seems a bit risky, could you view the source of the file and look for the comments on lines 4, 5 and 6 instead?


Please have look, is it fully merged?


I don't understand the reasoning behind this change from the commit message


Add this property to the README.


Placing an exclamation mark (!) after an optional value forces the unwrapping of a value. Forced unwrapping triggers a runtime error when the optional is nil. You can read up more on Optionals in the Apple Docs here:
https://developer.apple.com/library/ios/documentation/Swift/Conceptual/Swift_Programming_Language/OptionalChaining.html


Но оптимизация... :+1:


please make a review. We are providing two frontend properties: frontend/config.properties and https://github.com/UnifiedViews/Core/blob/version_1X/develop/frontend/config.properties


This menu should have the same order as the other ones: Java, Ruby, Python, C#, Erlang


@bradwilson is there any reason this isn't `await`ed?

I ask because I'm having to craft an equivalent type and method in xbehave and it struck me that this could be `await`ed and the method be made `async`.


I don't see how this will fix the problem. It'll just hide it no?


We tried that and the minification process, or some other part of the process, removes comments :-(

What seems risky about console.log?


Style:

```js
function getOptions (options) {
  if (typeof options == 'string')
    options = { valueEncoding: encodings[options] }
  if (typeof options != 'object')
    options = {}
  return options
}
```


I have a nitpick myself, though: This commit doesn't only change line.rs, contrary to the commit message.


The intention is to remove it from master once we confirm, in production, that this is working.


:+1: now it looks and feels perfect!, congrats!


Yes but DEV happens before BETA in a logical process, right ? So, why go back to DEV ?
Logic version should be ```const VERSION = '2.6.0-DEV';```, right ?


Realpath is what we want here, we should just run realpath on the overrides
before stripping the base dir or something like that.
On Mar 18, 2015 11:57 PM, "Remo Laubacher" <notifications@github.com> wrote:

> @KorvinSzanto <https://github.com/KorvinSzanto> then use preg_replace('[/+]',
> '/', $badString) which will do the tricky for 10 consequent forward
> slashes too..?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/concrete5/concrete5/commit/30f34d5e4bb6df574e699f758c111f9e3aa1e7c4#commitcomment-10279597>
> .
>


@legaultpierre :
- Good choice for Richard Jones; that's right
- and good find for Gateborg; it is a misspelling. Place doesn't exist. so yes, correct to Gotenburg University


まだMongo版実装してないな


Здесь другая оптимизация - сразу исключаем кучу вопросов типа "как убрать номер из ссылки топика", а кому действительно будет критичная такая оптимизация - настроит сам


Do you know that the code below ```gtk_main()```
```
g_bus_unown_name (name_own_id);
g_dbus_connection_signal_unsubscribe(system_conn, filter_id);
```
 is never executed?  When a desktop session exits, gtk_main() quits and doesn't return to main().


the width in vw simply doesn't limit the width e.g. in Firefox, in some cases it seams to work (dashboard), in others (classes) it takes the full width of the window. It's not reliable and long text are simply not legible!


Wouldn't it make sense to put the most recent releases of Groovy first?


Browsers that don't support console will throw errors. It just smells like there must be a better way. Even if you're not planning to get this to production, it would dirty the master branch and anyone else not watching this conversation would be none the wiser.


I can fix that, but obviously only by rewriting history.


Capybara default wait is two seconds. The internet's infinite knowledge indicates that Travis CI can delay Ajax responses by up to 10 seconds. (One such reference: http://spin.atomicobject.com/2012/04/27/intermittent-test-failures/ .) It's possible that a smaller delay may work, but considering a build already takes about 14 minutes on Travis, I'd prefer not to spend hours trying to calibrate Capybaras delay in order to save a few seconds on build time; we'd still be at the mercy of Travic CI's load.


@jeremyf Can you please build version 0.6.8 of this gem and push it up to rubygems, I don't have the permissions to do it. Thanks!


@bors: retry


but that breaks things, see comment by @emielmolenaar ?


Ich habe mal mit Transifex gesynct und bis jetzt scheint es keine Probleme zu geben. Hoffen wir, dass das so bleibt :) Außerdem bist du jetzt dort auch „Betreuer“.


Yeah, understood what the goal is. But `console` isn't the way to go :)


Should audiance be audience?


@oconnorr According to this: http://sourceforge.net/p/bitcoin/mailman/message/33222203/ there are upstream Bitcoin fixes to address the OpenSSL incompatibility. Could you look into that? I really don't want to keep outdated versions of OpenSSL around...


テスト結果の表示方法を修正しました。
たぶんこれで元通りの表示のはずです。


saw approval from metajack
at https://github.com/michaelwu/servo/commit/67666c4418e81b4ac30729d863854166ddca7ba3


merging michaelwu/servo/fix-gonk-warnings = 67666c44 into auto


All releases contains the actual commit id patched by prepare_release.py. You will only see the false id in the git repository or when exporting yourself without calling prepare_release.

For the nightly builds it is also important to use the actual commit id, because the previous release is no longer relevant. And this means we do not have separate id:s for the v2.1.5 tag and the release commit.

But you are right we should make sure to use the real one when moving it down to the previous releases in the `CHANGELOG`. This should be done right after release, I will do that.


michaelwu/servo/fix-gonk-warnings = 67666c44 merged ok, testing candidate = 8906d485


Also, `arr` is defined at the top.


Sounds reasonable :-)


The problem with using em is small window sizes (<50em). We solved that by using vw. If that does not work, we need to search for yet another solution but not go back to the one with the original problem


misspell  occurred - master/src/main/java/eu/unifiedvierws/master/application/MasterApplication.java
not unifiedvierws


Comentário super relevante do tipo, o repositório me obriga a comentar.


sure, makes sense.


I chose `OutOfNNException`, because `test_empty_namenodes_autoclient` asserts `OutOfNNException` to be thrown when no configuration is present.

Another option, towards which I'm leaning right now, would be to return an empty list here. This would be very similar to how it has been by now (not entirely similar, because in case of `HADOOP_HOME` present, but no configs present, `Exception` is thrown). With an empty configs list returned here, the `AutoConfigClient` will throw `OutOfNNException` anyway.


@refaelos couple of things:
First, I'm not an Objective-C programmer so have to rely on existing code and google for implementation.

Second, from what I read on postNotificationName there are only 2 parameters allowed: the notification sender, and the userinfo dictionary. As far as I can tell, the sender is used in the notification sender and userinfo is sent to the subscribed method....

Aaaannddd, after looking up NSNotification I see that object is a property on NSNotification so I don't need to put object into the userinfo dictionary.

*sigh* what a rookie mistake, not reading NSNotification doc closely.

I shall fix and resubmit


> All releases contains the actual commit id patched by prepare_release.py. You will only see the false id in the git repository or when exporting yourself without calling prepare_release.

I see! I never compared the releases to the tagged tree. That makes a lot of sense.


@legaultpierre, not really sure; there's a lot of Zhangs in our database. I think it's fine that you added him in.


それは別のやつですね 旧postはstatusに名称変更しました
あと今実装しました


You missed the second part of my comment. We need some standard way to
normalize the server reported directory in a way that we can replicate. If
this guy is using a symlink, we'll probably see two different paths for the
webroot and for the overrides anyway. Every file has a canonical path that
we need to be using instead of symlink paths or web server reported paths.

On Thu, Mar 19, 2015, 7:47 AM Remo Laubacher <notifications@github.com>
wrote:

> but that breaks things, see comment by @emielmolenaar
> <https://github.com/emielmolenaar> ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/concrete5/concrete5/commit/30f34d5e4bb6df574e699f758c111f9e3aa1e7c4#commitcomment-10286820>
> .
>


@Jan-Marcek great "catch"
@peterklimo Only config.sample.properties should be provided. config.properties is local user instance of properties, with custom settings - I believe it has been committed only by an accident.


Usually, a white list is more appropriate since a blacklist lets strange characters pass - for example a [*zero-width space*](http://www.fileformat.info/info/unicode/char/200b/index.htm).

Can you also add a couple of testcases for this method (or should I?)? Again, I'd very much like to see `re.sub` instead of `re.findall`.


It should only collapsed if the result of ```search``` is true. Also, it's probably a good idea to also null out ```mSearchMenuItem``` and ```mSearchView``` in ```onDestroyView```


@walterbrunetti

lo revise junto a mariana. saludos


@legaultpierre good choice. i confirm.


why?


But the code is not the BETA2 code (maybe ``-BETA2-DEV`` than, but it is weird)


@adamretter OK, just built a zip download and this worked ok, so is there something special I must do when updating?


You can use configurations for that:
```
"configurations": [
  { name="with-backtrace", platforms=["posix"], type="library", "dependencies": {"backtrace": "~master"} }
  { name="without-backtrace", type="library" }
]
```


NP dude.

Thanks for this contrib. Appreciate it.

Let me know when you're done and we'll merge it.


It does not work by putting this on the original field income_currency_exchange_account_id on res_company?  The constraint and this is not two times the same thing?


Should say:   Set the 'http_proxy' and 'https_proxy' environment variables.   (I'll add the 's' on my merge)


Best suggestion I have ATM is to wrap the console.log in a try catch block. However that puts more dirty code in master.


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


0.35.0.0 seems to work fine too .. at least on this project


PerspectiveActivity is used a lot at PerspectiveManagerImpl.

And JSWorkbenchPerspectiveActivity is used in dynamic js activities


This is a god class.This is doing everything.You need to have a Model,Service packages.Use HTTPClient and not just JS object.Use Spring as well


Make the code modular.Use Spring,RestTemplate,Models etc


Indeed. Actually the merge should not even fail.


79453838-AutomateSomeDocumentation (Myoldmopar) - i386-Windows-7-VisualStudio-12: Build Failed

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


Ok, it's some other bug. Thanks for trying @ademar :)


Why these branch whereas you do it unconditionally?


这个地方可写遇到的问题


79453838-AutomateSomeDocumentation (Myoldmopar) - x86_64-MacOS-10.9-clang: Build Failed

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


@mwarning: inet_aton it should not be modified in fw_iptables.c


Doesn't this line get us into trouble? I've stumbled across the following: with no or only a small energy oversampling factor, and an extreme value of energy_scale (like 0.9 or 1.1), one gets 'nan' counts in either the highest or lowest energy bin of the oscillated flux maps. I believe this happens because this function returns the scaled instead of the original energies (at least for all energy-cosine zenith pairs but those containing the smallest cosine zenith value), and when the fine oscillation maps are smoothed there are no probabilities to bin for the highest or lowest coarse energy bins, and the mean returns 'nan'? Could one do something like:
 ```
for ie,energy in enumerate(ecen):
    scaled_energy=energy*energy_scale if energy_scale is not None else energy
    for icz, coszen in enumerate(czcen):
        index = int((ie+1)*(icz+1) - 1)
        evals.append(energy)
        czvals.append(coszen)
        ...
        self.barger_prop.SetMNS(sin2th12Sq,sin2th13Sq,sin2th23Sq,deltam21,mAtm,
                                        deltacp,scaled_energy,kSquared,kNuBar)
        ...
        return evals,czvals
```


Fixed formatting issues


Thanks!


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


Let's be consistent, use python 2 or python 3  style


@ravwojdyla what do you think?


I still wait for the day people start to use a sane approach like PSR-2-R alias CakePHP-RIP :)


I guess that we can move string 29 here and pass only one argument to cleanup method


I have now :)

I'm using a list of official codes with set names to ease working out what set they are before comparing with magiccards.

I had NEM=ne thinking the JSON was using everything officially named. I've now found out that that one isn't, plus I've now got some strange codes left over which don't fit official releases. Most can be ignored as they look like they could be for promo cards.

Lots of strange and obscure sets popping up which could possibly hold a more recent printing of a card, so would need to be code matched, or set to ignore. If they are left in and have a more recent printing, it will use their code to try and find an image on magiccards.

Things like CST threw me for a second, it's not Coldsnap, it's the cards reprinted from ice age in the Coldsnap Theme decks (which magiccards doesn't have). RQS was more bizarre, it's the 'Rivals Quick Start Set' - first time I've heard of it and not on magiccards (as far as I can find) and it's some precon decks from 1996


//wo hai shi bu hui wan


Benötigt für NumberUtils.class


I guess so. You should probably rollback this change.

On Thu, 19 Mar 2015 20:08 Yolanda Septiana <notifications@github.com> wrote:

> Your version of "STATUSBAR_INSOVR_TOOLTIP" looks better.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yolapop/brackets/commit/4c6a0775dcee25e8e106644b3135f2a1479312ce#commitcomment-10284738>
> .
>


Why add in new variables if there the same value as the others?


We should be able to add options to Resolv::DNS.open considering #51


Look at this change it's dope


it hates you back :P


//hai shi bu hui wan


Its not a configuration, it isn't meant for server admins to tinker with:
it is just for overriding the drop behaviour of certain BCNM monsters
(beastmen mostly)


I guess you should leave this at 2014. We haven't actually created this translation until then, so yeah.


You could insert a variable definition, like `var message="served from frontend";`

Or whatever it is in CoffeeScript ;)


Trim trailing whitespace. It's a feature in almost every sane editor.


Na verdade isso prova que você já usou o Eclipse pra escrever Ruby. Eu me lembro que mais tarde você mudou para o Netbeans e me disse que era uma maravilha. :)


The fsync() is unnecessary: since the file is empty, there is no data to sync.

If you want to make sure the file exists after a crash or an immediate reboot, that would be a fsync() on the directory containing the file.

But this seems unnecessary: the container is still in ``/var/lib/rkt/containers/embryo`` at this stage, right? So there are no risks to to ``run-prepare`` it and confuse it with a non-overlayfs container.


sdhfsdkfhd kshfsdfshd


I thought so too, but if you don't put the contraints, you can still create a new account at this point and set the company to a wrong one, as the domain only filters what the dropdown menu shows you, it doesn't prevent you from linking a wrong record.
However, maybe puttin the domain on the res_company model would prevent that also, it would be interesting to test.


I'll put it in common


Sorry about changing a couple of parameters. I wasn't planing to change partially (I was planing to change all of them).
Now I know it is used by CRAB so next time I change anything I will consult you. If you want to change back the parameter names back, please do so.


Do we really need this print ?


s/filessystems/filesystem


&& is_child_theme() -- can save you some checks.


@neoeno would that be less dirty than console.log ?


Need to translate "or sign in" here.


No it's fine, don't worry. We might want to put a comment that this function is (only) used by CRAB3


I'd just add a `return;` command a couple lines earlier `if ( $is_avada )` -- don't modify the conditional.


What kind of move are you looking for?

> On Mar 19, 2015, at 9:47 AM, Dramacydal <notifications@github.com> wrote:
>
> why is it reverted? it has not been tested or there are errors?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/OWASP/rbac/commit/4e1d9e81f3b8fde0d865ba14fcaa4d664769c784#commitcomment-10285454>.
>


@kares I just tried to merge jruby-1_7 into master and got a list of conflicts. did not look into them but I am sure it is easier for you to resolve those conflicts


Need to translate "Send message to the charity" and "Sending..." here


Need to translate "remove" here


79453838-AutomateSomeDocumentation (Myoldmopar) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Master is no longer supported. I'm keeping it for historical reasons.


Need to translate the page title ("Contact us") on line 1 (can't add a comment there):

```erb
<% @title = "Contact us" %>
```


Is not needed to extend PreferencesResources.
The global preferences should be changed from user interface.


@jcrugzz This change still alows someone to pass in their codec. `this.options` is passed into `Codec()` which in turn checks that options object later. If we take `keyEncoding` as an example,  this is handled here https://github.com/Level/codec/blob/master/lib/encode-key.js#L5-L7


Исправил


Not needed to extend PreferencesService


Right


When you say `updating` do you mean doing a `git pull`? If so can you tell
me the process you follow?

On 19 March 2015 at 14:54, Jens <notifications@github.com> wrote:

> @adamretter <https://github.com/adamretter> OK, just built a zip download
> and this worked ok, so is there something special I must do when updating?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/eXist-db/exist/commit/fb4dbcac99d9a318d74ea743e2214310abd54447#commitcomment-10286995>
> .
>

--
Adam Retter

skype: adam.retter
tweet: adamretter
http://www.adamretter.org.uk


It is better to save blog related preferences on blog collection


Not sure if you are aware that ``logSetCodes(sortedSetCodes);`` in ``loadJsonData()`` creates the ``results\JsonSetCodes.txt`` which contains all the the set codes from the json feed. I have updated the code to add the set name if that makes it any easier and have uploaded the file [here](https://gist.github.com/lodici/36d7ad1bd02b8162ce7f#file-jsonsetcodes2-txt).


Eine Buchung kann hier nicht zur Rechnung führen. Es muss hier auf die Durchführung des Fluges gewartet werden. Es fehlen hier sonst wichtige Angaben.


```php
$oTopic->setBlog($oBlog);
```


89224666-FixNANProblems-Issue4557 (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1665 of 1670 tests passed)

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


hahahaha eu nem lembrava mais dessas paradas


We don't really use Django Sites. What we do is have configured BOOKTYPE_URL variable which is used for this. It means one database query less plus the url constructed a bit later would work for http:// and https:// cases.


```php
$oTopic->setUser($oBlog);
```


We want the regexes to be evaluated in a specific order, so the higher priority ones are tried first.  I'm actually not sure that doing it this way will guarantee that - need to consult a perl/UR expert!


P.S. Почему геттеры с большой буквы? $oTopic->**G**etUser(), $oTopic->**G**etBlog()?


Congratulations. Some major churn there with all the Unicode/control work!


Just finished - hehe, but thanks. This is going to need a lot of maintenance...


Thank you!! This is a huge visual improvement. :+1:


the id.length > 1 check seems to have broken swaggerize-hapi parsing of a swagger spec to generation the validations.

In this case "value" passed to refresolver:

#/definitions/Workflow

Since the id is "#' it doesn't use subschemas. This means it never finds a fragment, so fragment is returned from refresolver as "undefined", which means the "current" arg passed to resolver is "undefined", so then trying to evaluate "currrent.type" in the first conditional check causes the following error:

TypeError: Cannot read property 'type' of undefined


Hm, I am not too happy with that move, since that now looks like one of the real Smarty official README files, which it isn't. If you don't want it where it was, just purge it. It just has some necessary to know answers and questions for Serendipity Smarty caching configurations.


Done


It wouldn't break old browsers and @mconnell shouts at me whenever I catch exceptions so...


This works, but putting this line inside the logic statement just above might make the code easier to read, to make it clear that response info (name as well as text) is only given if there is a response for the employee.


rax retry tests


Ok, after this change, this is all that is left:

```
$ grep -w strtok *.[hc]
config.c:       p = strtok(p, ":");
config.c:       p2 = strtok(NULL, "");
misc.c: * strtok code, BUT returns empty token "" for adjacent delmiters. It also
misc.c: * (strtok strips them away and does not return them). Several other issues
misc.c: * in strtok also impact this code
misc.c: * strtok, with the caveat of returning empty tokens for the 3 conditions.
misc.c: * is what strtok should have been written to do from the beginning (IMHO).
misc.h: * Similar to strtok(), but properly handles adjacent delmiters as
misc.h: * empty strings.  strtok() in the CRTL merges adjacent delimiters
misc.h: * for any leading or trailing delims. strtok() strips those off
mkv.c: *        I would prefer it over strtok(), to allow something like
mkv.c: *        For now, live with strtok(), may be later I need a replacement
mkv.c:          lvl_token = strtok(mkv_param, ":");
mkv.c:                          lvl_token = strtok(NULL, ":");
mkv.c:          *start_token = strtok(NULL, ":");
mkv.c:          *end_token = strtok(NULL, ":");
mkv.c:          len_token = strtok(NULL, ":");
mkv.c:          dummy_token = strtok(NULL, ":");
options.c:                              range[i] = strtok(NULL, ",");
options.c:                              range[i] = strtok(costs_str, ",");
options.c:              dummy = strtok(NULL, ",");
pkzip_plug.c:/* Similar to strtok, but written specifically for the format. */
plugin.c:               cfg_names = strdup(config_param);       /* so strtok can modify */
plugin.c:               for (dll_name = strtok(strdup(cfg_names), ","); dll_name;
plugin.c:                       dll_name = strtok(NULL, ",")) {
raw2dyna.c:             strtok(Buf, "\r\n");
raw2dyna.c:                     // NOTE, some of these chars will never be seen in this app, due to strtok taking them out, or
SKEY_fmt_plug.c:// delim, then it is NOT equivelant to use strtokm() vs strtok.
SKEY_fmt_plug.c:        if ((p = strtok(buf, " \t")) == NULL)
SKEY_fmt_plug.c:                if ((p = strtok(NULL, " \t")) == NULL)
SKEY_fmt_plug.c:        if ((p = strtok(NULL, " \t")) == NULL)
SKEY_fmt_plug.c:// delim, then it is NOT equivelant to use strtokm() vs strtok.
SKEY_fmt_plug.c:        p = strtok(buf, " \t");
SKEY_fmt_plug.c:                strtok(NULL, " \t");
SKEY_fmt_plug.c:        strtok(NULL, " \t");
SKEY_fmt_plug.c:        p = strtok(NULL, " \t");
```


[Deployment available online](https://s.brightspace.com/apps/ipa/dev/6c4b2bcfd5a39658dc76c23c65734230e7cc0cd9/app.js)


I'm happy with it. @benbarnett what do you think?


Thanks, this will work beautifully. There's a typo in the proptypes section which throws a TypeError. Would you care to edit it, or should I merge this and fix it after the merge?


Well, but it's not a file that should reside in include/ and also not a file for docs/.

I thought it could hold some useful info, I don't want to completely purge it out there.


You will want to be more careful about what you stage and commit - looks like you probably reverted VRPN (or you made your changes directly in VRPN)


@amymok could you explain this? I don't understand it.


Assignment Branch Condition size for new_user_attributes is too high. [19.1/15]


Good afternoon,

Prompt please, what it is necessary to change in the initial blheli.asm file that the controler understood PWM 480Hz frequency?

I have WASP nano cpx. On the tail engine there is a PWM signal frequency near 480Hz.

These lines?
----------------------------------------
pca_int_check_1kHz:
; Check if pwm frequency is 1kHz
clr C
mov A, Temp1
subb A, #low (2200); If below 1100us, 1kHz pwm is assumed
mov A, Temp2
subb A, #high (2200)
jnc pca_int_restore_edge

clr A
setb ACC.RCP_PWM_FREQ_1KHZ
mov Temp4, A
mov Temp3, #120; Set period tolerance requirement (LSB)

----------------------------------------------

To change so:
----------------------------------------------
pca_int_check_1kHz:
; Check if pwm frequency is 0,5kHz
clr C
mov A, Temp1
subb A, #low (4400); If below 2200us, 0,5kHz pwm is assumed
mov A, Temp2
subb A, #high (4400)
jnc pca_int_restore_edge

clr A
setb ACC.RCP_PWM_FREQ_1KHZ
mov Temp4, A
mov Temp3, #240;Set period tolerance requirement (LSB) (I INCREASED TWICE)

----------------------------------------------------------------------------
Рeriod tolerance has to be increased twice?
I am right? Or still something should be changed?


Assignment Branch Condition size for update_user_tokens is too high. [21.12/15]


Line is too long. [102/100]


just "String patsy;" will do


Redundant curly braces around a hash parameter.


Use 2 spaces for indentation in a hash, relative to the first position after the preceding left parenthesis.


Indent the right brace the same as the first position after the preceding left parenthesis.


Avoid using {...} for multi-line blocks.


Surrounding space missing for operator '+'.


Space inside } missing.


Putting "Selection Notes for ..." makes sense only with the assumption that every time a response exists, it will have text associated with it. Is that the case? Otherwise you'd get "Selection Notes for <name>:" with nothing after it.


"nextInt(6) + 1" probably


Синтаксическая ошибка


Fix broken modal window due to bootstrap upgrade


89224666-FixNANProblems-Issue4557 (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1161 of 1163 tests passed)

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


@legaultpierre :
- K.S. White: is actually Kasey S. White, University Corporation for Atmospheric Research, USA
- S. Brown: is actually Sandra Brown, Winrock International, USA
- W. Easterling: is actually William Easterling, Pennsylvania State University, USA
- J. Price: is actually Jeff Price, American Bird Conservancy, USA
- S. Schneider: is actually Stephen Schneider, Stanford University, USA
- M.J. Scott: is actually Michael J. Scott, Pacific Northwest Laboratory, USA
- J.B. Smith: is actually Joel B. Smith, Stratus Consulting, USA
- M. Manning: institution is "National Institute of Water and Atmospheric Research"


@ahgittin recommends that `CloudFoundryPaasLocation` does not implement MachineLocation.


I would use the present continuous to make clear that at the time the log is printed, the setup is not yet finished: "Setting up stage1"


@aruijmen  waarom maak je een databank verbinding? Je hebt dit toch niet nodig?


Actually, it may be cleaner to just drop the hashing and reread it each time. As it is you read it on every access, and then re-read it if it is changed. May as well just read it return the current state of it.


This is certainly going to help facilitate the UX team's workflow for the home page news, nice work!  Everything looks great!


Андрей, запрос по почте и в личке отправляется
Владельцу-администратору?
Всем администраторам?
Модераторам?


Nice use of helper to get logic out of view. Can you think of a method name that makes it more clear here in the view what this method does?


HotFixPurchasedAirUnitTest (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@aruijmen idem hier. Je hebt ook hier geen databank verbinding nodig.