Why does this preservers order now? stdout is redirected to a file and stderr to the standard command line stdout.


ack


Sorry, it doesn't work. After a reset/Deep Standby of the STB the sort order is no longer correct. I have to copy a backup of permsort.xml to renable the formerly saved sort order (without german umlauts).


javascritp --> javascript


wow this is a cool decorator...


@shivapoudel We don't need this for the backbone-modal


@bigzz 请对照最新仓库的 zh/doc/README.md 稍微调整下格式再发一份 pull request，如果没有问题，后续会优先合并到上游的 develop 继续进行后续完善。待到 stable 后再逐步合入 master 分支。


Similarly, the CSS above will need to move into an SSX and the ```<script>``` tag for prototype.js will become unnecessary. I'm ignoring these considerations for now so that we can iterate quickly and decide on a UI, and then use that to design our architecture.


Totally agreed. I just want to make sure that none of the decisions being made will make it difficult to generate the user interface in different languages.


On 14-07-15 16:45, Alexander Pilz wrote:
> So when changing the title, we want also an ID rename, right?

I seem to remember seeing that that doesn't happen currently and that
there seemed to be a good reason for that.

Certainly it was just discussed linkintegrity needs to be fixed for
Plone 5 this week. But that's probably coming tomorrow.

:*CU#
--
    Guido Stevens  |  +31.43.3618933  |  http://cosent.nl

    s o c i a l   k n o w l e d g e   t e c h n o l o g y


「食費・懇親会費用はでます」に違和感
「交通費の支給はありませんが，食費・懇親会には無償で参加していただくことが出来ます．」？
改善案求


Is there a nicer way to assert the return value of a VimL function without writing it to the buffer and using `screen:expect()`?


haha wala ko kibaw ana, vic. gigamit ra nako ila api


Давно пора!


Clearly, I lack some knowledge here. What makes this dynamic and how would I fix it? All examples I can find are of traits and I'm not entirely sure how it applies to structs. Care to explain it a bit for me?


See #42


The [one of][random] was switched to a table as it was provoking strange runtime errors. Tables do not do this.


Yesssssss


Hi @kristophM ,

Sorry I didn't catch this when you submitted it -  the `resources :users ...` route goes outside and below the `devise_for :users` route. Lines 3 to 5 ought to be:

```ruby
  devise_for :users
  resources :users, only: [:update]
```

That ought to fix the error you've seen with `users_path(current_user)` method not being found.


Why you say "e.g.", you just have this single case.


Я не улавливаю, какая связь с именами новых переменных. Вроде ж самое главное -- что можно использовать один код для двух шаблонизаторов.


@kristophM In case you wanted to see the corresponding checkpoint where the above route configuration is introduced, see the "Interlude" checkpoint.


consider use lowercase here, like ctx


What if line is `None`?


And also extract it into method.


Can I ask why you don't want this?  It is only on mobile, as per the screenshot on https://github.com/mysociety/fixmystreet/pull/1086 to make it easier to report a problem on mobile. Had you not regenerated the CSS, so it didn't appear correctly for you, perhaps?


just to let you know: the cs fixer also supports a `.php_cs` file, which can be placed into the project root and then you can just call the executable which will then pickup whatever is defined in the `.php_cs` file.


果然是可以在本地删除掉哦。


真是很神奇的东西


Isn't `str(self)` better?


@djberg96 Could you please revert this change?  BOOL in windows is a 4 byte value that can hold either 0 or 1.  bool is a 1 byte value.  This change causes any call to ChangeServiceConfig2 when trying to set delayed_start to return "The parameter is incorrect."


Подумать (для шаблона): у опций, которые имеют ограниченный набор возможных значений, выделять их каким-то образом (например, "возможные значения", а дальше -- списком).
Здесь плохо, что у нас дублируется "по умолчанию". Если бы такой список привести, то повторять "по умолчанию" не пришлось был.


这程序TM的完全自己研究，太TM有成就感了。


Shame :(


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


hmm, ya missed it, i figured TapAllTargetPlayerControlsEffect would trigger the targetspell im slowly writing most of it without looking at examples, ill be more vigilant


Nice catch :+1:


what about pre 335.50/6.02 commits?
 Between fbca2bf and 654332e
https://github.com/TrinityCore/TrinityCore/issues/14419


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 138](https://teamcity.labkey.org/viewLog.html?buildId=260814&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 1 Build time: 00:04:27
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest.initializationError: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

```


it has nothing to do with the theme. You need to upgrade to a snapshot that includes the updated PHP module.


True. I think I added the second chunk before discovering the first problem. In my defense, this direct exit is easier to reason about than going through the next two ifs and post-loop code.


Я же могу в каждом методе сначала инициализировать контекст, а в конце очищать?


Знаю. я проверял так


With 40px it looks nicer IMHO, it take only 3 lines for heading.
What do you think?

> We create simple & meaningful
> applications for our clients that
> grow their business.

VS

> We create
> simple & meaningful
> applications for our clients
> that grow their business.

Or is it intention that heading is over 4 lines?


I thought line numbers without files raise an AssertionError


false?


no stderr is redirected to stdout of the same process so suddenly we get
stderr *and* stdout via stdout

2015-07-14 17:42 GMT+02:00 Makman2 <notifications@github.com>:

> Why does this preservers order now? stdout is redirected to a file and
> stderr to the standard command line stdout.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/6f6ba4546b46b28dd55cb50e449abb4eb9833c97#commitcomment-12158829>
> .
>


Здесь нужно написать, что подается на вход, что получится на выходе (идем туда-то, берем то-то, получаем то-то).


It will be done, eventually.


@Raesu Turns out I was not working on your repo when I got the centering working. So, you can get it to work by 1) commenting out line 59-62 2) adding a new width constraint like you did for the center constraint - here's the code
```
        self.imageWidthConstraint = [NSLayoutConstraint constraintWithItem:_mediaImageView
                                                                 attribute:NSLayoutAttributeWidth
                                                                 relatedBy:NSLayoutRelationEqual
                                                                    toItem:nil
                                                                 attribute:NSLayoutAttributeNotAnAttribute
                                                                multiplier:1
                                                                  constant:100];
```


doesn't really matter does it?


@tcavaletto  ny reason why you use Session instead of ViewState? We try to avoid using Session because a person may have a page open in multiple tabs, which share the same Session


Did you really mean to tag prereleases of Julia and all these packages? `0.3-` means `0.3-prelease`, not `0.3 and above`.


than it'll just output `None`


Sounds good.  There's no hurry, I'm just getting things on github so it's easier to collaborate.


Your call my friend. It's also your website. ;) Change it if you want. ;)


there will be more added later, I intend to refactor this object into SourceRange holding column and end poisition and that stuff


yeah :)


中英文周边要空格


"состоит из подключений"
С третьего раза только понял, что имеется в виду. Надо переформулировать.


hello github


uhm, and?


Mind that if we rename a folder, all content get's reindexed and by that also previews regenerated, unless we decouple that explicitly.


SQLQUERY is not defined


I would be precise with exceptions^^ If you add stuff later you change the docs imo :)


Hm yeah that's good


Noted for future char set coding.


没来得及细看，先基于最新仓库的 zh/doc/README.md，拉最新的仓库，再稍微调整下吧。


There should still be a `maktaba#ensure#` here even if codefmt doesn't call with empty ranges since the no-`-lines` case is an anomaly to the command line interface. Commented on #50.


so this case can't occur I think


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


Did you try it with https?  It should work, right?

This is new code - why put it in with TODOs in it?


Why rely on the `displayName` instead of `ReferenceBookPage::shouldOpenNewTab: -> true`?


added


Looks effectively like what I did at https://github.com/ambientBOX/cordova-spotify-plugin/commit/dbf3e28737748d0e06a4c376841771bd254390b1, which is working so should be good. Thanks.


Good point! Will fix.


Why use boost format?  Why not just `+`?


Its not new code


`Box<T>` where T is a trait, gives dynamic dispatch through type erasure (we don't know what type we have in the box, but we know that it implements `Middleware<D> + Sized + Send + Sync`). To make this static, you lift the typeparam so the following will be static dispatch:

```rust
struct Foo<M, D> where M: Middleware<D> + Send + Sync + 'static {
    access_granted: M
}
```

The downside is that it would introduce another type parameter, but I don't think that matters so much here as people probably won't be using the struct directly in their own types (they would likely be generic over middleware)?


Will get on it, doing the free chick-fil-a thing now.

Eric

Sent from my iPhone

> On Jul 14, 2015, at 10:28 AM, Steve Iannopollo <notifications@github.com> wrote:
>
> This might be the issue: https://help.github.com/articles/receiving-email-notifications-for-pushes-to-a-repository/
>
> It looks like you, the repository owner, have access to the repository settings. Can you check to see if the repo is set up the way the help article explains?
>
> —
> Reply to this email directly or view it on GitHub.
>


@macumber it does, for me.


There is not only a simple cast. I revert the change made by PR #7439, which introduced the warnings.


I would add a case where you test the exception raising


Will this style still work if there are multiple emails passed?


Please, change `isdigit` to corresponding `lit_char_*` helper (maybe, `lit_char_is_decimal_digit`).


https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/sound/soc/jz4740/jz4740-i2s.c?id=refs/tags/v4.2-rc2#n95

would need a patch as well?


why you removed return new Alertify(); ?


@tcavaletto oops, never mind.  @edmistj explained to me why you guys did this and it makes sense :)


You actually don't need to set this at all, just imagerelay_url..


ahh okay, so both are redirected to the file
ack :D


Actually it would be good to change the pointer for the "&w3xmlschema2;" citation in xep.ent too....


Ah sorry! I forgot to push the tags. They should be here now.


Sure will fix. Avoiding manipulating state name string is good.


to your subdomain or custom url.


remove base_url here too.


Nothing is required. Not even a coffee script plugin. Grunt handles coffee / js natively.
Could be scary, but honestly, I think the removal of curly braces, semi-colons and commas makes it easier all around.

[Grunts 'getting started'](http://gruntjs.com/getting-started)
> The Gruntfile.js or Gruntfile.coffee file is a valid JavaScript or CoffeeScript file that belongs in the root directory of your project, next to the package.json file, and should be committed with your project source.

I'd go +1 on this, but we'll want the current browser previews to land before implementing. Let me know.


For the Long or for the enum or both? Why is that better?


> Are you sure we're running 32-bit right now?
> Is the runtime we pull down to execute the tests 32-bit?

On Windows, yes.


Seems that there is a mistake, maybe there should be `&& (ecma_number_to_uint32 (ecma_string_to_number (property_name_p)) < len))`.


Yes for the compareTo, and yes i will drop it (also agree this is a bit arbitrary)


swig里两个if块只有只有一行不一样，可以压缩一下


EvansHVACObjects (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


sdfsafaf


Interesting.  Yeah on Linux we're running 64-bit.  How can we change the bit-ness?  It seems like 64-bit should be the first-test platform


Cheers.

Do you test this anywhere? Is this needed?


I have eventually used `operator->()`, in c774546. It is not straight-forward to use `.m_ptr.p` because the comparison methods should then be promoted to friend. With operator->() one gets access to the pointer without any need for friendship.


@href Not sure yet and to be discussed...


it would be more portable to use `unlink("Rperform_Graphs")`


另外，这样显示感觉好乡土风啊


ack


This is my commit


(Just noticed https://github.com/dotnet/corefx/issues/2150 ... let's continue the discussion there.)


idk if I realized that `max` works like this!


@akuefler I think the original intent was that the user could select which callbacks to activate by listing their names, or providing '*' to mean all of them, or leaving empty for none. Do you think that would be useful?

Also, there's never a need to explicitly test `if x == True:` since, if `x` is known to evaluate to a Boolean, `if x:` alone is enough.


setwd is confusing as you use it here.

if you want to save the file in the Rperform_Graphs folder, then do

```r
png.file <- file.path("Rperform_Graphs", sub(x = file_names[[file_i]], pattern = "*.[rR]$", replacement = "_time.png"))
png(png.file)
print(ggplot_object)
dev.off()


The compiled CSS dir is not supposed to be in source control (provided that we're planning on writing SCSS only and not actually writing CSS).


We can add fast-path for `uint32-represented property names:
```
if (str_p->container == ECMA_STRING_CONTAINER_UINT32_IN_DESC)
{
  index = str_p->u.uint32_number;
}
```


Yesterday i thought that was not possible, but re-reading the code, i now think this is possible, so i will handle that case as well.


The command runcode is very nasty and yucky and if it's required, I would suggest moving the command somewhere else instead of putting fle_utils.testing in INSTALLED_APPS. But I would really :+1: for getting rid of it rather than re-introducing something that spawns ugliness into our codebase and our docs.


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


Will fix.


`res.json(users);` can be placed outside `else`. That `else` is not needed.


`res.json(user);` can be placed outside `else`. That `else` is not needed.


Can you please upload the xml file as well?


Change of interface. Options hash instead of `includes` keyword argument. Needs decision.


@Alisha: Do reply to jeromie's comments once you have
addressed/incorporated the feedback. Thanks.

On Tue, Jul 14, 2015 at 7:07 PM, jeromie <notifications@github.com> wrote:

> @alisha-ajency <https://github.com/alisha-ajency> move to separate less
> file
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ajency/btwebsite/commit/67bc1b86c2b34816fa8bde671cbaed40ba6ee43d#commitcomment-12155774>
> .
>


loooove it! :joy:


forgotten comment ? :P


@Makman2 be careful with this one, links may need changing!


There is code in `ecma_op_array_object_define_own_property`, similar by functionality:
```
   if (property_name_p->container == ECMA_STRING_CONTAINER_UINT32_IN_DESC)
    {
      index = property_name_p->u.uint32_number;
      is_index = true;
    }
    else
    {
      ecma_number_t number = ecma_string_to_number (property_name_p);
      index = ecma_number_to_uint32 (number);

      ecma_string_t *to_uint32_to_string_p = ecma_new_ecma_string_from_uint32 (index);

      is_index = ecma_compare_ecma_strings (property_name_p,
                                            to_uint32_to_string_p);

      ecma_deref_ecma_string (to_uint32_to_string_p);
    }

    is_index = is_index && (index != ECMA_MAX_VALUE_OF_VALID_ARRAY_INDEX);
```

Maybe, we should use the code for the function, and use the function in the `ecma_op_array_object_define_own_property` also.

Also, maybe, the function should output numeric value of index, if `str_p` is an array index. So, second conversion outside of the function would become unnecessary.


reack


reack if @Makman2 was careful


Will removing these make it so that the variables will not be able to be interpolated into the autoresponse?


reaack


Needs decision if we want to support aliases. We could do the underscore conversion automatically instead of using a manual mapping.


Just for the `Long`. Not sure if/how it works actually (auto-boxing magic?): http://stackoverflow.com/questions/18850930/java-integer-auto-auto-boxing.


Remove pry


What do mean, is it needed?


EvansHVACObjects (evanweaver) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Yes, the bug is upstream... :unamused:


Yes this is correct. The event **will** be sent ONLY is the state was correctly fixed  (and then, in that case, state is also updated, ...) Will add some clarification..


```python
self.assertIsNot(widget, obj)
```


Same remark, don't know why?


По-моему это лишняя проверка. Если число недель окажется не целым, значит `$this->getFirstWeekStart()` или `$this->getLastWeekStart()` вернули некорректное значение, а это уже баг, а не исключение.


```python
self.assertIsNot(widget.choices, obj.choices)
```


It would have been nice to have documentation for this change too :/


```python
self.assertIsNot(widget.attrs, obj.attrs)
```


I didn't know either until I tried it..

On 15-07-14 09:07 AM, Andrew MacDonald wrote:
>
> idk if I realized that |max| works like this!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SrivastavaLab/bwg_names/commit/11e76143626ef45a94182484fb3d3c3e86b73536#commitcomment-12159363>.
>


yes... the has interface is still backwards compatible, and it's the only way to pass through all acceptable parameters to the thrift layer without having to enumerate them explicitly, and then pass them down explicitly in every step.

If we're moving the knowledge about how to process these options to the GeocodeRequest class, there is no need to include an explict list in this method.

Thrift performs it's own validation anyways.

The main loss caused by this change is that there is no implicit documentation about acceptable parameters. But that can be solved with explicit documentation.


#113 - This should be done through multiple hits and not sheer code magic.


Timesync, no capital s, is what I've been using.


我找到无势力部队以及跳出的原因了


Funny, I never saw this before in `Ledger.cpp`!  I could swear I read every inch of that file - but you're absolutely right, this was already in Ledger.cpp.  You learn something new every day.


EvansHVACObjects (evanweaver) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Very inefficient way of doing it. Check out [this post](http://math.stackexchange.com/questions/9259/find-the-sum-of-all-the-multiples-of-3-or-5-below-1000) and its first answer.


Fixed in b356dc4e07915521db1e768d6357e3d982877a6e, thanks.


The library already supports one alias, "includes".

There is the question of supporting multiple aliases (i.e. "bias", "autocomplete_bias" and "autocompleteBias" all work) or just one ("bias").

If we do support aliases, a manual conversion is much faster than going through each key of the hash, converting it to string, applying a regexp, converting back to symbol and reassembling the hash.


This is passed to LoginContext via the "principal" property. Is the format of that principal not the same as the gssServiceName below?


Very inefficient way of doing it. Check out [this post](http://math.stackexchange.com/questions/9259/find-the-sum-of-all-the-multiples-of-3-or-5-below-1000) and its first answer. You're doing it in `O(n)` but it could be done in constant time


This pull request is pretty old. Has this package died?


Looks like most of this code is common across mysql and postgress implementation. Maybe some refactoring we can do later.


Same problem as above


Looks like we lose the results if there's a glitch.  We still want to know how many hits there were.  Honestly, Crit Glitch is less of an issue as there is an implied 0 hits there.  Simple fix I'll push.


Everything else looks ok


I noticed sometimes that there are orphaned ycmd processes but this change brings back the issue from #201


Yup, exactly my thinking.


Cool, Thank you airlift! :+1:


Thanks! Noticed them only 5min ago...


yeah... done


use one of the rfc sites like (something).example.com


LGTM


"e.g." consistency ftw


I did wonder about that. Let me add a new commit with those omissions.


It looks like you accidentally added this file. It's not a big problem, but you could remove it from the repository: https://help.github.com/articles/remove-sensitive-data/  (you can treat this file as 'sensitive data')


FWIW, if the value being on HostConfig is the new "standard", then it would be better to check for the value there first, no?


Why the rename?


Created pradels/vagrant-libvirt#420 to capture the remaining issue around calling ssh_info on multiple machines in parallel to generate the inventory file on libvirt.


Note that you will be doing a *force push* to github. Normally this is dangerous, and you should always be aware that you are doing it. Make sure that you push the changed data to your branch.


Note that in the rest of this method we use double quotes (`"`) instead of single quotes. It's nice to keep this consistent in the file.


Do we maybe want to get that in 0.7.x ?
Oh, and while the compiler probably does away with this: Do we need to cast double to double? (Line 129, 155) ?


Fascinating - what's going on here?  Reading your bug fix in the cpp code is more exciting than it is enlightening.  :-)


Тип `DateTime` вместо `string`.


This is not something that you did, but it's a bug - "use slugified artists if we asked to, otherwise use the title"...??. You could add a fix for this too. I think it was copied from title slugify code above and not correctly changed


Ohh, finally! Now I can use commando much faster! But what about items 10+? How they will work in this case?


This line is getting a little messy now. Why are you replacing both the artist and title string? Are there artist strings with `/` in them? A better place to put this might be on line 273, after we create the `title`.


having issues trying to get any ellipse to populate. working with .8-dev version of leaflet, keeps kicking back a TypeError: this.callInitHooks is not a function. Line 251 of leaflet-src.js


See below here, there is also another method `download_release`. This does the same (but saves all files in a release). I think this will have the same issue. You should fix it in both places.


Added for #51


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


I cannot really see this happen, except a user passes a null value for a non-Nullable parameter


Another thing that isn't yours, but can be fixed now:

In function definitions and function calls you do not add spaces around `=` symbols. So

    def download_mp3(recordingid, location, slugify = False):

should be

    def download_mp3(recordingid, location, slugify=False):


Don't forget to put back previous sdoc value when you are done executing the content.


@agl - Having NOOPs in this case may be appropriate. My assumption when using BoringSSL regarding API changes was if there are no side effects then a NOOP method is appropriate.  However if there is behavior change, or something that requires detailed attention, then API change may be justified.


@ChronoMonochrome what is this stuff for?


Idea: we could mark specific patterns in the style guide as "Deprecated" or "Legacy".


@agl - Great thanks!


DummyUserFormViewModel does not exist in the namespace: vm_dymmy


good partner!


It is a bug in the initialization of the internal `weak_ptr` that Ledger gains as a result of deriving from `std::enable_shared_this`.  When the `shared_ptr` has a `const` type (as in `shared_ptr<Ledger const>`, and when said type derives from `std::enable_shared_this`, then the broken libc++ code tries to call an invalid assignment operator to do the initialization.  The fix is to do the same work that the weak_ptr assignment would do, but without calling it.


In gulp, `return` is used to signify that a task should block other tasks that depend on it. No true tasks depend on `test:mocha` being run, so we can remove the `return` here.


function not work on negative number
you can try  mes callfunc("F_InsertComma",-777);
then will display -,777


Does that require altering all our existing deployments? All the tables?

Also did not quite understand that:
> Note that INT will be used in PostgreSQL, not BIGINT (MySQL doesn't support
bigserial). Hopefully, this shouldn't be an issue - we don't expect any table
to have over 4,294,967,295 rows.


Hmm... not sure. Must have been a mistake. Sorry, let's get that fixed.


There is a 200ms delay (see https://github.com/Komodo/KomodoEdit/commit/468afaf#diff-fbbe8710ac9b735bbaa1657f58bf5ba6R1040). As long as you press each consecutive number within that timeframe the numbers will add up. So pressing 11 will work as long as you don't type like a granny.


You keep forgetting to change this name :)


斯蒂芬金卡是短发是劳动节asdjfklajfd
akljsdjfa;jdfsf
dskfjasljl
gotoAndplay


sadfafasdf


sadfafasdfsf


Testing a comment form the git hub console


This broke the RSpec TextMate bundle. It needs to insert unescaped content since it converts the backtrace to links.


asdfadfsadfsaf


Yeah I fixed that now.  I thought you wrote this, I didn't think I used monad notation.  Maybe I blacked out and did it then, not quite sure but this definitely didn't work...


@robclewley Yes, that's what I originally had it mind, but I changed it to just using booleans because I didn't have it fully implemented. I'm happy to change it back.


What issue does this correct? I don't doubt you, just genuinely curious. Some inline comments might be a good addition, too.


As `Object#try` is Rails-only, it's prudent to suggest favoring a `:respond_to?` or other "core language" check.


ack for everything other than the name of the Test class - could you change that ?


@amsardesai the one inside the dist folder is the actual file people using, this should not be removed.


I think this is a good use case for a default parameter, ex. `content=''`


Is this pref configurable? I mean through UI, I know about ko/prefs SDK.


And I just had that module thing in cause I was playing around in ghci which needed it and I forgot.  ALSO ALSO I fixed a real bug, the whole server culling wasn't working AT ALL CAUSE I'M A DUMMY but it's working now


So if you want to set up your machines to run now that would be AWESOME


damn


No not atm, just through JS. It'd have to toggle based on the state of `commando_navigate_by_number` and I didn't want to invest the time required to do that right now.


we need a bear for that


> Does that require altering all our existing deployments? All the tables?

No, the DDL is functionally equivalent, except that the new version will have `bigint` instead of `int` (larger column type).

It would be nice to fix the record_id constraints though (`default null`).

> Also did not quite understand that:

In PostgreSQL, `serial` uses an `int`, not a `bigint`. I cannot use `bigserial` unfortunately because it is not supported by MySQL.


Released v0.0.3 to NPM


Closed #161


This commit creates ordering constraints between fencing operations. Currently, the ordering type is "pe_order_optional", but it still serializes fencing operations.

It must make sense for the old stonithd from heartbeat.  Is the current stonith-ng supposed to handle concurrent fencing operations well? If so, does it make sense to allow performing fencing operations in parallel? Would there be any potential issues?

Since in a big cluster, if it needs to fence multiple nodes with  fencing devices based on long timeout such as sbd, it can be _very_  inefficient...


Why not have everything in one content list?


There comes me now into mind what happens if you just do
```python
sp = SourcePosition()
sp.file = None
sp.line = 5
```


ack


then you're not nice because I didn't use properties

2015-07-14 18:38 GMT+02:00 Makman2 <notifications@github.com>:

> There comes me now into mind what happens if you just do
>
> sp = SourcePosition()
> sp.file = None
> sp.line = 5
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/edb726a3e103fed6cb8bc95b5d2801ae5bc8e2b5#commitcomment-12160002>
> .
>


@akuefler I recall using a '*' for selecting something else in fovea - maybe that code is useful to help finish what you intended. In any case, if it doesn't disrupt your current solutions, maybe it can go back. Leave some comments in the code if you haven't finished implementing it yet as a placeholder for where the handling code will have to go, and an explanation of what '*' means and what the code will do. E.g. that the options are a list of strings, '*', or [] (or '' or None?)


IMO we don't need to cover that.

2015-07-14 18:39 GMT+02:00 Lasse Schuirmann <lasse.schuirmann@gmail.com>:

> then you're not nice because I didn't use properties
>
> 2015-07-14 18:38 GMT+02:00 Makman2 <notifications@github.com>:
>
>> There comes me now into mind what happens if you just do
>>
>> sp = SourcePosition()
>> sp.file = None
>> sp.line = 5
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/coala-analyzer/coala/commit/edb726a3e103fed6cb8bc95b5d2801ae5bc8e2b5#commitcomment-12160002>
>> .
>>
>
>


You don't need the global $post anymore but looks good.


Can you go ahead and assure this references the Satellite Capsule repos instead of Satellite outright?


Putting it in `.jshintrc` would then allow JSHint to thing `it` and `describe` and a few other things belonged in the global scope for the entire project, which is wrong. The way I did it is wrong, too (looking back). Should be `jshint mocha:true`.


I had a comment but lost it. This is fine.


@tdhock The following are `g_info.data` after loading `*chunk_common.tsv` and `*chunk1.tsv`. I have two questions: one is where does the `PANEL` come from? the other is as shown in following screenshots, each object in `*chunk1.tsv` object is smaller than that in `*chunk_common.tsv`, how can I copy data from objects in `*chunk_common.tsv` to corresponding objects in `*chunk1.tsv`?


But do we want to be consequent?


Hm why is this an instance field? Are you using `lambdaVisit` in more than one method?


Grammar in commit comment


don't you mean renderFilters here?


so are we removing support for non-activation key registrations completely?  I am possibly OK with this, as long as we think it is viable and/or would be used virtually 100% of the time vis a vis non-activation key registration.


Given how many of these subclasses are passing through empty string, content should probably default to it


If this becomes a recurring issue, let's explore that. Otherwise, I'm just fine to revert this commit for now.


Hm, I don't think this is what you want to do. First, I'd store the return value of the `parse` method in its own local variable (just to keep things clear). Second, you want to have the node accept the visitor.


WDYT @sindresorhus? The commit I linked up is in the boilerplate so that's how we're planning to do it in other packages too.


:birthday: :boom: :fireworks: !!!!


No. Any lead data must be included in the `text` or `html` variable values.


function xxxx = test1()
zxc=0;
ok1=0;
ok2=0;
while zxc~=1
url1=0;
url2=0;
sum=0;
in=1;
op1=0;
op2=0;
display('*********************MENU******************************************');
display(' 1.ENTER FIRST URL');
display(' 2.ENTER SECOND URL');
display(' 3.OPTIONS');
display(' 4.RESULT');
display(' 5.EXIT');
prompt='ENTER YOUR CHOICE : ';
cho=input(prompt);
switch cho
case 1
ok1=1;
prompt = 'ENTER FIRST URL :';
url1=input(prompt,'s');
if isempty(url1)
url1='ffgfgsd';
end
display('DOWNLOADING.... PLEASE WAIT...');
[filestr,status]=urlwrite(url1,'img1.jpg');
if(status==0)
prompt='retry,download failed';
asd=input(prompt);
else
a=imread('img1.jpg');
figure, imshow(a);
end
case 2
ok2=1;
prompt = 'ENTER SECOND URL :';
url2=input(prompt,'s');
if isempty(url2)
url2='ffgfgsd';
end
display('DOWNLOADING.... PLEASE WAIT...');
[filestr,status]=urlwrite(url2,'img2.jpg');
if(status==0)
prompt='retry,download failed';
asd=input(prompt);
else
b=imread('img2.jpg');
figure, imshow(b);
end
case 3
prompt = 'do you want image size to affect the overall result(y/n) : ';
op1=input(prompt,'s');
prompt = 'do you want image colour to affect the overall result(y/n) : ';
op2=input(prompt,'s');
case 4
if(( ok1==1)&&(ok2==1))
ok1=0;ok2=0;
[c,d,e]=size(a);
[f,g,h]=size(b);
i=c/d;
j=f/g;
A=a;
B=b;
z1=0;
z2=0;
z3=0;
z4=0;
if(c*d)<(f*g)
z1=(c*d)/(f*g);
else
z1=(f*g)/(c*d);
end
if c==f && d==g
imgMirror1 = flipdim(B,2);
x2=rgb2gray(imgMirror1);
y2=rgb2gray(A);
z21=corr2(y2,x2);
imgUpsideDown = flipdim(B,1); %# Flips the rows, making an upside-down image
x2=rgb2gray(imgUpsideDown);
y2=rgb2gray(A);
z22=corr2(y2,x2);
if (z21>0.98)
z21=1;
end
if (z22>.98)
z22=1
end
end
% Get means of first image
meanRed1 = mean2(A(:,:,1));
meanGreen1 = mean2(A(:,:,2));
meanBlue1 = mean2(A(:,:,3));
% Get means of second image
meanRed2 = mean2(B(:,:,1));
meanGreen2 = mean2(B(:,:,2));
meanBlue2 = mean2(B(:,:,3));
if meanRed1<meanRed2
zr=meanRed1/meanRed2;
else
zr=meanRed2/meanRed1;
end
if meanGreen1<meanGreen2
zg=meanGreen1/meanGreen2;
else
zg=meanGreen2/meanGreen1;
end
if meanBlue1<meanBlue2
zb=meanBlue1/meanBlue2;
else
zb=meanBlue2/meanBlue1;
end
z3=(zr+zg+zb)/3;
rotim1=B;
n=0;
while(n<3)
rotim1 = imrotate(rotim1,90);
[i,o,p]=size(rotim1);
if c==i && d==o
x4=rgb2gray(A);
y4=rgb2gray(rotim1);
z41=corr2(x4,y4);
if z41>0.98
z4=1;
end
end
n=n+1;
end
if i~=j
A = imresize(a,[400 300]);
B=imresize(b,[400 300]);
end
x0=rgb2gray(A);
y0=rgb2gray(B);
z0=corr2(y0,x0);
if op1=='y'
in=(in+1);
sum=sum+z1;
end
if(op2=='y')
in=(in+1);
sum=sum+z3;
end
sum=(sum+z0)/in;
display('SIZE COMPARISON RESULT :');
disp(z1*100);
display('SECOND IMAGE FLIPPED VERTION OF FIRST :');
if(z21==1)
display('yes');
else
display('no');
end
display('SECOND IMAGE MIRRORED VERSION OF FIRST :');
if(z22==1)
display('yes');
else
display('no');
end
display('SECOND IMAGE ROTATED VERSION OF FIRST :');
if(z4>.98)
display('yes');
degree=((n-1)*90)
else
display('no');
end
display('COLOUR COMPARISON RESULT :');
display('RED:');
disp(zr*100);
display('GREEN:');
disp(zg*100);
display('BLUE:');
disp(zb*100);
display('OVERALL COMPARISON RESULT :');
disp(sum*100);
prompt='press enter to check netx images';
asdf=input(prompt);
else
prompt='sorry please enter the urls ';
asdf=input(prompt);
end
case 5
zxc=1;
otherwise
disp('input error');
end
clc;
end
end


It's probably sensible to put the deregistration attempt increment here as well - although it's possible to track that slightly earlier, I think it's slightly nicer/more readable to have all three treated as similarly as possible.


(Still hate this idea... why can't `loaded` be a member element?  Not part of this change, I know...)


Interesting approach, it makes your Element class a bit bulkier but I like the way you were able to reuse the components later on.


This is a nice touch


This string-name, ‘standard’, is incompatible with 2014.7’s type, str, which persisted until 2015.5.3:

https://github.com/saltstack/salt/blob/v2015.5.3/salt/config.py#L300
https://github.com/saltstack/salt/blob/v2015.5.2/salt/config.py#L289
https://github.com/saltstack/salt/blob/v2014.7/salt/config.py#L259

[root@mcp01 ~]# salt-minion -l debug
[DEBUG   ] Reading configuration from /etc/salt/minion
[DEBUG   ] Using cached minion ID from /etc/salt/minion_id: mcp01.local.undercloud.dev
[DEBUG   ] Configuration file path: /etc/salt/minion
[INFO    ] Setting up the Salt Minion "mcp01.local.undercloud.dev"
[DEBUG   ] Created pidfile: /run/salt/salt-minion.pid
[DEBUG   ] Reading configuration from /etc/salt/minion
[ERROR   ] Invalid keyword 'standard' for variable 'master_type'
[root@mcp01 ~]# cat /etc/salt/minion
master: localhost
ipv6: False


/home/fender/Scrivania/E2/data
/home/fender/Scrivania/E2/doc
/home/fender/Scrivania/E2/doxy
/home/fender/Scrivania/E2/include
/home/fender/Scrivania/E2/lib
/home/fender/Scrivania/E2/m4
/home/fender/Scrivania/E2/main
/home/fender/Scrivania/E2/po
/home/fender/Scrivania/E2/tests
/home/fender/Scrivania/E2/tools
/home/fender/Scrivania/E2/AUTHORS
/home/fender/Scrivania/E2/autogen.sh
/home/fender/Scrivania/E2/ChangeLog
/home/fender/Scrivania/E2/configure.ac
/home/fender/Scrivania/E2/COPYING
/home/fender/Scrivania/E2/doxy.conf
/home/fender/Scrivania/E2/e2reactor.py
/home/fender/Scrivania/E2/enigma2.bb
/home/fender/Scrivania/E2/enigma2.pc.in
/home/fender/Scrivania/E2/GlobalActions.py
/home/fender/Scrivania/E2/keyids.py
/home/fender/Scrivania/E2/keymapparser.py
/home/fender/Scrivania/E2/lamedb
/home/fender/Scrivania/E2/LICENSE
/home/fender/Scrivania/E2/LICENSE.GPLv2
/home/fender/Scrivania/E2/Makefile.am
/home/fender/Scrivania/E2/mytest.py
/home/fender/Scrivania/E2/Navigation.py
/home/fender/Scrivania/E2/NavigationInstance.py
/home/fender/Scrivania/E2/NEWS
/home/fender/Scrivania/E2/PowerTimer.py
/home/fender/Scrivania/E2/pyo_patcher.py
/home/fender/Scrivania/E2/README
/home/fender/Scrivania/E2/README.md
/home/fender/Scrivania/E2/RecordTimer.py
/home/fender/Scrivania/E2/ServiceReference.py
/home/fender/Scrivania/E2/skin.py
/home/fender/Scrivania/E2/stamp-h.in
/home/fender/Scrivania/E2/timer.py
/home/fender/Scrivania/E2/timers.xml
/home/fender/Scrivania/E2/upgrade.py
/home/fender/Scrivania/E2/userbouquet.orf.tv
/home/fender/Scrivania/E2/userbouquet.premiere.tv
/home/fender/Scrivania/E2/userbouquet.srgssr.tv


//
//  ViewController.h
//  zikirmatikson
//
//  Created by Ahmet Fahri Yener on 25.06.2015.
//  Copyright (c) 2015 Ahmet Fahri Yener. All rights reserved.
//

#import <UIKit/UIKit.h>

@interface ViewController : UIViewController
@property (weak, nonatomic) IBOutlet UILabel *sayi;
@property (weak, nonatomic) IBOutlet UILabel *toplam;

@end

//
//  ViewController.m
//  zikirmatikson
//
//  Created by Ahmet Fahri Yener on 25.06.2015.
//  Copyright (c) 2015 Ahmet Fahri Yener. All rights reserved.
//

#import "ViewController.h"

@interface ViewController ()

@end

@implementation ViewController

- (void)viewDidLoad
{
    [super viewDidLoad];
    // Do any additional setup after loading the view, typically from a nib.
}

- (void)didReceiveMemoryWarning
{
    [super didReceiveMemoryWarning];
    // Dispose of any resources that can be recreated.
}
int r = 0,hepsi;
- (IBAction)sayiArtir:(id)sender
{
    r++;
    NSString *sayiArtir = [[NSString alloc]initWithFormat:@"%d",r];
    _sayi.text = sayiArtir;
}
- (IBAction)sifirla:(id)sender
{
    r=0;
    hepsi=0;
    NSString *sifirla = [[NSString alloc]initWithFormat:@"%d",r];
    NSString *toplam = [[NSString alloc]initWithFormat:@"%d",r];
    _toplam.text = toplam;
    _sayi.text = sifirla;
}
- (IBAction)toplama:(id)sender
{
    hepsi += r;
    NSString *toplama = [[NSString alloc]initWithFormat:@"%d",hepsi];
    _toplam.text = toplama;
    r=0;
    NSString *topladiktansonra = [[NSString alloc]initWithFormat:@"%d",r];
    _sayi.text = topladiktansonra;

}

@end


How come this instead of setting the margin-top: 15; for .cockpit-log-panel .panel-heading ?


another way to do this is to just set tag = 'p' for this class. This way you don't need to mess with init everytime. ex.
```
class P(Element):
    tag = 'p'
```


Hopefully this addresses all your comments at once: I tried 'return' for a really long time, and tried it on the `local last_osdmap_epoch=` line too, but I guess some nuance on the behavior either of `$(func_name args)` or of `local foo=` is beyond my understanding, because checking `$?` after `local last_osdmap_epoch=$(osdmap_get foo)` would **always** return `0`.

I'm not afraid to admit this is a less elegant hack because I didn't find a better way to accomplish this.


According to http://wiki.ros.org/base_local_planner#Robot_Configuration_Parameters
```
~<name>/holonomic_robot (bool, default: true)

    Determines whether velocity commands are generated for a holonomic or non-holonomic robot. For holonomic robots, strafing velocity commands may be issued to the base. For non-holonomic robots, no strafing velocity commands will be issued. ```


I agree. But the fix is not to make `loaded` a member, its to support construction of a `Ledger` from a `LedgerInfo` and two `SHAMap`s. Then the function that sets up the `SHAMap`s can return whether or not they were loaded successfully (and only call the `Ledger` constructor upon success).


ack


Hey @m-mcgowan , can you loosen up this requirement to simply be "the expected version should begin with 4.9.3" instead of the exact string match? This simple check is a great addition to the make file, but it can't include the datestamp after the version though because https://launchpad.net/gcc-arm-embedded/4.9 appears to take down links to all but the latest for a particular https://launchpad.net/gcc-arm-embedded/+milestone/X.Y-YYYY-qN-update style release, and will be hard for users to match your version exactly.


No. But the previous implementation treated `cc` and `bcc` as strings and therefore they couldn't be validated. There isn't currently a way to pass an array in a mapping, and that is the way to solve this problem properly.


lol what? :D


ack :)


You can remove this comment now.


so happy for this


Ouch!  So previously, we recomputed the fees when we copied a ledger!  That was silly.


uhm you shouldn't need to fiddle with that. Only i18n test is doing that. Other tests should simply use existant translations in default directory right?


@alexj, note that in order to properly support multiple emails we need a way to pass an array of emails rather than a single string with a comma-delimited list of emails. This is a change to the UI and possibly the API and handler.


I ran this code successfully on a  JeeNode v6 with the RF12B Tx/Rx using an Arduino Uno R3 + Ethernet shield W5200 + RF69CW gateway.  It's important to change rf69.cpp so that the RF bit rate is the same as for the rf12.  I used 38323Hz so the relevant lines of the register config file are:

    /* 0x03 */ { REG_BITRATEMSB, RF_BITRATEMSB_38323},
    /* 0x04 */ { REG_BITRATELSB, RF_BITRATELSB_38323},

The frequency for the RF12 transceiver is set by ::RF12_initialise - the 5th argument is airKBPS.  A value of 8 corresponds to 38323 Hz.  Tim Wilkinson's library has a handy Excel look-up table, RFM12B_AIR_KBPS_Calculator


The way this test framework works is that every test creates a new AuthenticationTest object - so if you put cleanup code (like the `reset_count()`) calls in the AuthenticationTest destructor, they'll be run after every test and you can avoid repeating this code.


On a second thought... this is wrong. Definitely wrong. bombcurs should be a WithDecorable upgrade.


Why do you need to lock to set a property?


Honestly I didn't even realize we've been adding a username field in the `passport.js` files. Since there hasn't been a `username` field in the user schema, they would just get removed I believe.


I feel this is the incorrect way to do it (xpost https://github.com/chalk/ansi-styles/commit/a42e1209861e4aa7a67d0237487314f2adf6fbf5). Adding mocha to the global `.jshintrc` means any incorrect usages of globals `describe()` or `it()` in non-test scripts will go un-noticed by jshint.

The better way would be to include a comment at the top with `jshint mocha:true` for specific test files. This allows all scripts to inherit Node.JS globals, with the addition of Mocha globals just for tests.


Why is this using old-style Config?  Aren't we supposed to use the new `BasicConfig` in all code going forward?


can't you simply import i18n and compile translations in the run_tests.py main method? This is coala specific, the test helper is generic, right?


@mikepetersonccv can you share?  I was wondering the same.


xpost: https://github.com/sindresorhus/node-module-boilerplate/commit/0bd0a44a1719fdd2f2e2c96d33c059eadb9283e1#commitcomment-12160201


Looks like this selfClosing keyword argument is never being used?


removed


updated


why is this line needed?


commit message is wrong isn't it? You're just moving the code?


I'm surprised you need these two lines - if this test fails without them, that suggests some other test isn't cleaning up after itself properly.


I also feel that it doesn't really belong in the global `.jshintrc`, as @Qix- pointed out already.

You can use the extend functionality and add a `.jshintrc` under `test/` or add `/* mocha:true *. Both seem cleaner to me.


I'll take another crack at it later today or tomorrow, depending on how my current work progresses. If you know of a better way though just let me know ;)


The file is opened/closed every time you addEvents or addDatapoints? That hardly seems efficient (granted, if you use CSV for this, you probably deserve it to be slow, but still).


Last we discussed, i thought we would end up with 2 schemas one for  PostgreSQL and one for mysql. What i am seeing is that instead we are trying to converge our existing schema. So far, looks like the changes are fairly minor, but i don't see any path forward for the 'datetime' that does not exist in PostgreSQL. Does it mean we will get rid of 'datetime' as well. That one worries me a bit considering the mess we had around date + TZ in the past.


Oh - I thought the default directory was a system wide directory ?


:volcano:

Those signals are in `indico.core.signals.attachments` (even though they might be imported in `indico.modules.attachments`'s `__init__.py`)


It doesn't look like you took my comment about this piece of code into consideration here.


@dcs619 this is a special case where they intentionally want the selected date to stick thru postbacks and even if they open multiple tabs


whoops


What about fast exponentiation?


......Can't believe that was the issue the whole time. Good catch


Turn "Tutorial" into a link maybe?


That's still racey


这就是吐槽么


@Awk34 Yes currently it's just ignored so it's not a big deal.

I also just realized that I created theses commits based on master so there are some things different in canary. Notably `.travis.yml`, `express.js`, and `contributing.md`


@adrianob can u put i18n here?


Do you need to call `free_txdata()` to free this?


That would be nice. But those fields are part of `Config` (for example `TRANSACTION_FEE_BASE`). Feel free to improve


@jbnicolai do `.jshintrc` propagate upward to allow overriding of specific parameters? Or would we have to copy all of the root `.jshintrc` settings into the `test/.jshintrc` file?


Test suite needs `@requires extension ldap` put it in class docblock

https://phpunit.de/manual/current/en/incomplete-and-skipped-tests.html#incomplete-and-skipped-tests.requires.tables.api


Sorry, but this is bullshit.


The L1 CPU caches already consist of block RAM. With this feature, we are caching block RAM, which makes little sense.


inherited **from**


Seems like elsif is needed because a condition (response == "11") is needed.


I believe at least one of those `removeMaterial` calls is inside `Contribution` in which case it *might* get called from Paper Reviewing.


I'll take care of it and push a version over. Need to roll version number anyway.


We should ensure jetty version matches across modules. Doing so may be difficult as Hadoop will bring in an old version; this can be addressed later though.


you may be right, the other tests may either not depend on the language or simply import i18n test which would take care of that... still this should not happen in this file as it's project specific.


PVR_TIMER_NO_PARENT has a semantics, which is not "no valid client id".


I remember Googling this previously. Kinda sad that it makes a difference, there is a good explanation here: http://stackoverflow.com/a/10782297

The key part of that is:
> The elements represented by the two sequences share the same parent in the document tree and the element represented by the first sequence **precedes** (not necessarily immediately) the element represented by the second one.

At the end of the day it's because browsers read HTML and CSS top to bottom. The browser can't apply styles to preceding elements, only later elements.


comment your modifications please, i will update the api soon


preceeding -> preceding


Whoooo! That is crazy shit man... I want to know more ;-)


Actually the best would be to use com.xpn.xwiki.internal.template.SUExecutor with the document author since it does not make much sense to execute xclass stuff with content author.


nit: use consistent indentation throughout file. looks like 2 spaces.


<img width="441" alt="ekran resmi 2015-07-14 19 53 16" src="https://cloud.githubusercontent.com/assets/6675722/8679311/9364d4d8-2a62-11e5-9fe8-38ffb1f82168.png">


looking for someone who is interested in scripting/improving the wp-creator, pls see https://github.com/malcrom/WaypointCreator/issues/6 :)


nit: white space


and I don't see how this sets the value for the tests as they're using subprocesses so they may be untied to this, not sure though....


Right, will change it

On Tue, Jul 14, 2015 at 10:26 PM, Lasse Schuirmann <notifications@github.com
> wrote:

> you may be right, the other tests may either not depend on the language or
> simply import i18n test which would take care of that... still this should
> not happen in this file as it's project specific.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/e18f1ed2697efe7e32bc4f55ce87f514f8f95acc#commitcomment-12160359>
> .
>


@Qix- in general the nearest `.jshintrc` when recursing upwards is picked up. However, you can use [`extends`](https://github.com/jshint/jshint/releases/tag/2.5.1) to include others.


it's sep**a**rate, no sep**e**rate. Please remember this for all other commits :P


And: what's with pythonic syntax:
```
key1, key2 = value1, value2
```
?


I did exactly what you wanted. I changed it from having side effects to returning elements and he appending.

matthew.

> On Jul 14, 2015, at 9:52 AM, Maxime Petazzoni <notifications@github.com> wrote:
>
> It doesn't look like you took my comment about this piece of code into consideration here.
>
> —
> Reply to this email directly or view it on GitHub.
>


BTW, the definition of PVR_TIMER_NO_PARENT: const unsigned int PVR_TIMER_NO_PARENT = 0. This is definitely not what we want.


@jecluis thanks for the improvement and explanation. i understand your rationale behind this change: if fail, fail early.

will give it a shot tomorrow. it's always fun to fight with bash. =)


> What i am seeing is that instead we are trying to converge our existing schema.

Yup, I'm worried otherwise both will get out of sync.

> Does it mean we will get rid of 'datetime' as well.

Nope:

https://github.com/killbill/killbill/commit/8d28f02103b01f608240692de6acc482af372c88#diff-856ed6cf66ecb1bf6aa3722bc2281e27R2

I'll make sure to fix the code for this only in the PostgreSQL part. No datetime change planned for MySQL (DDL or code).


`return int(self.obj.is_protected)`


> So, just to be clear - I thought title, firstname and lastname should be available so that someone could pass this type of data in, for example, text:
> Dear {{title}} {{lastname}},
> etc etc.

Declaring `lead.first_name` in the request variables makes that variable something the user can map *to*. There's no need to include it because the integration does not use that variable.


@dineshjain2911 @harshadyeola @gau1991 Why we manually fix this typo ?
We already have pull request pending for this.


NO!


this class should use Hadoop's `Tool`/`ToolRunner` API. That way configuration can be passed to the application in the "usual" (hadoop) way. See the `Tool` Javadocs for example, or check out the queryserver's main. https://hadoop.apache.org/docs/r2.6.0/api/org/apache/hadoop/util/Tool.html


This is not changing the values for tests - only for compilation (I assumed I'd need it for compilation)

The line which you commented on i18nTest (asking what its used for) is setting the directory for tests. They would import the i18nTest because it has the `set_lang()`


> On Jul 14, 2015, at 12:56 PM, Daniel Beyrer <notifications@github.com> wrote:
>
> Seems like elsif is needed because a condition (response == "11") is needed.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/Tr3rhodes/blackjack-1/commit/4098f503a5390d12105f7ac4239b2a386edf8d68#commitcomment-12160344>.
>
ok…I thought since it was only one if/else it wasn’t…I think that was done when u stepped out and I forgot to ask u about it when you got back…Thats my fault


@jbnicolai that has my vote. `test/.jshintrc` with `extends` for projects where `test` is a directory; header comment (`jshint mocha:true`) for those with `test.js` in lieu of a folder.

Thoughts?


Same don't forget comment. You might also consider adding XWikiDocument APIs to com.xpn.xwiki.internal.template.SUExecutor to let it handle setting/restore stuff (and have a central place where we deal with this kind of thing is probably better anyway).


i guess we can assume that deleted attachments have already been removed from non inheriting children? otherwise better use `folder.all_attachments`


we don't do pattern matching

2015-07-14 18:59 GMT+02:00 Makman2 <notifications@github.com>:

> And: what's with pythonic syntax:
>
> key1, key2 = value1, value2
>
> ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/fad02bb1e39fb79ab7f017c545dcdd26265fd56c#commitcomment-12160408>
> .
>


I didn't change that logic. We can have a discussion with jack if you like. I presume it's due to the multiple go routines accessing the same file.

> On Jul 14, 2015, at 9:51 AM, Maxime Petazzoni <notifications@github.com> wrote:
>
> The file is opened/closed every time you addEvents or addDatapoints? That hardly seems efficient (granted, if you use CSV for this, you probably deserve it to be slow, but still).
>
> —
> Reply to this email directly or view it on GitHub.
>


Heya,

The --enable-mpi part can be removed easily enough. That's just from configure.in. Initially I thought that auryn could also be run without mpi, which is why I'd added it.

I'll have to look at the flags that autotools passes. They can be overridden by defining environment variables like CFLAGS and CXXFLAGS as far as I know, since this is how we do it in Fedora packages. (I'm quite sure options can be added to the configure script, if they aren't already provided by default)

I can look into it sometime, but I'm flying to CNS this week and not carrying a laptop so it'll probably be when I get back. You could either merge the branch now and I could open a pull request for this later, or you could wait until I figure it out and merge it all at once?

Cheers,
Ankur


as strings have no meaning at this level and every value is just a string

2015-07-14 19:01 GMT+02:00 Lasse Schuirmann <lasse.schuirmann@gmail.com>:

> we don't do pattern matching
>
> 2015-07-14 18:59 GMT+02:00 Makman2 <notifications@github.com>:
>
>> And: what's with pythonic syntax:
>>
>> key1, key2 = value1, value2
>>
>> ?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/coala-analyzer/coala/commit/fad02bb1e39fb79ab7f017c545dcdd26265fd56c#commitcomment-12160408>
>> .
>>
>
>


Ok makes sense. Comparing pointers does not work ;-)


This was already fixed, but I did roll the version to 0.3.0 given all the changes. Sync up your directories when you get a chance.


no, *to*


Well, why not?  Why not use `isFieldPresent`?

Do note:  `isFieldPresent` does more than checking for -1 field index.  The difference is that we might have an entry with `STI_NOTPRESENT` - in which case this code would be wrong!


и aasm и state_machine нужны?


A little more comment here?   "This constructor retrieves an existing ledger from the database."?


hashrocket не используй уже


8080 is a commonly used port and is highly likely to conflict with other random services a user might be running. Since the query server is using 8765, you could take 8865


So you _don't_ want users in the transmission group to have access? Hm, I guess the nixos module needs some changes then.


The comment is original


Are you answering the question, or restating it?


It was absolutely an oversight. Sorry about that. Send a pull request, or wait for me to get around to it -- I'm planning to make a new release in the next 3-4 weeks.


What does this constructor mean?  (And what's `hasPrefix`?  that I honestly don't know...)


merging


reac


reack


у тебя была уже колонка status, на нее и нужно было вешать, а не создавать дубликат


And that's what you get for working on different machines to do your PRs :-P

Fixed.


original comment


Calling this from inside the loop seems wrong. _vector_clear() should be called from outside the loop. (I did look at sol-vector, and it will not crash, but I guess that this isn't what you intended.)


aasm добавляет хелперы и ты можешь просто писать `@hotel.pending?`


This worries me a bit. You shouldn't have to change anything in the vendors directory.


Thanks! Noticed them only 5min ago...


Can we somehow get rid of this method?


Also, one of these us 64-bit, the other three are 32 bit.  This seems to be correct but I want to make sure that we'd get a failure if we were accidentally using the wrong field size.

(Fields have a constant bit size, right?  `sfReferenceFeeUnits` is always 64-bit, etc...? So why can't we enforce this at compile time?  Just a thought...)


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


ack


Se ha añadido esta línea de código. Revisar trascendencia


<img width="1792" alt="screen shot 2015-07-14 at 12 03 25 pm" src="https://cloud.githubusercontent.com/assets/1072382/8679583/0e59c8ac-2a21-11e5-9ce1-5a4b1a6eafb6.png">


0xACCEDEFF - ACCessible Executable DEFinition Format


а что тут проверяется? Ты просто выбираешь опцию с дропдауна


Se ha comentado esta línea por no ser transparente para el usuario.


ack, that looks way cleaner


Why don't we assume that ``Id`` is always ``Option[Int]``? The same assumption is being made in ``LinkingTable``.


啊


Sorry!

There were no callers in the RSpec code base that passed `escape_backtrace` and it's a private API at this point so it seemed safe to remove.  It makes sense that the textmate formatter could use that since it originated in rspec-core.

Is it feasible to refactor the textmate formatter so it's not coupled to RSpec's HTML formatter?  Since the textmate formatter isn't part of RSpec anymore, it's really useful for us to be able to treat the HTML formatter's internal methods as private implementation details we can refactor as needed.


Al buscar las claves primarias, es lógico pensar que estas mantienen la versión original.


Wow ... you spotted it !!

Updated it
https://github.com/HerculesWS/Hercules/commit/ebfc9c3f8d740414b13fff86bad44d077d7ae81e
![Image of new F_InsertComma](http://i.imgur.com/Wkn4qTq.png)


thanks, Dave!


no point in leaving this .exitstatus here, although it's harmless.  feh.


no point in leaving this .exitstatus here, although it's harmless. feh.


Does something fail if you don't pass it in? I see a ClientBuilder.newClient() with no parameters in Jersey 2.19. I wouldn't expect it to be needed since we're just using the client to make simple requests in these tests anyway -- the stock config should be fine.


лучше в скоуп


We're being terrible about checking these MRI tests - thanks for catching up on them @bjfish.


Maybe `CorePlugin` instead? The module is called `core`.


Could we make the comparison "lexically greater or equal to 4.9.3 20150529"? The problem is that the UI on launchpad doesn't expose UI to download q2-20150529, only 20150609. It might still be downloadable, but you'd have to reverse engineer the URL structure to figure out what the download link is.


key


This could have been a tiny, very clear change only affecting one line...


Typo in commit message body: `These macros`


We switched to the github repo, because there are several fixes which are not released in the npm package yet.


I'm not sure how....we need the Manifest to pass to ```to[T]```.


ack


It works fine on openATV. Could you please send me a download link to your /etc/enigma2/emc-permsort.xml (please don't just paste it here or the character encoding might change), and tell me which file system your directories are residing on (e.g. ext4, NTFS)?


Not needed


ipdb!


it seems an auto generated id shouldn't be Option[Int] it should be Int since at the database level it's never null.


Fixed in 718c089debbef1f6052ddc11e573aceeb4a195b9


а где ты используешь `@search` и `@hotels`? И `@hot` не лучшее название для переменной


You don't think its important for the reader to understand that we're setting the new open ledger, in the context of choosing a hash for constructing `mHeldTransactions`?


Более дешевый и правильный способ -- проверять Psi элементы в строке с помощью XDebuggerUtil.getInstance().iterateLine() и там искать PsiComment и PsiWhiteSpace

Можно посмотреть пример в PyLineBreakpointType


в этом случае можешь использовать deliver_now. Для асинхронности есть другие библиотеки


Support new parameters
```
callfunc "F_InsertComma",<number>{,<precision>,<separator>}
```

```
F_InsertComma2( <number>,3 )
```
![Image of new F_InsertComma](http://i.imgur.com/Wkn4qTq.png)

```
F_InsertComma2( <number>,2,"_" )
```
![Image of new F_InsertComma](http://i.imgur.com/bOHZrj4.png)


mm why so?
I was about to make a pull-request with modernizr without the html5 shiv, and adding a new js file (separated html5shiv) loaded in the head like we do with the ie fonts hack (but for ie<9)


Agree, it can be enforced at compile time. And I have a framework for doing so but it needs some tweaking.


тот же вопрос что и в hotels_controller


This looks great.


Hmm, if there are more than 12 digits after the decimal place,  this results in an exception.   This needs fixing,  though I did write this as a quick proof of concept to see what the performance impact would be,   not so much to be as robust as possible.


Thanks for your comment.


自己试试就知道咯


reack


I think I remember it now, it's about setting height and offset, we need it as we need it for the sticky header.
Maybe we can consider a timer as for the sticky header.


yes, the LoginContext needs format like 'serviceName/host', and the gssServiceName expects 'serviceName@host'.


Can we now delete https://github.com/azavea/leaflet-plugins?


Because the syntax changed I think this should be released as `0.9.0` instead of `0.8.5`


I think this is a serious pull request.  It looks like this on my screen without the fix:

<img width="766" alt="screen shot 2015-07-14 at 10 20 15 am" src="https://cloud.githubusercontent.com/assets/120596/8679844/00acfcba-2a12-11e5-8a19-704fadbc014e.png">

Seems like the line break needs to be after "satellite.js methods...".


Steve,

	I checked the “Send from author” option after putting in the two email addresses “notifcations@github.com <mailto:notifcations@github.com> and steve@iannopollo.com <mailto:steve@iannopollo.com>”.  I am assuming this is what we want because I want you to get a notification when I make a commit, correct?

	I can remove the latter email address once we verify things are working.  I did not fill out the “Secret token” because I do not operate my own server.

Eric

> On Jul 14, 2015, at 10:28 AM, Steve Iannopollo <notifications@github.com> wrote:
>
> This might be the issue: https://help.github.com/articles/receiving-email-notifications-for-pushes-to-a-repository/ <https://help.github.com/articles/receiving-email-notifications-for-pushes-to-a-repository/>
> It looks like you, the repository owner, have access to the repository settings. Can you check to see if the repo is set up the way the help article explains?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/EricRicketts/good_clean_games/commit/964815e8da07ac5853a4ae1cfe183130c6541e6a#commitcomment-12157064>.
>


на будущее - тест говорит сам за себя, поэтому такие описания избыточны
а то выглядит как код с такими комментариями :grinning:
```
@hotels = [] #create empty array of hotels
@hotels << hotel1 # push hotel1 to hotels array
...
```


yeah I guess that makes more sense.  :laughing:


users not confirm https://github.com/TrinityCore/TrinityCore/issues/15044#issuecomment-121256800
Why this?


remove blank space


95574472_ip_measure (DavidGoldwasser) - x86_64-MacOS-10.9-clang: Tests Failed

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


I'd suggest the subtleties of `&mut str` are worth leaving out here, but I'll try to rephrase to not say '`str`s are immutable.


thanks! you the man!


After the initialization of the Loot-System, I give every head of a CommandTeam a KeyDefaultChest.


wrong message


I'd name it "isFirstLine"


Not to mention that the check in salt/minion.py is still comparing to 'str'


@jiaha: oh you are right. Thanks! changed that here: https://github.com/JuliaLang/METADATA.jl/pull/2893


@jfoug ``all2john.py`` will handle more stuff than simply .pcap files (very soon).


95574472_ip_measure (DavidGoldwasser) - i386-Windows-7-VisualStudio-12: Tests Failed

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


We might need a better name than ``all2john.py`` though :)


ok, sounds good


这是段落。

这是段落。

这是段落。

段落元素由 p 标签定义


Why not use debug() instead of console.log()?


Oh, no, an error! This should end "04-Solo_Guitar". There's still time to correct it, but not much.


To break
lines
in a
paragraph,
use the br tag.


Did you test it?


So, aren't the service providers in this cookbook responsible for creating and managing those files?  And doesn't this really attempt to duplicate functionality with the provides/supports stuff?


95574472_ip_measure (DavidGoldwasser) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


@hnadler Did you test it?


ack :)


Yeah this looks like rebase strangeness that was missed.


no it is valid. every time the spell fades after 3 sek, you pulled everything in range without this fix.


this should be re-introduced, it's the white background of the sidenav inner
I will make a new PR including #209 (if still needed)


Ehm... we have Mvx.DebugTrace or something similar, use that instead maybe?


все использования конструктора сейчас выглядят как new TheRFunction(Collections.singletonList(name))
Правда ли что нам нужен конструктор от списка или его можно упростить?


I'm talking about renaming the variable, not adding the comments.


EvansHVACObjects (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


updated introduction.md with the code school course of GIT


ack


http://www.gvgai.net/  I'll do it tonight.  Also what do you think about
building something for this?
On Jul 14, 2015 12:36 PM, "Max Dietz" <notifications@github.com> wrote:

> So if you want to set up your machines to run now that would be AWESOME
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mdietz94/MAAX/commit/900466ee3081384da5fb10ef147cdd97b72253ab#commitcomment-12159963>
> .
>


Anyway I really like the new button! (though the cross needs some adjustments, I'll make a PR with small css fixes)


@Rushor :+1: xm ok.


why `open_ledger` when all other variables here are named in camel case?


So, `debug()` prints debugs that can be skipped if --verbose or `DEBUG=false` is passed.

In this case, we are talking about the output of the command line, so the command line too will ALWAYS write to `STDOUT`.

For example `--version` prints out `0.2.6` through `console.log`


春晓

    春眠不觉晓，
      处处闻啼鸟。
        夜来风雨声，
          花落知多少。

注意，浏览器忽略了源代码中的排版（省略了多余的空格的换行）。


"size of render" -> "size of the render"


EvansHVACObjects (evanweaver) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Yes, but in practice it has performed very well for testing.  Stuff in Godeps/_ is just copied over.  It's never changed manually.


Typo: "qunatum".

I think "floored" should be replaced by "rounded down"


I am pointing out the difference between these two mechanisms, and suggesting we use `isFieldPresent`, because the current code does not check for `STI_NOTPRESENT`.


I guess... I still want it to hide behind a flag so someone can turn it on when there's a problem.


Final Version :sunglasses:


I had jQuery and underscore in the global scope so it could be used anywhere. (browserify sets the global object to window).

I really only had to do it for jQuery so anything in the `vendor.js` file or inline on the page had access to it.

I later just removed jQuery from app.js in favor of loading it from the CDN. (Mostly everyone would have that cached).

`global._ = require('underscore');` isn't really required, more of a connivence thing. It's more an an anti-pattern though so we should just remove global and require it where necessary.


"at a same" -> "at the same"


"at a specified" -> "at the specified"


So am I


No this is on purpose. I kept the Solo Piano and so kept the Solo category.


Optionally, you can change this to
```
if e := newEvent(); e == nil {
  dps = ...
} else {
  es = append(es, e)
}
```


"floored by" ->"rounded down to"


@mpetazzoni was there something else you wanted here?  I thought it addressed what we wanted.


coloque um titulo


Here is the code:

    if cfg.DualStackedPeers {
        done = ipv4 != nil && ipv6 != nil
    } else {
        done = ipv4 != nil || ipv6 != nil
    }

If setting is `true`, both ipv4 and ipv6 are required.


Yeah I think overall it would be a nice improvement to have some kind of unification trace. I can see it being useful in a couple of places (like this).


Other parts of the text say something like "rejected with an InvalidStateError".  I think you should use the same style here.


*instead*


This was just for testing/debugging the proxy.  I never worried about making it efficient, but having a single open file is fine.  Not worth address in this diff


There's `MvxDebugOnlyTrace` but that's never used.


Mayhaps, but I was saving the 0.9.0 version for when we removed the legacy support.  At this point, it's really not practical to change the version anyway since it's been release for quite some time now.


Hmm.
1) Part of the reason I requested [Enter] then [Ctrl-D] in the other programs was due to this issue. Where hitting Ctrl-D on a line that already had text would have no visible effect (somehow seems it's not interpreted as an EOF character because it's still 'part of the line'?). But then hitting [Ctrl-D] again afterwards (i.e. two times in a row, echoes the mimic onto the same line and then exists. And of course, going to a new line (and thus a new input?) and then hitting [Ctrl-D] would successfully end the program. I don't think I understand fully what's going on there.
2) As for that `diff` command, yes, I get the same exact results from those two commands, and thus a `0` output already with the current program.

I have to admit I do think there are funny things going on (like the Ctrl-D thing) but I'm not exactly sure of what the problem is. I think the one thing I would want in both


This breaks existing code. No one expects a material with a null native pointer (GVRRenderData.getMaterial() now can return one). Workaround is to start checking what getNative() returns but this needs attention.


typos: get -> gets, virutal -> virtual


This should be if/else, the return value is the same.


@wenzul it is exactly what @davidcalhoun mentioned. I know, not very useful, but a minor improvement nonetheless.


Yeah wasn't sure the "right" stylistic way to format this


EvansHVACObjects (evanweaver) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


I think you should rephrase the sentence "The returned promise is rejected...."  When I first read this, it made it sound like the promise is always rejected.  I think it's better to say "If quantized frame is ...., then the promise is rejected...."


Submitted upstream
https://review.openstack.org/#/c/201680/


ack


The Forwarder spawns goroutines that buffer/handle the points it sees during it's construction.  This resolves races with changing default properties.


Two things that should be mentioned
* macvlan doesn't allow the virtual interface to communicate with the host
* macvlan and ipvlan are conflicting and cannot be used for the same master device


I'm fine with this.  @mpetazzoni I'll let you merge the pull request when you're happy.


Thank you!


add that line in the file tensor/__init__.py


master https://github.com/owncloud/documentation/commit/a82363a21df597b1ec8185258756644e669de34e


you're right, fixed in 2ceec6ff687cf5b9e40bb3486c53e6445cf9b2f5


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

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


Would a good style be letting `main` do its default behavior of returning `EXIT_SUCCESS` (i.e. not making it explicit), and only doing a `return EXIT_FAILURE` when necessary be a good way to go?


This looks like it can overflow modelName.


@JoelKatz Any comment?


`resetresetresetresetresetresetresetresetresetresetresetresetresetresetresetresetresetres`


логично тогда объявлять как

private final Queue<String> myMessages = new LinkedList<String>();

тогда не возникает вопросов к геттеру.


Is "Buffer&&" a typo? Should it be "Buffer&"?


I think the syntax here should be:
 ```
<thead>
  <th>...</th>
</thead>
```
This way you have a sepearate CSS attribute you can mess with, without effecting the rows. Take a look at my branch ```u\joshmutus\html``` to see how I was playing with column highlighting.


develop (axelstudios) - i386-Windows-7-VisualStudio-12: Tests Failed

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


`perp()` is exactly like rotate; in 3D space, you need an axis to rotate around.


'down' could just mean a random, one-off '503' error


You are guaranteed these fields will be present - the fees object is synthesized and those objects are all required.


It is rvalue, used for move constructor.


ping @sle-odoo: don't forget it for v9 packaging :)


@bklare Is this transform different then ```plugins/imgproc/roi```?


Code has been updated later in the cs_set_user_model commit. It doesn't use anymore that.


Wow! I was wondering where the extra 5KB (unminified) came from in the 2.2 builds.


:-1: rubbish commut message


This is a neat thought, I like that. I have picked up the habit of wanting to simplify booleans as much as possible, but I like better the idea of saying what you really want. :+1:


Confirmed working on my new build
Thanks


HW/Course Project/Course Project 1-Pager.md

Our customers are small businesses. The specific segment I'll be analyzing are our Dental customers.


But yes base + men would make more sense.


IF this cosmetic change needed to be made - which I claim it does not because it's YAIN(*)  - it should have been made in a different commit to a commit which changes functionality.

Renaming variables because the names give a wrong impression or are useless is a good idea.  But renaming something from English to acronymese is a step down for readability.  If you really feel that someone will forget what this variable does 30 lines later, call it `openLedger`.

(* - Yet Another Incomprehensible Acronym)


Haha, good point.


The data is private... will this be a problem? I'm hoping that this can be a useful analysis for my team upon completion.


not a huge improvement :-(


> But what if someone did want to map a value to title and then include it in the outgoing email?

You keep replying to me at the top-level PR. Lets discuss this change here instead.

If someone wanted to include first name in the outgoing email, then they would include the `{{ lead.first_name }}` variable in the value mapped to the `text` variable. Again, the declared request variables are things that the user can map *to*. They have nothing to do with making fields accessible to mapping values. Because the integration never uses `lead.first_name`, there's no need to send that value to the integration. Everything that goes in the email message must be included in the `text` and/or `html` strings.

The mapping would look like this (notice there's no mapping to the `lead.first_name` request variable):
![pastedgraphic-1](https://cloud.githubusercontent.com/assets/569/8680245/a533760c-2a25-11e5-9a2d-a01f2e213d44.png)


Typos in commit message body:
* `This adds`
* `In practice, `


Actually, I'm confused.  What is `open_ledger` and where is it defined?


This is wrong, it is a very complicated instruction to emulate so I preferred to leave it as inline assembly until a good implementation comes. Same for LWR, SWL, SWR.


Use interface instead of implementation in declaration


develop (axelstudios) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Is this okay? You downgraded from Django 1.5. I think it'd be better to just use this as an opportunity to upgrade Django to 1.7 or 1.8.


Yes, it is written on the page linked in commit message.
In short, capstone reported incorrect sizes of immediate operands.
This might have been improved since then, but I did not recheck yet.


Static initialization should be below all the field declarations


Added upstream
https://review.openstack.org/#/c/201350/


In the above example, the consumer would receive an email  with the text body "Hi Bob, thanks for your interest in our product!".


Submitted upstream
https://review.openstack.org/#/c/201680/


Changed non-final field to final as it was never changed


The meaning of `done` is to stop searching for more IPs, not to indicate that the request is valid. That condition is here: https://github.com/chihaya/chihaya/blob/97cb3c7cb60bbb130a4226716352048856564b6e/http/tracker.go#L178


you shouldn't. The issue comes wrong id returned by capstone by analyser.


To ensure a compat path I'm going to work with @ljharb to push `_.keys` fixes of lodash to es5-shim `Object.keys` and make es6-shim `Reflect.enumerate` support IE8.


+# JavaScript-DOM-UI-Teamwork


Thanks for the quick reply. So it is C++11 syntax. I'm porting to Java, so I'll have to think about this one.


why do you change it? it is not the same instruction.


I'll test this out before merge..
Thanks a lot.


Shouldn't we just have a single method to tell if a field is present, and not use a method in some case, and code that's part of that method in others?


Oh... because of how the instructions play with the upper/lower parts of the register? Then I guess I will have to use the old emulation in asm and start with rewriting it into C.


This line has to be directly above the call to `gettext`.


YUCK! We need to remove the `z` parameter from `me.Container.addChild`...


Same note here - this line has to be directly above the call to `gettext`.


@Jasonbhaas @AlexBailey20 , is this now finished?


add comment


Maybe I missed something here but the code looked identically, right?


Not the fstab!


The STObject hierarchy of classes could certainly be improved - beyond the scope of this PR. And not a short-term priority.


remove import


Are you sure operand(1) is returning a sign-extended immediate in case SLTIU? if not, it may be wrong.
You should not try that kind of factorization.


Its a constructor dispatch tag, in `Ledger.h`.


`Renderable` constructor should accept the `z` parameter following `x` and `y`.


Rather than cutting and pasting this function over and over, please extract it into a common helper function.


Removed the additional one


Yup!


So does es5 shim fix this?


We are using the suffix `OL` to mean Open Ledger and `LCL` to mean Last Closed Ledger. This is already established in existing code, and this commit merely brings it in line with other code. It didn't need to be in a separate commit - the substantive change is only two lines. Give me a break.


@kkaefer @brunoabinader: FYI, this `nudgeTransitions()` method comes from b8388168dd130c67c77254565cdb576df7905915 on the 0.5.0 branch. Is it still necessary, given #1548 #1891?


Not at the moment but it will. I'll be posting issues to es5-shim and es6-shim to cover the difference.


So-called "Mangled Mesa" has all its symbols prefixed by "mgl" instead of "gl" (ditto for glX: "mglX").  It may ease transitioning if we searched for the first capital instead of explicit testing for prefixes.

That said, Mangled Mesa exists solely because of the problem[s] that libglvnd solves, so I'm not sure it's worth it---a better long term solution is probably that Mesa should just drop support for name mangling.


According to Sweetman yes... SLTU takes a register and SLTUI an immediate. Both (unsigned)


Oh, I just read your comment below. This change is because `Container` is still modifying the child position when it is added.

Now we have two things to remove instead of just one. :stuck_out_tongue:


is that necessary? from ARM version, there is no such line before a `_[call(operand(...)]` so I was under impression there is no such need.


sure, can come from an environment variable. Or just let the user know he needs to enter a password


longer story: needed to swap bebas neue in for bebas because something about every version of bebas on the internet renders incorrectly in firefox (and presumably IE -- all text using bebas had a negative top margin). i think that this is because the characters [dollar, eight, question, S, s, asciitilde] had the error "missing points at extrema." i wasn't that interested in figuring out how to fix the font file.

http://stackoverflow.com/questions/14155634/negative-top-margin-in-firefox-and-ie-when-using-font-face
http://stackoverflow.com/questions/3400296/font-face-problem-firefox-adds-padding-chrome-does-not


string formatting is more error prone and more memory efficient since strings are immutable.


so weird. have hosts *never* been able to update topics before this? @mehulkar


Yes it unconditionally sets $ra.


base is in opeand.mem


Cool, will you be adding ES5 shim to the test suite? I see `IE8` is still in the sauce labs matrix


Yay! I got through to him :wink:


commit message: please remove period


Instead of doing as many as 9 comparisons - O(n) - especially since you're doing string comparisons, you may be able to optimize by creating a jump table:

    function processAudioNode() { "true" block }
    function failToProcess() { "false" block }
    var understoodNodes = { ".mp3": processAudioNode, ".wma": processAudioNode, ... }

This approaches O(1).  However, in doing so you sacrifice clarity.  Consider benchmarking.


oh, then it doesn't make much sense :-(


Yea i Got it bro :)
I already fixed the xposed problem with MIUI 6 but as you said i must use the miui binary which located inside XHDPI directory to fix the booting problem, ok i do this but the rom still stuck at miui logo
I made a fullota package with your patchrom but the same problem still exist too .....

This is my MIUI 6 ROM its good but the main bug i face is that the gallary force close when open photo and click on send

http://forum.xda-developers.com/galaxy-s2/development-derivatives/rom-blackra1n-t3036945


Are you unable to use Windows.Foundation.Uri to canonicalize?


:+1: D


ok, well I also have MIPS issue with capstone: it returns invalid ID for instruction in MipsInstructionAnalyzer. I don't know if there are some hidden side effects in the way it is used since the *disassemble* stage seems correct while the *decompile* stage is giving use random ID while the rest appears correct (good string, good detail).


now you get an ack ;P


So would returning an `EXIT_FAILURE` perhaps be a better idea here? I chose `continue;` because I thought it would be reasonable behavior to tell why is nothing going to be done, and then do nothing (i.e. just wait for the next input).
I guess I could silently do nothing, but I thought that might be confusing, and that doesn't really go with your thought about not using `continue`


A simpler approach than a jump table would be to create a `SupportedExtesnions` lookup object. `if(SupportedExtensions[extension]) {   } else { }`


you need to compute the effective address (ea), to extract some lower bits to know which bytes in the memory may updated with bytes of the register. And you must take into account the endian mode too.


Changed to that less verbose method above, thanks.


Mmkay, since the program wil exit with the name of the exception, and it's an invalid expression to not input a number, I guess simply doing:
```
else {
    stack.push_back(stoi(curr_input))
}
```
would be a valid choice.


@obiot I left some feedback comments.

My only real concern is removing the automatic z-increment that the map reader relies on. I suppose it should be a truly supported feature of `Container`, like autoSort. Maybe call it `autoDepth`.

When `autoDepth` is enabled, the Container keeps track of the last z-index added, instead of the map reader: https://github.com/melonjs/melonJS/blob/fd7b6540d1747d4f9aeeafafa5959350b28ef597/src/level/TMXTiledMap.js#L310-L311 And the container can be smart about whether or not the object being inserted is renderable (and thus has a position vector).


most implementation uses a loop to do so. And so it is a little more complicated to turn it into a IR code.


Even when it is disassembled? I thought that only mattered when writing assembly :-(


> Cool, will you be adding ES5 shim to the test suite?

Yep, I have it local. Tests will fail for a bit and that's ok.

> I see IE8 is still in the sauce labs matrix

Ya, I'll keep it in the matrix. I want to make sure we have a valid compat path.

> If yes, should tests not be left mostly as is?

Should be very little changed.


This was probably an error from moving from an old draft to the RFC.
The -2 is because the len/type is not to be included in the count for this TLV type.
Section 6.7.1 explains that.


Cool. I'll get AA once you push it to master, I assume in branch is still development and not stable. Anyhow I missed your previous comment, I'll try to reproduce bug in ImGui with DX9.


agreed - if we have to take time out to ask, the name isn't helpful.
I vote to call the package 'concurrency' and to avoid inventing acronyms for packages in the future


Note: the new constraint broke the sending of `AccountCreationInternalEvent`, see https://github.com/killbill/killbill/issues/346.


You definitely wrote this, I checked git blame.


Sorry, I don't quite follow why you would want to unlink the result file.


npm should work fine with this, it'll load helpers.js from the same dir as xml-parsers.js. We use this pattern in other files.

Here's a signing.js example from src/main:
```js
$ grep -ri signing.js *
list-objects.js:var signV4 = require('./signing.js')
minio.js:var signV4 = require('./signing.js')
multipart.js:var signV4 = require('./signing.js')
simple-requests.js:var signV4 = require('./signing.js')
upload.js:var signV4 = require('./signing.js')
```


This is because webtest's TestApp is a bit annoying when dealing with unicode. You can get this to pass by adding `content_type='application/x-www-form-urlencoded; charset=UTF-8'` into your post() parameters. Then you also have to encode your entry_data:
```
entry_data = {
        'title'.encode("UTF-8"): u'a ɶ character'.encode("UTF-8"),
        'text'.encode("UTF-8"): u'another ɶ character'.encode("UTF-8"),
    }
```
You'll also need to authenticate your user: `test_login_success(app)`


You can definitely speed up _write() by doing a past path read of the status register within _write() and only doing the whole _waitforstatus() thing if the operation has not already completed at that point. In my experience, the page write happens so fast that by the time you poll it returns completed.


getComputedStyle 메소드가 reflow(+repaint)를 발생시키기 때문에 비용이 큰 메소드라는 것을 알게 되었습니다.

Jquery는 어떻게 하나 궁금해서 .css() API를 찾아 보았더니 API문서에 "Get the value of a computed style property for the ..." 문구가 있는것으로 보아 .css() 를 사용하여도 getComputedStyle() 를 사용했을때 발생하는 문제를 피할 수 없을것 같다는 생각이 듭니다.

css 속성에 접근하는 방법중 document.styleSheets[0]와 같은 방식도 있던데 이 방법은 너무 사용하기 까다로울것 같다는 생각이 듭니다.

결국 html 문서 자체에 목표이 되는 태그의 style attribute에 값을 미리 할당해둔 후
javascript를 이용하여 element.style 형태로 접근하는방법밖에 없을 것 같은데
이방법은 html = 의미와 구조, css = 스타일 이라는 관점에서 보았을때는 별로 좋지 못한 방법인것 같습니다.

교수님께서 주신 질문에 대한 답이 궁금합니다!


Of course, I just created the issue #176, thanks for your time.


I see, I'll revert it then.


I've used file(COPY...) before but then the problem is that if you edit a .doc file and rerun make nothing happens. Only a rerun of cmake triggers copying of the files. That's why I used the 'copy_docs' custom target.


Yes, the tests in that file fail. That's why I changed it. ;) The client deserialises the message and it needs the Jackson provider.


What if I want to show alert with status of of execution?
In case of error I will see 3 alerts with every message, but I expected only 2 ("Sending request..." and "Error :(").
Do you know any solution?


Thanks. My font code is such a mess and now I'm just using much simpler code, getting data from stb_truetype high-level helpers and I'll assume it just works better. (not committed yet)


Ah yeah okay. Hm, my thought for this is to simply have a boolean of whether the last expression processed was a number, and so then I can check that boolean right before processing a number.


Not a solution


Hey ok to leave the file named as airpair for credit? I'm in the middle of writing a big blog post about this library and a few other pieces and I'd like to link to the commit.


Format


default variable as own method


ack


delete comment


The website uses javascript, so if you look through it you can find the
save url function and view the algorithm. It is the same one we use :)
On Jul 14, 2015 7:38 AM, "Daniel Mauritzson" <notifications@github.com>
wrote:

> Nice! Just curious where you fund this info? :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EmmittJ/PoESkillTree/commit/2bd56f265fcc987cdb03ae31cfe5b2c8b4d1c01b#commitcomment-12157291>
> .
>


I am getting this error:
> Key 'AdvanceForLoopToLambdaDescription' is not found in localization properties file: bundle.properties


Yeah, this went along with my experiences from [earlier][1], that I still don't understand :P
[1]: https://github.com/genkimarshall/exercises/commit/d7c45dcc1b1f71613e6ea752e651c337ea4adfe5#commitcomment-12161278


I've used this to make sure that translator.py is run from the right directory, directly calling python here produced files in the source directory instead of the output directory...


something's wrong with this sentence. Is it the `to` that's not needed?


german is bad :)


You're right, it's better.


The author was intending to call the native material ptr set, right after this constructor. Do you think it would be better way to delete this constructor, and instead create a combined one that has material setting?


`hung intermittently`


Não tá puxando do WS as informações não?


@Jaimefrio I am facing trouble in porting this part of the function. I can rewrite this function using fused data type but how can I get the minimum and maximum values of the corresponding data-type in the fused function? The function uses the minimum and maximum values of each datatype to restrict the values of iterator. I am thinking to use if condition for every case but that will make the code lengthy. Please help me.


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


show all instance variables of model


Thanks misko, I'll look into this later today :)


Corrected DISM command example. (Removed extra "/Online" and corrected "/Enable-Feature /Telnet-Client" to "/Enable-Feature:TelnetClient"


I will check that.


Getting a warning here:
> The field AST.JLS3 is deprecated


Okay, that makes sense. Good to know for the future :)


Yes, Fixed for 2 spaces indent


Trace web will run on 8865 port


My interpreter:

```
/* SLT */
__forceinline void Allegrex::Instruction$< 0x0000002A, 0xFC0007FF >::FastInterpret(Allegrex::Context & context, u32 opcode)
{
    PROLOG; context.do$SLT(rd(opcode), rs(opcode), rt(opcode)); EPILOG;
}
with:
        __forceinline void do$SLT(u32 rd, u32 rs, u32 rt)
        {
            if (rd != 0)
            {
                gpr[rd] = (s32(gpr[rs]) < s32(gpr[rt]));
            }
        }

/* SLTU */
__forceinline void Allegrex::Instruction$< 0x0000002B, 0xFC0007FF >::FastInterpret(Allegrex::Context & context, u32 opcode)
{
    PROLOG; context.do$SLTU(rd(opcode), rs(opcode), rt(opcode)); EPILOG;
}
with:
        __forceinline void do$SLTU(u32 rd, u32 rs, u32 rt)
        {
            if (rd != 0)
            {
                gpr[rd] = (u32(gpr[rs]) < u32(gpr[rt]));
            }
        }

/* SLTI */
__forceinline void Allegrex::Instruction$< 0x28000000, 0xFC000000 >::FastInterpret(Allegrex::Context & context, u32 opcode)
{
    PROLOG; context.do$SLTI(rt(opcode), rs(opcode), simm16(opcode)); EPILOG;
}
with:
        __forceinline void do$SLTI(u32 rt, u32 rs, s32 simm16)
        {
            if (rt != 0)
            {
                gpr[rt] = (s32(gpr[rs]) < simm16);
            }
        }

/* SLTIU */
__forceinline void Allegrex::Instruction$< 0x2C000000, 0xFC000000 >::FastInterpret(Allegrex::Context & context, u32 opcode)
{
    PROLOG; context.do$SLTIU(rt(opcode), rs(opcode), simm16(opcode)); EPILOG;
}
with:
        __forceinline void do$SLTIU(u32 rt, u32 rs, s32 simm16)
        {
            if (rt != 0)
            {
                gpr[rt] = (u32(gpr[rs]) < u32(simm16));
            }
        }
```


Why was this removed? Your tests are now failing on Travis CI.


This is just a more convenient way to manipulate memory, so that we do not have to worry about cleaning up the memory where the key used to be.


"— slti and sltiu both sign extend their immediate fields." darn it :-(


"framework excellent"?


Yes I read that you are correct.


Isn't this creating a hard link though?


EvansHVACObjects (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


Yeah, I can pastebin the js if you want :) I guess it isn't exactly
un-minified since names are hard to redo, but it already readable
On Jul 14, 2015 11:12 AM, "Daniel Mauritzson" <notifications@github.com>
wrote:

> Okay, that makes sense. Good to know for the future :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EmmittJ/PoESkillTree/commit/2bd56f265fcc987cdb03ae31cfe5b2c8b4d1c01b#commitcomment-12162076>
> .
>


The details here need to be worked out.


Neither `absoluteCanonicalUri` nor `toString()` don't fix such paths.


See https://travis-ci.org/mdarefin/Convert-For-Each-Loop-to-Lambda-Expression-Eclipse-Plugin/builds/70830872.


@ghislaineguerin @mmeiers
I've added two gradient mixins here.. an 'angled' and 'vertical' version.
These can be customized and made into classes as needed, then dropped into your elements with '@extend' .  Hoping these are flexible enough for reuse.  Thoughts?  Thx!


sp va en mayuscula


The key functionality we need:

- upload new content for a file.
- download the content of a file.
- check whether a particular ETag for a file is up-to-date.


This is an example of how I have it running in ubuntu at the moment:

```
# This is required, or php_pear LWRPs fail
link '/etc/php5/conf.d' do
	to		'/etc/php5/mods-available'
	action		:create
end

php_pear 'mailparse' do
	action		:install
end

execute 'php5enmod_cli_mailparse' do
	command		'php5enmod -s cli mailparse'
	not_if		'php5query -q -s cli -m mailparse'
	only_if		{ ::File.exists?('/etc/php5/mods-available/mailparse.ini') }
end
```


&str


Indeed, but in my use case initializing the L1 CPU cache at startup would be difficult. The idea here is just to allow storing firmware code in blockram when we don't want or can't use SDRAM and booting on the preconfigured blockram at startup.


`before_action` lepiej przenieść do `application_controller` wtedy na każda strona będzie wymagała logowania, bo inne kontrolery dzieciczą


Nah, that is fine. Now that I know I can find it when I need to.


Awesome! Thank you for the update and explanation! :)


This is intended to be used should the client choose to download the file in chunks.


Yep, it is really helpful when deal with functional changes to the tree.
Although not much help with jewels...
On Jul 14, 2015 11:17 AM, "Daniel Mauritzson" <notifications@github.com>
wrote:

> Nah, that is fine. Now that I know I can find it when I need to.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EmmittJ/PoESkillTree/commit/2bd56f265fcc987cdb03ae31cfe5b2c8b4d1c01b#commitcomment-12162188>
> .
>


ten loader bych udelal tak aby nahradil tu `fa-times` ikonku a dal mu `fa-fw` at se nemeni velikost toho tlacitka pri loadovani. jinak dobry :)


jo teda jeste misto mergovani rebasujeme, kdyztak ukazu