Do we correctly reject the "special" fdnames here?


Вообще, для нужд аунтификации используется модуль Passport (либо аналогичный такому). Не стоит изобретать велосипед.


**require-jsdoc/require-jsdoc**: Missing @jsdoc for componentWillMount


**indent**: Expected indentation of 5 characters.


**indent**: Expected indentation of 9 characters.


**indent**: Expected indentation of 4 characters.


**indent**: Expected indentation of 8 characters.


TicTacToe MiniMax algoritm


The conflict you're encountering is why the concept of ContentHandlers was invented.

However, in this case, keep in mind that it is _you_ that is wanting to sneak in a replace by using a response handler. They don't have to be in the same process[ing]. You're finding it convenient to do so and long term that would probably be expected (thus ContentHandlers) but the interface for ResponseHandlers predates all those ideas. Of _course_ it is going to have some problems. However for the time being it should be representative of its intentions: Handling ResponseHandlers.

Does that make any sense?


надо бы его в bower.json перенести, как и все либы использующиеся только на фронтенде


@lindamartin WDYT about representing `exit()` as `FunctionCallTree`? (to be consistent with `echo` and others)


LGTM


mongoose сам зависит от mongodb, поэтому тут это избыточно, только в случае явной необходимости.


Thanks for the explanation. I added the fix to the pull request.


Ack as a first step. We may later want to collect more of this (directory is around 12M on my test systems and very compressible).


Nm, have pinned our requirements to 1.3.2


Почему бы не вынести в переменную данный require?


we use promise


@lindamartin Do you think that `NamespaceNameTree` should be in `declarations` package? I would put it better to `expressions`


Det ända rätta sättet


Сделай проверочный вызов своей функции.


Nice contribution! I thought about this as well.


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


Jag tänker avsätta dig


done https://github.com/w3c/charter-html/pull/131


@edolstra: You also removed recurseIntoAttrs (i.e. no more qt55 on hydra)... intentionally?


Наверное логично было бы вернуть результат.


This should be deleted now. If people want to specify a URI, they should be able to put it in the string param


Thanks for change, but that don't works.


в общем, комменты ты прочиай внимательно. Да, как ты и сказал, это никак не подход в котором используется ангуляр и аналогичные библиотеки.
обязательно добавь .gitignore


@mbootsector made a wonderful job, and all congratulations goes to him.
Let's now try to understand SMP more deeply.


It returns pointers because it needs the null/None case. I think coverity is wrong, or at least I checked it with valgrind which told me there were no leaks.


ok


The event is what's being destroyed here, not the request.


Agreed on both! I was a bit unsure about whether the invite_ before max_uses was helpful or just complicating things, so will remove it after all.

Also, no, it doesn't actually need to be accessible, I was thinking wrong/changed plans halfway. :) Unit test rewriting inspiration would be appreciated!


@lindamartin you forgot to use class field here


@vilchik-elena yes me too, but according to the definition of the [echo statement](https://github.com/SonarCommunity/sonar-php/blob/master/php-frontend/src/main/java/org/sonar/php/parser/NewPHPGrammar.java#L527) in the grammar ``echo ("hello")`` is represented as ``echo`` and ``parenthesised expression`` and on a language point of view I am not sure it sure be represented differently. WDYT ?


Повторно эту информацию неоткуда брать. Тогда необходимо предусмотреть перегенерить пароль и повторно отобразить новую информацию.


This is wrong.  `uart_getc_nowait` returns the character read (which with this change is then just discarded).  To do `sdk_uart_rx_one_char` the character needs to actually be placed in `*buf` before returning.


@vilchik-elena indeed agree! I will fix it.


@bakhshb @amar441
Why are we creating objects here?
a) They should be loaded via fixtures
b) What if we want to use this step for something else than saved search results/queries? (inflexible)
Can't this be a simple `click_on arg1`?!


Why reload the event here?


src/Tests/Plugins/Тесты_ЗагрузчикИзПодсистемКонфигурации/ObjectModule.txt

Ух какую опечатку нашел. Она появилась в ядре в структуре результата тестирования и везде распространилась :(
Сейчас исправлю.


All method names are not according to the Java spec.. please changes those


Why is this class inside the TCMURI class? What benefits does it have over the old implementation?


Works a treat everywhere now


Bionerdy@users.noreply.github.com


thanks @arjitc i wil be waiting for that!


Small nit, but couldn't we break from the loop after setting finalized = false?


@vilchik-elena yes indeed I thought about it but did not wanted to do it during the migration to not extend it too much. So I just created the ticket not to forget about it: [SONARPHP-550](https://jira.sonarsource.com/browse/SONARPHP-550)


But who does delete the object?


@vilchik-elena No indeed agree it should be in ``expressions``.


@artbear, как же не исправлено сообщение пользователю... Вот же оно


@vilchik-elena indeed, thanks!


I hate it when people are sämst


@PWKad ,
Following this and noticed the CSS tag for styling was not updated. I am also curious, was the normal styling for this suppose to be what Chrome was showing for appearance? If so the "dialog" selector, or now the "ai-dialog" selector would need some base styling. If you wanted I could assist and create a PR if needed.


@mstembera Ok, I have removed the line.


OK, the problem is here, this path is relative to doc subdir. (i.e. it is looking for doc/man/ceph_selinux.8 which does not exist)

You were right that ceph_selinux.8 is not in EXTRA_DIST and hence, it is not being packaged properly (if you use make dist). There are some other issues with the man page, though. Can you stash this chunk? I'll create a PR that will hopefully fix that behaviour alongside some other issues.


@gut is this "if" correct? What if the immediate fits in the word size? No instruction emitted?


Actually, looking at the tests, it's not actually necessary to use the setter method there, so I removed those parts.


:ok: :1234:


Thank you! This commit saves me from the hassle of rebuilding them on my own. :clap:


nvm, I see that in the following commit other code follows it. Although as an isolated commit it wouldn't hurt to add a break, but forget about it.


So it will not work on Node 0.10, right?


Have you considered using `onLongPress` instead of `onPressIn`?


what this do?


Looks like this should be calling `fully_destroy`


I'd recommend just changing the description to "The office document contains malicious content." Then adding the detections in self.data:
```
self.data.append({"Anomaly": "The file appears to have no content"})
self.data.append({"Anomaly": "The file has no edit time"})
self.data.append({"Macro": "The file has X macros"})


Ya I know for this it has no relation. In this case it should probably be a two stage initialization or lazy initialization


:D - thanks!


The library should only expose one global variable, and IE8 does not have forEach.

Here's an (untested) quick alternative:

```js
window.drops.remove = function(selector) {
    var el = document.querySelector(selector);

    var events = [
        'dragenter',
        'dragover',
        'dragexit',
        'dragleave',
        'dragend',
        'drop'
    ];

    for (var i = 0; i < events.length; i++) {
        if (document[events[i]]) {
            el.removeEventListener(events[i], document[events[i]], false);
        }
    }
};
```


Why would it not?

https://v8docs.nodesource.com/node-0.10/d8/d83/classv8_1_1_function_template.html#a97c5f0632523e9ec66b035ee608dd879

https://v8docs.nodesource.com/io.js-3.0/d8/d83/classv8_1_1_function_template.html#a97c5f0632523e9ec66b035ee608dd879


Transition 'z' is uncontrollable


Please revert. You can't do anything in a signal handler that allocates or frees resources or interacts with any global state except a sig_atomic_t.


:clap:  I like this.


Oops, thanks.

I just checked, and realised uart_rx_one_char() seems to be otherwise unused - there's a prototype for it that was added to sdk_private.h early on for esp-open-rtos only, but that's all I can find.

Any objections to just removing it?


For the wiki: templates/2k11/admin/style.css in ISO-8859-1? That should be UTF-8 I think.


Need to add an empty line

In fact, there were no conflicts


@racardoso this emmit does not call the lowering function for ```addqi``` and the immediate can be greater than 16bits. We can fix temporarily adding the second tmp register, but the final fix must refactor all ```VptrToReg``` calls to avoid use the tmp register.


```diff
- job.contractType) | trans
+ job.contractType)|trans
```


Is there hope that this "Pull Request" will be ever merged into "master" of "omnibud_updater" and we will see it in "1.0.5" release (if it will happen at all)?


yes kind off:

The problem is that CommandRegistry.get(vertx) should be async and provide a callback to be sure it's deployed in Vert.x, but thing is that the deployment in Vert.x is only there for cleaning the registry when Vert.x close and the deploy is not so important, however it makes this test fail. So well :-)


Technically, I agree. But at least my version of said file is in ISO-8859-1, and I see no reason why it *needs* to be in UTF-8.


And I prefered to keep it sync for the sake of simplicity.


Why Rabbit is here a special case? Is it the only transport that doesn't have "HasSupportForDistributedTransactions" flag set?


comment


You forgot Python 3.3 here.


unterstütz nzb.to retention filtering?


This file should actually be kept here to avoid hacker attacks on servers which list contents of a folder lacking the index.php file.


This is inconsistent, as the `LICENSE` file states the Unlicense, not the MIT license.


We could run into encoding issues when we mix the css files, we could have issues when users in other languages add tokens that are not part of ISO-8859-1. Or am I mistaken and all internal files are not UTF-8? Then okay, not now.


Well, it's a library not a console application.


Would it be enough to do something like this?

```js
o.when(el, "drop", function(event) {
    if(options.drop && options.drop(event, options) === false) {
        return;
    }

    o.readFiles(event.dataTransfer.files, options, function(files) {
        o.uploadFiles(files, options, options.complete);
    });
});
```

This way, the `options.drop` method can stop the upload by explicity returning `false`, or even change the `options` object.


The change is OK -- very nice!


@vilchik-elena actually I would prefer to remove the prefix on call to ``PHPCheckTest.check``, WDYT ?


This isn't the correct URL. It seems you copied this over from another project.


> We could run into encoding issues when we mix the css files

That's why the wiki page says to remove the fontello comments, by the way.

> Or am I mistaken and all internal files are not UTF-8?

I'm pretty sure this stylesheet has never been in UTF-8. I'm not sure if it would impose an issue for it to *be* in UTF-8, but as of now, there is no reason for it to be in UTF-8.


browser script crashed due to very large amount of data, both for sieve method as for bruteforce method


I don't know, that's why it's so weird. Other code like this has existed since the code generator was added.


It seems that vertx-shell is strongly couple to a single vert.x instance. Does that mean it couldn't be used for new commands that start and stop new Vert.x instances? That might be useful in the future.


teste


Adding an appropriate test (to dialyzer/test) that shows its effect would also be nice...


@lindamartin we should rename `NAMESPACED_NAME ` into `NAMESPACE_NAME `


Good


Was testing out this isn't finished yet, renaming the dialog element causes an issue with `cancel` in the controller not actually closing the dialog I am working through.


done via `match` as is in other places


This chunk probably is an error, isn't it?


Why would you want to do something as silly as that? ;)


This is not really about secondary, but about borders. I would prefer if this could be implemented in `_button.scss`.


@lindamartin i would rename this to smth like "getThrownClassName", otherwise it's not clear from name what this functions does


Isn't this redundant with tox? The [py.test documentation](https://pytest.org/latest/goodpractises.html#integration-with-setuptools-test-commands) (where you seem to have copied that code from) even says that it's usually better to use tox.


Well, we do need to sorta decide what our ultimate plan is for ESDK compatibility.  I'd prefer not to get rid of it completely and then have to reinvent it later for the compatibility layer..  On the other hand, it arguably doesn't belong in core/app_main.c if it's just a compatibility wrapper..


NTS: Point `repository`, `homepage`, and `bugs` fields back to `adamschwartz/chrome-tabs`.


Also what is easy for maintainers might not be for others.


We should remove this section or add a comment why it's a comment


e6bb6005ecfad6271b814d46132de80da09eac80 and 167cbdd17c5bd12cc0616621f0d469ed96b0bd41 replace these with iconfont icons.

I'd like to stress again that I agree that replacing all the icons with SVG icons at some point will be the right choice, but it should be done using SVG sprites and all at once. As of now, SVG sprites not ready IMO.

As for adding new iconfont icons, please see https://github.com/s9y/Serendipity/wiki/Add-icons-to-the-backend-iconfont.


Well I was totally wrong about the destroy damage, I just didn't have the
right info. :)

If you have the time, take a look at:
https://github.com/mikesa83/en/blob/master/db/update_079-pending.sql

Starting at line 307; I think everything is sourced, it doesn't create a PR
for this file, it just let me update it, but 307+ is all new crap. Of note
I have not tested the page typeuse change, but it doesn't work now, and
should not break anything if that is incorrect, I believe it is though.

-Mike

I'll have some free time again at the end of the week I hope, and maybe can
knock some of the annoying persistent but less crucial bugs out.

On Mon, Oct 5, 2015 at 11:30 PM, everial <notifications@github.com> wrote:

> Cool - I remember we've often gone back and forth about what seems "more
> right" - nice to know you found something more official. =)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mikesa83/en/commit/c915bac6cb6b77a4cf9f59359ed6e9543d1f89a2#commitcomment-13606594>
> .
>


here's what I've been told: for backward compatibility, if that Property has no value at all, that transport is assumed to support DTC because that's what the "old" transports did back then (MSMQ, SQL). Except for RabbitMQ, which doesn't ;) (as well as any other 3rd party transport which doesn't set that property and doesn't support DTC. But we're not aware of one)


This only works on Linux, the agent now no longer compiles or runs on anything else.


What do you mean, exactly?


(Yes, I realize that the “to start/to end” icons do not work very well with the “left/right” icons.)


Tiny nit here, but we can fix this later:

I don't think it's a good idea to intermix the semantics of the return value with implied context given how other parts of the application function (i.e. the wallet). Strictly speaking, this condition indicates an error which should be propagated up the call stack and handled at another level so that this function can be more readily reused in different applications.


Why? It won't show up in the manual.


@lindamartin Ok, let's do this. But again, if we do this, we should do this everywhere


`uservalidation.ValidateUserName, uservalidation.ValidateGroupName`.  I had debated factoring those out into a referenceable package.  How would you feel about doing that in general (not specifically for this pull)?


Nice work! :+1:


Hah! Smart. :P


Correct

On Tue, Oct 6, 2015 at 1:48 PM, Tim Bussmann <notifications@github.com>
wrote:

> here's what I've been told: for backward compatibility, if that Property
> has no value at all, that transport is assumed to support DTC because
> that's what the "old" transports did back then (MSMQ, SQL). Except for
> RabbitMQ, which doesn't ;) (as well as any other 3rd party transport which
> doesn't set that property and doesn't support DTC. But we're not aware of
> one)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Particular/NServiceBus/commit/efa47d2546c5667a299b74caee953aa2db8dc5d2#commitcomment-13612999>
> .
>


Just to spare time researching git history why it's a commented section


First of all, this won't fit into int16_t.


That sounds like a material for a bizzare desk case ;) Wouldn't it be safer to check if it's a transport that we know supports DTC instead?


You can use while (1) and return

However, binary search tree is cool


I'm actually thinking we should probably be going the other direction with this (making the newlib stdout stuff do CR/LF conversion the way the old ESDK routines did).  It is rather standard behavior and is now another way in which we potentially deviate from what folks may be expecting based on previous experiences with (I suspect pretty much all) other SDK/OS/toolkits for the ESP8266..

Actually, this also brings up a larger issue, which I think I'll open a separate ticket for:  We should also have a way for people to override the default stdin/stdout if they want to (for example, redirect stdout to UART(1) instead, or to some completely non-UART thing).


We're not running that check in our ci. I think you can safely remove that. ;)


same as above


Could do with a high-level comment that we do the read and write separately to minimise the time we're holding the write lock?


this is probably debugging code. Should not be in the PR


Why a class?


OK, I'm merging this as it's the current approach, if we start using travis again we will have the same behaviour. thx!


Regarding the flooding attack, an important consideration is that no state is allocated upon ADD_ADDR2 reception (differently from the case of SYN+MP_JOIN). This means that there is no DoS effect, i.e. during/after the attack it will be always possible to advertise new addresses when the ADD_ADDR2 packet contains the proper MAC. The only outcome of such flooding attack is limited to CPU overload, as you mentioned.

To solve the CPU overload, I was thinking about waiting some time when an ADD_ADDR2 is received before accepting new ones. During this waiting time, incoming ADD_ADDR2 would be discarded. After all, address advertisement on MPTCP is a "best-effort" procedure. However, this might enable an even worse DoS attack: a continuous flooding of incorrect ADD_ADDR2 packets might just trigger the waiting time continuously, letting the correct ADD_ADDR2 "starve".

Perhaps it would be useful to setup some tests on the flooding attack to understand the real impact on the CPU.


aDD


Бинарным файлам в репозитории не место


Great!


INT_PRECISION again


more INT_PRECISION nonsense (:


nice Shay. I thought I liked puzzles :)
nice going buddy--good read!


unless fo.valid?


I just looked and this function isn't part of Espressif's SDK API - it's not in any header or source file that I can find in the Espressif SDK releases.

Probably the most likely candidate for someone who has been using it is someone using esp-open-rtos where we added a prototype for it. At this stage of development I feel OK breaking the API for that.


Also, your change breaks the parser. Double quotes ;) Reverting that now.


Мусор в репозитории


@lindamartin I think it's better to rename method or pass FuntionCallTree to it (else signature seems weird)


Can we convert to decidegrees everywhere? Too many conversions using 10.0f for my liking.


why "r || " ?


@lindamartin And what about functions considered by this rule? Is it possible to declare user functions with the same name?


@eddelbuettel - It builds 100%, no problems. :)

I agree, the pitdate should default to today() if empty, but should always be included, otherwise the data is a little "lost", not being anchored in time.

If you need a result that has a date/datetime column, I can construct an EQS example and share with you. Would need your Bloomberg ID so I can MSG it?


The event listener should probably be removed in `destroy()` as well, along with the `undelegateClick()` call.


@lindamartin does this method really require 2 parameters? Second is a part of first


There you go Dan


There you go https://github.com/REBELinBLUE/deployer/commit/09b6918acc54185591cc6ef5b441bead73141ada


@auchenberg Good point. I usually never put those in my modules but instead try to strive for a working and realistic example directly in the README.md file - but I guess that puts a limit on the complexity and number of examples. I'm in the middle of a major refactoring of the module - I'll find a good place for the examples while doing that :)


It's not really coupled, however accessing a Vert.x instance remotely needs Vert.x for networking (telnet,ssh).

A command could provision new Vert.x instance independently of the Vert.x instance it uses.


Can these two be squashed into
`metadata = {"path": os.path.join(self.workflow.source.workdir, EXPORTED_SQUASHED_IMAGE_NAME)}`
?


// TODO: Validate URL and only accept it if source is set?


@lindamartin I would suggest to move these strings into class constants


I just wanted to say that we should add `>= 0.4.4` to `Requires: python-docker-scripts` in the spec file, but I don't see such a requirement there at all. How come ?


So you are a gitmaster now :)   Nice.    'git-bash' on Windows, or their fancy 'github desktop' ?

So how about if we always return the date as the first column? Ie instead of
```r
if (!is.null(date)) res <- data.frame(pitDate=date, res)
```
in `beqs.R`, we always set it?

I'll get you the account handle once I get to the office.  Likely Kevin Saunders @ Ketchum Trading but I'll check.


LGTM


c will never be 0x00. So this condition should be:
```
if ((c > 0x00 ...
```


Nope. If it's absolute, there's no need to make the path relative to anything. Making an absolute path relative from a relative path results in an error.

Did this change break something on you?


Why did you declare a nLockTimeFlags variable when it never gets modified? Why did you redeclare it in this scope?


nice  change !  much better now!


It should also probably be `~6.0.0@beta` instead of `6.0.0@beta`...


@pfpmeijers was meant imho


$ java -jar target/dependency/webapp-runner.jar target/<appname>.war


Changes commited to Remove_logon_logic_from_Program_cs branch.
Could you please look at this?


I don't think `dictionary_map = discover()` is straightforward.
discover for what?


source and source.present?


Yup!


you've made `dict_list` become a `tuple`
so I don't this this var should be called `dict_list` anymore.


The remark is not concluded :cry: !


Ok?


Rename as `discover_dict()`?
or what's your preference?


github desktop, it's awesome! Not a gitmaster yet... still earning my stripes with a few hiccups and bumps...

Returning the pitdate as the first column is the ideal solution.


ddd


do not use instance variables when you have attr_accessor


it looks like `zdicts` is same as `dictionary_map`?
(they both are the return object of `discover()`)


@igbasov, can you explain this?


Yes, that's what I would have suggested.

That's why we make releases instead of having everyone run off master :-)


repo.try(:present?) and commit_has.try(:present?) and ...


@lbianc actually the lowering is never called. I have another refactor using vasm and lowering all instructions who calls VptrToReg. But, i have a problem with vret and storesd because they call instructions that are not "vasmfied".


This already fixed in later commit (ad4d1d0)


Please use a placeholder for URL on t() function.

https://www.drupal.org/node/322774


Can you send me a sample data set? I'm open to adding this in. If you needed it someone else may at some point as well.


Merge


Hmm... i will consider `tuple` as kind of immutable list
So... what's your naming convention?


The whole point of the viewholder pattern is to reduce calls to findViewById and reuse those references...


@lindamartin As discussed, this field should be renamed. May be `importedNames`?


Seems so, in the latest version this does not appear (the above should not even run).


#6 Done


Hi, please make sure that your changes also work on Windows.


#7 Added some more around the URL helper


Thanks my mistake.


I'm afraid the ambiguity is still there. "Details" only link to the GPLv2 license text and not, as one would expect, to a list of files that are GPLv2+ and other that are not. Are there actually any GPLv2-only dependencies?


I may have still been on kitkat when I made that change.. but it seemed the code that loaded gralloc.drm.so was looking in /system/lib/hw but originally gralloc.drm.so was ending up in /system/lib/


Ouch, this needs to be changed


Indeed. Add a property to keep track if constructed with selector/element?


@lindamartin i think this piece could be replace with
```
String name = tree.namespaceName().fullName();
if (RAW_EXCEPTIONS.contains(name)) {
...
```


Ah damn, essentially it links to the wrong file.

> Are there actually any GPLv2-only dependencies?

No.


Both of your projects would prove very interesting—lots to dig up! I'm wondering about the exigencies behind the <figure> and <figcaption> tags. @Lhyen, that's a great resource to start digging there. These tags are also linked to print media, as such conventions exist particularly in scholarly docs.

@naka0157, you wondered if every single <figure> needs a caption. Well, in academic texts, yes. :-) I wonder if these tags are also supported by web accessibility tech.


Thanks @bramstein above solution is working fine. I have added this code snippest on my existing code this is working fine. Now webfont  loader is working fine with  unquoted numbers in font family names.


Bom dia,
Realmente vi ainda ontem que vários programadores estão reclamando sobre problemas no envio com esta nova tag. Aparentemente é problema na sefaz de sp vou manda um questionamento para eles e lhe aviso.


LGTM


maybe just `dicts`


Good reference to have :+1:


:blush: :grinning:


maybe change it to something like `get_dictionary_map()`, `get_dictionaries()` or somthing else?


Quick question: Why doesnt the BlazeDataSoruce/downsampling work any longer? Will there be a fix anytime soon?


Changed "start.sh" to "pdns.sh"


yes :)


This was my first pull request on github, I got some conflict warning but I lost in its interface.
Sorry if it hard to merge


I guess that's similar to my own conclusion (after my OP): While there's no extension API for replacers yet, things are bound to be somewhat hacky. However, since this API for loading response handler extensions does exist already, might as well piggyback on that hook for the time being.


ok, I will rename them into `get_dictionary_map`


hummm...

you want to emit something about `dest = src1 != 0 ? src2 : src3`, right?

you want something in x86 like:
```
test src1, src1
cmovnz dest, src2
cmovz dest, src3
```

First you must need to know that *CMOVxx* are expensive instruction, so you'd better to have:
```
mov dest, src2
test src1, src1
cmovz dest, src3
```

if *src2* and *src3* can be memory operand, using *cmovnz* would not save you a read access to operand *src2* because it is done *BEFORE* checking whether *dest* must be updated. So using a pair *cmovz/cmovnz* instead of *mov/cmovz* is not a smart choice in fact.

Besides, if one source is a constant, you can put that constant in *dest* and save an extra instruction:

*src2* is constant:
```
mov dest, constant
test src1, src1
cmovZ dest, src3
```

*src3* is a constant:
```
mov dest, constant
test src1, src1
cmovNZ dest, src2
```


If its reallly bothering u , be free to remove it.But people that download the mod from here will be affected.


fine :smile:


не очень хорошо. Что будет в случае с классом `Foo::Bar`


+ lower_case нужно


Ok


No, but then when is it interesting to have a relative path here?


pet peeve: reassigning reference parameters.  Reassigning the reference doesn't mutate the original, but whenever I see it I get nervous.  Mind choosing a new name?


множественное число было бы тоже круто, ну да ладно


@lindamartin you forgot to test name value (which probably should be done on line 80). So you got issue here
```
use Foo\SomeException as Alias;

class ClassName {
    function test() {
        throw new Alias();             // OK but there is issue
    }

}
```


Good point.


why not ¯\_(ツ)_/¯


@vilchik-elena Indeed!


It has become an important mainstay in web design and development. It's a document that helps designers consider a particular state of the DOM before they begin to make their own mods by writing their own CSS. It would enable you to learn a lot about browser behavior.

One more note: there's more than one way to do it. So, to compare strategies would be interesting. Here aresome links:

- [Eric Meyer's post on his website about reset.css](http://meyerweb.com/eric/tools/css/reset/): Also, links to the original bog post
- [Eric Meyer's reset.css on CSS Tricks](https://css-tricks.com/snippets/css/meyer-reset/): good for the comments
- Short [post](http://cssreset.com/scripts/eric-meyer-reset-css/) that some designer put out there about it. Good for the comments, not the post content.


There is no boost-random package in SUSE. The dependency appears to be satisfied by boost-devel.


@SuperstarGamer how is it?


Хочется посмотреть на обработку эксепшенов. Я их не обнаружил в коде. + сделай плз pull request с исправлениями. проще читать и коментить.

В целом не плохо. Спасибо


removed!


@lindamartin let's refactor this condition so that it would be more readable
```
return isImportedGenericException(namespaceName) || isGlobalNamespaceGenericException(namespaceName)
```


I did it on purpose because I like the capability to run `setup.py test`.


Good catch. I'll fix that.


Later, for SUSE the groupadd/useradd logic will be moved into a separate package and this code will be replaced by a dependency on that package.


Also a copy-pasta from fictionary.


In "src/webserver/src/WebServer.h", if you remove #include "KnownFile.h" you need to add #include "kademlia/kademlia/SearchManager.h" to compile WebServer.cpp without errors, otherwise map and some wxWidgets define will be undefined.


I checked many sniffs, I cannot find any summon action or an area trigger involved, if you have such data please provide it.
@Kittnz: it's not the same npc: http://www.wowhead.com/npc=37528/spire-frostwyrm-ambient


I was trying to add a comment after the paragraph about the user advocates, but for some reason the blue box disappears about halfway up the post.

Anyway, I too noted the mention about the user advocates being non-technical and mostly women. I wonder if there was some arrogance on the part of the technical experts at play here. Since, as I recall, some of the end users were to be incoming freshmen, so they don't have technical experience yet. This would lead me to believe that the user advocate input would be valuable.

I also wonder if the mention that they were mostly women means Haas noted, and felt, gender discrimination.


Thanks for looking through the changes. This would (hopefully) have been cleaned up in a later commit, but I had to stop working on this at the time, and I never quite got back to it. Ish is something I only occasionally come back to because it's my least useful project :-)


Cannot_hold seems redundant, as we already have can_hold.


In section 8.8.3 of ISOBMFF , it says that the flags are stored on a 32-bit integer, with the following structure:
```
bit(4) reserved=0;
unsigned int(2) is_leading;
unsigned int(2) sample_depends_on;
unsigned int(2) sample_is_depended_on;
unsigned int(2) sample_has_redundancy;
bit(3) sample_padding_value;
bit(1) sample_is_non_sync_sample;
unsigned int(16) sample_degradation_priority;
```
HTH


@racardoso Good to know, this refactor will help! I've just made a temporary fix using the tmp register.
The lowering is called in the function ```optimizePPC64```, i.e., before emitting instructions.

About the lower call, there are three moments:

1. Creating VASM
The VASM is "created" using the notation "v << ...", where we "buffer" the VASMs that we want to emit.

2. Optimize
The function ```optimizePPC64``` is called and VASMs that have a lowering function are replaced with the VASMs that are inside the lower function.

3. Emitting VASMs
Here the lowering cannot be called, since it is used to replace VASMs, and at this moment we already have all VASMs created and we cannot replace them.

So, the lowering is called, but not at that moment (function ```VptrToReg```). Just to clarify.


Oh and this new class introduces a disgusting box around the room. I would try to change it to what my inline-css previously achieved, perhaps you know the name of an appropriate class.


One problem is that given that MP4Box.js currently creates a fragment for each frame, there are some fragments which don't start with a key frame. Chrome and Firefox seem to have issues with those fragments.


@lindamartin anytime i see `namespaceName.name().text()` i'm wondering is it right that we are ignoring `namespaceName.namespaces()`. I don't know right answer here :) Do you?


You'd claim that anyone inspecting this code should easily be able to reason about why this action is correct?  If you have any doubts, start this section with a comment demonstrating on an example.


Merge directly on github


aaa


The product needs to be configurable. You don't want to show RHEL users Fedora bugs and vice versa.


yes, sure this looks OK, but I wonder why I return `null` from `getExternalSourceArchives()` when no external source archive is found. I should return the empty set. I'll probably change this.


How you find it?) With static analysis tool?)


@lindamartin could we improve readability of this method too?


@bdunne : this does not seem right, can you fix this, please?


I think it would be better to move the `JobManager` retrieval logic into the `JobClientActor`. Then the `JobClientActor` would be responsible for finding the current leader and resubmit the `JobGraph`. The `Client` would simply wait for the `JobExecutionResult` or an `Exception`.

That way, we would unify the retry logic in one place which would reduce a little bit of complexity.


Ok, I already corrected that. I'm almost sure it had to be with only one space, although it could be the same case as in median, that it required an additional one.


I would add the issue number as we do this for all fixes. `(gh#SUSE/machinery#1154)`


refactoring and revise the display method for displaying the last row.


5628 & 5629 Alliance side
5630 & 5631 Horde side


Did you mean to link to [this commit message](https://github.com/mpv-player/mpv/commit/a5507312093adb99fc80f09276561eb333999cba)? I'm sorry but when I read that I still don't feel enlightened. It first talks about removal of a GPLv3+ library which permits going back to GPLv2+ from GPLv3+. Then it says that this is possible due to the removal of **GPLv2-only** code (i.e. not GPLv3+), and now the readme says that it's "mostly" GPLv2+. So it's still not clear which of the license versions it's OK to distribute under.


close #4


@lindamartin Please fix bug (see comment on line https://github.com/lindamartin/sonar-php/commit/c5d7ab56f5a0a261a2d14276d1f9da0d11d6534f#diff-b22b424147fc56ca293e1c489a92c20bR110) and do some refactoring. Otherwise looks good!


LGTM


On 05/10/15 21:43, Sebastian Moeller wrote:
> Allow all cake overhead keywords to be used in any
>  order
>
> The overhead keywords for cake, basically symbolic names for overheads
> commonly encountered at edge links to the home have had a peculiar
> under documented feature in that the sequence of these options was
> deciding which of the optins was used. But this behaviour did not
> occur for all keywords (mainly to those that can be considered full
> overhead specifications like pppoa-llc). A few keywords like via-ethernet,
> ether-fcs and ether-vlan could would still be valid to modify the overhead
> value given by the full-overhead keyword. This behaviour while consistent
> is really difficult to explain to new users, so this commit changes the
> logc of all keaywords to be additive, so all evaluated in place.
> This allows non-sensical specifications like "pppoa-llc pppoe-llc"
> to be used, but if the user asked for it they probably wanted it...
> Note that the keyword "raw" will set the overhead to 0, but any keyword
> value occurring later in the command is still applied; the same logic applies
> to the "overhead N" option to specify the overhead numerically.
>
> Signed-off-by: Sebastian Moeller <moeller0@gmx.de>

Ack.  Will wait a couple of days before applying whilst the API/ABI
changes go in (or not) and deal with resulting issues if any.


>Did you mean to link to this commit message?

Why would you think that? No, the Copyright file.


this does not seem to be proper use of gettext, please, fix it


Yeah, let me a little time to try!
At first sight it seems great!
From my point of view, I'll modify the "extra" to leave in modules such as "Fixed aspect ratio" and the new that may come.
Thanks @andri !


Doesn't work :(
Some debug output is shown below:

    Testing sorted-help-message
    ------------------------------------------------------------
    stdout                                                      passed
    stderr                                                      failed (diff: /c/projects/ctags/Tmain/sorted-help-message.d/stderr-diff.txt)
    run.sh: line 47: syntax error near unexpected token `('
    run.sh: line 47: `    <(print_help | extract_short_options) \'
    exit                                                        failed (diff: /c/projects/ctags/Tmain/sorted-help-message.d/exit-diff.txt)
    2

Any ideas?


Rayner, N. A., Parker, D. E., Horton, E. B., Folland, C. K., Alexander, L. V., Rowell, D. P., Kent, E. C., Kaplan, A.  Global analyses of sea surface temperature, sea ice, and night marine air temperature since the late nineteenth century J. Geophys. Res.Vol. 108, No. D14, 4407 10.1029/2002JD002670

HadISST ref.


Hello,
         I used this <http://vimawesome.com/plugin/flattened> website.
Yes, the spelling error too.
ᐧ

On Mon, Oct 5, 2015 at 7:14 PM, Romain Lafourcade <notifications@github.com>
wrote:

> From what I can see, the instructions given on that site:
>
> Plugin 'romainl/flattened'
> :source %
> :PluginInstall
>
> seem correct.
>
> Where did you get that url? It has two errors: the 1 in place of l and
> flattended in place of flattened.
>
>  https://github.com/romain1/flattended.git
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/romainl/flattened/commit/6e8448c21380a1151510355f12e9856a5af5990f#commitcomment-13587388>
> .
>


why here the scope "uid" is hardcoded? I used different scope in our app, and get exception here: ```KeyError: 'uid' ```


This comment is not important - what is important is to mention that c_start() could be potentially called multiple times (and not only once) if the output from show routine were to overflow a page.
Thus it needs to handle the "restart case"


Could you also add a unit test for the multi sort, please?


Hello World


ah. right :+1:


Hi, in my environment, an error raised here.
I think this should be like `prototypeSlots.forEach(slot() => {` ?
Please check it.
Thanks.

```
Using Jest CLI v0.5.9
Failed with unexpected error.
/home/ubuntu/gamba/client/node_modules/jest-cli/src/lib/moduleMocker.js:133
        prototypeSlots.forEach(slot => {
                                    ^^
SyntaxError: Unexpected token =>
    at exports.runInThisContext (vm.js:53:16)
    at Module._compile (module.js:413:25)
    at Object.Module._extensions..js (module.js:448:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at Object.<anonymous> (/home/ubuntu/gamba/client/node_modules/jest-cli/src/HasteModuleLoader/HasteModuleLoader.js:21:20)
    at Module._compile (module.js:430:26)

at Object.Module._extensions..js (module.js:448:10) cd client && node_modules/jest-cli/bin/jest.js app/scripts/helpers returned exit code 1
```


Teste es gut, bevor ihr Easy-Thumbnails mit S3 verwendet. Offiziell unterstützt es keine Cloud Storage.


To finish https://github.com/dcorbacho/rabbitmq-management/commit/8f571f2bb2e8ff304e3a7bd8c26c77a177a46639#diff-ac90722518a5f0098e156e925c380a01R9 this function should and support the pagination, this function should return the total number of pages. i.e. `{pages, Pages}`


The div element hasn't really changed—much. It behaves roughly the same in HTML5 as prior standards (of course, I bet there's some iterative changes and variation, but I'd have to dig). Yet, semantically, the div is intended for creating containing block elements that do not necessarily fit the "molds" or "boxes" that HTML5 provides. @CLC-4662W is correct that you can ID a <div>, providing your own semantics to the generic containing block. Think of the div as a wicked cool customizable lego block.

Another dimension of this conversation about HTML5/<div> tags: Where do you think the idea to create these other semantic tags came from? Consider <nav> in the scope of our writing practice framework: Recurrent, goal-directed writing activities within a particular broader culture and smaller communities/groups. Web designers and devs used to write <div id="nav"> and variations of it (See a post about <nav> at [HTML5doctor](http://html5doctor.com/nav-element/)). Such recurrent writing practices led communities to agree that such tags should be standardized. Of course, the details is where things get really interesting, but I hope this adds some more thought-fodder for everyone. :-)


Ah, that's probably just how the Java REST runner renders the JSON attributes.


Is there a good way to get a free port with this block? I added a port getter that can't be merged and feed in "0" and "" to port field and wind up with port 0 returned.


@tallforasmurf could you comment? Should it be put back?


Hmm so i dug into seq file internals and finally figured why my simplistic patch would cause the looping as you reported last time. c_start() can be called multiple times if output overflows. This patch looks good. You can add my ACK !


> Thanks for the new report and suggested fix

You are welcome. Thanks to @lorensen for trying out potential fix and reporting back.

> I guess I'll wait a couple of days before I commit the "clang guard" changes to the official DCMTK repository.

Makes sense.


add gitter badge


ATM Skeleton for project.


Yes I agree. I thought the same.

Sent from my iPhone

> On 06 Oct 2015, at 14:44, David Festal <notifications@github.com> wrote:
>
> yes, sure this looks OK, but I wonder why I return null from getExternalSourceArchives() when no external source archive is found. I should return the empty set. I'll probably change this.
>
> —
> Reply to this email directly or view it on GitHub.
>


hmm actually i guess we can just drop this hunk entirely, if all callers use get_reqid()?


@nickandrew Thanks Nick, I will put this on my TODO list.. As I said I will be using DS18B20s


Should we be casting like this? I've just been promoting to `Object`.


First commit :heart:


Trailing period?


Do you think for consistency the naming of all dialog elements should change to have "ai-" appended to the dialog elements? Just a thought I wanted to share.


Yeah I'm not sure about things like this. If String#freeze doesn't return a DynamicObject though, this is probably the least of your troubles. In this case I guess I can just return Object though. Will fix.


Typo? Or have you been using `dupped` right along and I just didn't notice? It's local, so not a big deal. I'd just rather match if I'm not conforming (I've been using `duped`).


change:void setup() {
  ESP.wdtEnable();
to:void setup() {  ESP.wdtEnable(WDTO_0MS);
NOTE: This was caused by a change to the Arduino IDE. Version 1.6.1 did not require the argument WDTO_0MS but 1.6.5 does.        From: qoreyshi <notifications@github.com>
 To: internetofhomethings/ESP8266AnalogMux <ESP8266AnalogMux@noreply.github.com>
Cc: Dave <surfnturf57@yahoo.com>
 Sent: Monday, October 5, 2015 11:45 PM
 Subject: Re: [ESP8266AnalogMux] Update AnalogMux.ino (8d8e814)

hi
how error code(thanks for you):
AnalogMux.ino: In function 'void setup()':
AnalogMux.ino:403:17: error: call of overloaded 'wdtEnable()' is ambiguous
AnalogMux.ino:403:17: note: candidates are:
In file included from C:\Users\Administrator\AppData\Roaming\Arduino15\packages\esp8266\hardware\esp8266\1.6.5-947-g39819f0\cores\esp8266/Arduino.h:236:0,
 from C:\Users\Administrator\AppData\Roaming\Arduino15\packages\esp8266\hardware\esp8266\1.6.5-947-g39819f0\libraries\ESP8266WiFi\src/WiFiClient.h:24,
 from C:\Users\Administrator\AppData\Roaming\Arduino15\packages\esp8266\hardware\esp8266\1.6.5-947-g39819f0\libraries\ESP8266WiFi\src/ESP8266WiFi.h:32,
 from AnalogMux.ino:13:
C:\Users\Administrator\AppData\Roaming\Arduino15\packages\esp8266\hardware\esp8266\1.6.5-947-g39819f0\cores\esp8266/Esp.h:84:14: note: void EspClass::wdtEnable(uint32_t)
 void wdtEnable(uint32_t timeout_ms = 0);
 ^
C:\Users\Administrator\AppData\Roaming\Arduino15\packages\esp8266\hardware\esp8266\1.6.5-947-g39819f0\cores\esp8266/Esp.h:86:14: note: void EspClass::wdtEnable(WDTO_t)
 void wdtEnable(WDTO_t timeout_ms = WDTO_0MS);
 ^
call of overloaded 'wdtEnable()' is ambiguous—
Reply to this email directly or view it on GitHub.


Yeah typo.


It's no really needed the default_url, I can use the default structure. The point is that maybe someone create a new menu with an existing page, in that case the URL maybe doesn't have the default structure (/page/view_name). Then if I move the menu to other one who has a view with seo_url it's ok but if later I move the menu again to other one who has not a view with seo_url the default structure could not work. I can change it to default structure anyway and assume the default structure it's going to work always.


No more fixes needed.


Yes, absolutely.


OK


only raspbything.groovy worked as the device type


How exactly? It is just a list of sort expressions, it does not have any logic behind it. We are not unit testing plus/with facet/queryFilters/resultFilters/facetFilters either, because they are just a lists. How are you unit testing the multisort in query?


I guess you'd get a `ClassCastException` here, which could be easier to debug that propagating something further down the chain.


You mean special names such as "connection", "unknown" and "stored"? Well, I don't see a reason to prohibit these I must say, there's no benefit in that. Moreover we explicitly say that the names aren't unique anyway in the man pages, and by extension you can say that this applies to these names too: there's no guarantee that the special cases are the only cases these names are used.


If `tst[0]` is usually a string, why do we have to do the `issubclass` check as well?


Wow, I don't even remember this.  :heart_eyes_cat:


aha, disregard this. I see it is fixed in next commits.


Just if I add a sort with "plusSort" that the "getSort" is appropriate.


Oh, OK, that's a nice catch. My bad.


Can you give me an example of the return value provided by your oauth server token info url?


In general I think this is good, but I think still think we should tag the non OP-TEE gits also, since we are talking about doing a **stable** commit. So I suggest that the final thing before tagging and merging is to do a
```
repo manifest -o my_stable_xyz.xml -r
```
And use the automatically "_produced_" commits for all git's expect the ones for OP-TEE gits, I think we should replace those with `refs/tags/1.0.0` as indicated above.


NP: "it is usually **a repository**" or "**they are** usually repositories", and "SUSE" (!)


this is a bit ambiguous - but thats fine - I can fix up some of these things in the end myself !


Lets us add a build time check in C file as it doesn't work in header file.
In arc_mmu_init() at the end
+       /*
+        * vmalloc size sanity check,
+        * Can't be done in processor.h due to header include depenedencies
+        */
+       BUILD_BUG_ON(!IS_ALIGNED(CONFIG_ARC_VMALLOC_SIZE, PMD_SIZE));

Please add that and then my Ack !


Yes I get the with  .debug areatriggers
You have reached areatrigger 5630.
You have reached areatrigger 5631.
You have reached areatrigger 5629.
You have reached areatrigger 5628.


@abnero DWMR. The new imageWidthConstraint must be added to the contraint collection of the view in order to take effect.

You can see that code and the effect in the app in this screen shot.

![screen shot 2015-10-06 at 9 00 19 am](https://cloud.githubusercontent.com/assets/9028552/10309151/ea07664c-6c08-11e5-9cd7-2fc13bcd0dda.png)


* Be sure to use `jQuery` version 2.1 or newer when using `AngularJS` 1.4
 * `jQuery` 1.x is not officially supported
* Be sure to load `jQuery` before all `AngularJS` scripts, otherwise `AngularJS` won't detect `jQuery` and animations will not work as expected


Ah, now I see! That is very clear to me (as long as the link is fixed). Thanks!


Yes, it should be const *and* static.

If the field is declared non-static, then it is allocated on the stack, hence the "const" doesn't really get us much, since the compiler has to push the field onto the stack when entering the function. However, if we add a "static" then the field becomes a global constant, which can be placed in a read-only section of the ELF binary and thus mapped in, without any further work during execution.


Toni


instruction SETxx never sets the bits 8-31/63 of a word, so be aware if a PPC instruction always implies that zero-extension of a register when dealing with 8-bit operations.


1.
2
3
4
5


Why was this changed? Now every import I had that used to `import Something from 'app/folder1/folder2'` breaks.


Actual related issue is #28


thanks, updated


@martinpovolny Please assign to the best person to fix it. Thanks


any reason? or just cleaning hacks... i have that too...


Give it a try. I don't know if we run in troubles with "... but if later I move the menu again to other one who has not a view with seo_url the default structure could not work ...". Lets see.


Closes #10


we should maybe define ````Object { asView { ^this } }``` instead? Or some reasonable subclass.


@epwinchell : I figured out there will be more like this one so I'd like to find them all and get fixed.

So forget about it, I will hopefully fix it everywhere at once.


@martinpovolny Which part isn't right?  It looks to me like it does the same thing it did before.


This revision is clean.


:thumbsup:


This doesn't do what was requested. The result should be `[0, 2, 4, 6, 8, 10, 12, 14, 16, 18, 20]`.


I think it is OK to remove this method in 3.8. We shouldn't forget to note it in the News file then.


Hmm, Isn't `im_func` an old name for `__func__`?


What's this?


Yes, Apple Watches are in the same provisioning profile.


It's string only when the discovered test is INSTRUMENTED. Otherwise it's a class.


Also, the bare `%` characters in the comment cause RPMLINT to complain.


I had problems with the blue box again. It seems to be related to clicking the "expand" button. Anything that was added in the expansion doesn't get a blue box.

Along with her discussion of Vygotsky, Haas mentions that the Bronte children were an exception to the statement that individuals do not make the tools and signs from scratch. I had to look up my history on the Brontes to learn that she was probably referring to the worlds they created and sometimes referred to in their novels. Once I learned this, I thought about Tolkein. In the Lord of the Rings trilogy and The Hobbit he created many tools and signs. Maybe I'm missing something in Haas's point, but I think Tolkein is also an exception. Maybe to varying extents all fiction writers are exceptions.


Yeah, my stuff is in I think: https://github.com/json4s/json4s/pulls?q=is%3Apr+is%3Aclosed+author%3Ayanns


@jdegraft when will you remove these?


yep, __func__ is 2.6+ (and IIRC we need to be 2.5 complaint, right? Or 2.6?)


功能尝试


I prefer `size()` to `length()` for consistency between types. However, if you also do a lot of Java development, it may make sense to stick to `length()` to avoid an extra burden when context switching between Java and Groovy.

I like how you've made the method deterministic for when word lengths are the same. I assume you know that for the purposes of the exercise, you can just do `sort { it.length() }`? Just want to check.


i made a application to make this thing easy on linux :) here it is https://github.com/keshavbhatt/UThrottle


Let's move this function to the `BindingSystem` class as a static method and call it `initializeFromContainer` so that you can have this api `BindingSystem.initializeFromContainer(container)`. Then, let's make the static `instance` property be a getter that lazily initializes the instance, so it can be called without having to initialize the binding system. The `initializeFromContainer` should then just call the private `instance` property to get the binding system, then register all its services with the container. Finally create a private object key that must be passed to the constructor, so that it is not possible for a consumer to actually instantiate the binding system. You can see an example of this in the logging library, with out how loggers are created.

As an alternative to all that, we could just export BindingSystem as an object, rather than a class. The object would have the same methods as above, but of course no constructor or instance properties. It could have private properties for each of the dependent services and still have the intializeFromContainer method though....I think I might like this idea better. What do you think?

Regardless of the fix, we will need some additional apis for observing/unobserving arrays and maps as well. Once we have that we might need to rename the apis we have to be `observeObject` in order to differentiate it from the other cases....or perhaps we can handle it with a single method that picks the appropriate observation method based on the type? One trick to that would be observing arrays vs. observing the array length property.


integration*


Just checking: are you expecting exceptions here? If not, please log at WARN or SEVERE level. If so, perhaps you should log at FINE(R) level.


// TODO: Validate URL and only accept it if source is set?


You don't need a `DataOutputStream` for this, just a plain old `OutputStream`.


Porque você ta mandando as coisas por javascript???


bah que cagada! Sempre erro na digitação.


@rvonstange DWMR. Keep pressing your advantage!


teste de integração diferente de teste funcional, tem que entender isso..


Para poder executar os eventos na tela, tem outra maneira?


O que isso aqui ta fazendo? Não ta testando nada


If you remove the border-width it can cause a strange moving if another stronger rule has a border-width on mouse over, for example. This is happening on Liferay portal.


então tu sugere que eu mude o nome do teste, para teste funcional? Mas eu ainda terei que utilizar a annotation Integration para poder manter o server up enquanto o Junit executa os testes, acho que não tem problema ?


@lyndonbartlett88 DWMR. Good work on this. Keep it up!


Do not use one character variables, we are not mathematicians.


If we further make _registry a pointer instead of a reference, we shoudl be able to `= default` all copy and move operations.


why public?


Só está testando par aver se o sistema está  up and running, não eh uma boa prática?


Haha, I love initial commit messages. I did book quotes for awhile, but they are harder to look up when you are in a hurry.

Morse code initial commit: https://github.com/TatumCreative/npm-on-tap/commit/6e3f8ff7f5f0dc912fe4acf719357dd2702a706b


@lyndonbartlett88 DWMR. The next checkpoint will take your skills a step ahead. Jump in!


It's unfortunate that we have to go through such lengths to introspect which statuses are good. Maybe we can have more formal identifiers of what a good test type instead of the label mapping?


This is not the way to do it. See the **--[no]-debug** and do it the same way.


@ldoktor 2.6 is fine, since our lowest common denominator is RHEL6


@TheMCPEGamer :)


I think focusing on a particular trajectory as linked to HTML5 video and video codecs might prove to be one interesting route. Another is the <canvas> tag. Both limited the need for Flash for interactivity. Plus, Flash never worked well with search engine / web crawlers. Lots of dimensions to this, which indicates that you will need to consider what dimension to craft in more detail. To help you, consider how your historicizing project can set you up to create documentation in the DITA project.


Now there is a new SwingLibrary release that depends on recent Abbot newly at Maven Central. I don't know if that affects the approach taken here?


why here?


This class shows how to present a unified type across say iOS 8 and iOS 9, to implement a protocol. Yet it's implementation varies depending on the OS. This will only work because the protocol isn't forced to be a generic constraint, which allows the clean typealias to be defined on line 116.

At the moment, however, this fails to compile for OS X, because `SystemAddressBookRegistrar` (which is a wrapper around `ABAddressBook` is only defined for iOS).


@omearac It's hard to say, there might have been an upstream change. It will require some investigation. But our policy is not to ship examples that don't work, and this problem was not deemed a showstopper for 0.10 (typical OSS conundrum: we have no idea what the actual usage rate of BlazeDataSource is, but estimate it to be currently low)

Blaze integration is definitely an important story for us but it may take some time for it to stabilize as both projects are still maturing. Since Blaze exposes a REST API it's possible you could potentially use an `AjaxDataSource` to load data, and it's also possible the `BlazeDataSource` still works with some version of the Blaze Server (AFAIK no changes have happened on the Bokeh side). But we did not have time to characterize what version that might be.


@abadams It is possible to construct non standard ``Type`` objects (e.g. ``Int(15, /*width=*/1)``). From a safety point of view I think it would be better to explicitly handle the ``64`` bit case and throw an error for other unsupported widths.

Slight aside, is it legal for the ``Type`` passed to the ``make()`` method to have a width > 1? I ask because I don't see an asserts checking this and I thought the broadcast nodes were supposed to be used for filling vector lanes with an identical constant.


This code above implements the same behavior without cause any trouble with border-width.


Cheeky! :smile: It's a good idea for date-based work though, so I commend you.


Hi Denis,

The attached config didn't come through. Would you mind updating the hocon
sample instead?
https://github.com/DigdeepDigital/snowplow/blob/kinesis-redshift-dripfeed/4-storage/kinesis-redshift-sink/src/main/resources/config.hocon.sample

Thanks so much!

On Tue, Oct 6, 2015 at 1:13 AM Denis Mikhalkin <notifications@github.com>
wrote:

> Hi Justin,
>
> I've made a push after some testing. It did not go through stress-testing
> just yet (hopefully tomorrow) but it worked on our site, so should be good
> enough at least to try it out.
>
> To run, use the following command: java -cp kinesis-redshift-sink-0.1.1
> -Djava.util.logging.config.file=logging.properties
> com.snowplowanalytics.snowplow.storage.kinesis.redshift.SinkApp --config
> redshift-config.hocon
>
> You'd probably want to have your own logging.properties file - the code is
> chatty at this point in time.
>
> See attached the config file for the example configuration. Fill in
> whatever is not commented out. Note that this dripfeeder uses the COPY ...
> SSH mode, so you'd need to configure your EC2 for Redshift via SSH access
> (as in AWS Redshift documentation). I found that access via Private IP does
> not work (even if requested manually) so try using Public IPs.
>
> Thanks.
>
> Denis
>
>
>
> On Tue, Oct 6, 2015 at 11:35 AM, Justin Ramos <notifications@github.com>
> wrote:
>
> > Sure thing! Thanks.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/DigdeepDigital/snowplow/commit/31cc095de07167ca3b6023bc70571bad8d99ee19#commitcomment-13605002
> >
> > .
> >
>
>
>
> --
> Denis Mikhalkin | CTO | Digdeep Digital <http://www.digdeepdigital.com.au/>
> |
> 0421 171 928
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DigdeepDigital/snowplow/commit/31cc095de07167ca3b6023bc70571bad8d99ee19#commitcomment-13607702>
> .
>


Hello kevzlou and thank you your great job.

In 1.3.3 I was doing :
```
collapsibleItem.addHeader(header);
collapsibleItem.addContent(content);
collapsible.addItem(collapsibleItem);
```
and it was OK.

Now, I try :
```
collapsibleItem.add(new MaterialCollapsibleHeader(header));
collapsibleItem.add(new MaterialCollapsibleBody(content));
collapsible.add(collapsibleItem);
```
but it doesn't display anything.

Am I doing something wrong ?


@mtbc yes I also saw that. This means we should be able to bump `versions.robotframework.swinglibrary` when reactivating the Robot tests for Insight.


Oh, that asymmetry, right.


@bdunne : The gettext call seems wrong.  Unless you have catalog entries for all the values of `button_name` + `" this provisioning request"`.

Should be done this way: https://github.com/ManageIQ/manageiq/issues/3775 - part "String Interpolation"

I figured there are more like that so I want to detect and fix them all at once. Hopefully I get to it.


- Remember to remove not used code


The spread operator in this case is null safe, so the condition is unnecessary.


@rclai yep! This cuts down on making lots of aliases for the root folder. Perhaps we should leave the app alias there too?


// NOTE: Simulating that iframe was processed on the server


can we use NULL instead. please?


I check the book but there is no follow up problem.
But to save more space, one can use a bit vector instead of a boolean array.
If no extra space allowed, one can use to two loops：a for in for, to compare each character with any other characters.


Thank you! Though actually, both of the `open` lines (and any similar) should be changed to import io and use `io.open` -- encoding is not an acceptable keyword parameter in Python 2.  Would you mind doing this and the import `io` if it's not needed?

In Python3, `open` is just an alias for `io.open` so it shouldn't change anything for Python 3.  Thanks!


ghf


is wellformedness enough? Actually ref_to_page returns a None if r0 (the starting page_ref) is not in the s0,
so I would also state that the page_ref (r0) in the initial find state (fs0) must belong to the store (s0)?


Sim. Não tem problema, cria uma classe só pra ter essas anotações e os testes irão extender dessa classe, entendeu?


OK, let's polish that.


Isso ai não é teste, o que você verifica no método? nada, ele só entra na página..


- The greeting example should not be include in the exercise


Sim, to ligado, vou fazer isto no próximo commit!


driver.getelement().sendkeys();


I recommend declaring return types, even in Groovy.


I'm not sure whether I agree, given that running `tox` or `py.test` is actually more convenient, and doesn't require  to add additional complexity to the `setup.py`. But is you really like to have that feature, note that this class can be simplified:

```python
class PyTest(Command):
    user_options = []

    def initialize_options(self):
        pass

    def finalize_options(self):
        pass

    def run(self):
        import pytest
        sys.exit(pytest.main([]))
```


Typo, there's an extra `E` at `TEST_TYPE`.


O vei e já pergunta para os maçons dai, como que eu executo o selenium no snap-ci, tive que tirar o pipe de teste pq tem que fazer umas magicas no snap


Porque o setup tá vazio? Coloca o driver.get(url) aqui...


@After melhor que @AfterClass


- This class is not necesary


I'm cutting down the `app` prefix all over now because this is quite nice. Will there ever be a conflict with an NPM? What happens if you create a root folder called `react`, will it explode?


Irá no próximo commit.


CSS INLINE? OMG


Blz, removerei ele no próximo commit.


Why? I thought we do not care about syntax errors as long as cppcheck does not crash.


tá mas e como eu executo um "click", tem como por ai?


- You can create a delete method that clean up the DDBB


I do wonder whether zero is an appropriate return value for an empty or null list. Something to think about. The test and implementation are otherwise very good.


If we remove all the overriden javadocs (and I agree that they don't add much compared to the parent's), then we might as well not repeat the method definitions. That would leave only `getColumnDefinitions` in this interface.


Se eu te disse que estou com esse problema? hehe, é meio foda


eu botei o afterclass, pq eu estou olhando o selenium executar, pq eu estou verificando se executa o projeto corretamente, mas sim o certo seria @After


OK, that's a bit more advanced than I was expecting :smile: Are you happy with what the annotation is doing?


@troyliu0105
1. 夜间模式现在差文章详情页（webview 加载的）没改，查了半天也没改成功，有空你帮忙看下。（抱歉，代码写得跟屎一样，见谅）
2. 关于 FABProgressCircleBehavior，貌似显示上还是有问题，所以我先去掉了，你也可以看一下。
3. 接下来，我这没什么了，就看你有没有什么要跟新的，明天还有一天，8 号一早升到 0.7 release。


Letis crai tugueder!


test comment


I will create an issue for this, but we must remember to start "protecting" all stdio calls with a #if defined(CONSOLE_DEBUG)  of some sort.


Yes, I'm expecting exceptions if the datasetId doesn't exist, for example.


Right. I'll fix that.


@lbianc: The `addqi` lowering means "replace a addqi with a `ldimmq` and `addq` because the immediate is too big to fit on a `addi` assembler instruction".
When that `if` fails, the `addqi` vasm stays in the code and later on it's emmiting a `addi` assembler. See [here](https://github.com/PPC64/hhvm/blob/b61e333b3038fa5a243414db43ce01e5903b3d8f/hphp/runtime/vm/jit/vasm-ppc64.cpp#L308). I didn't remove the `addqi` vasm implementation. That lowering is not replacing it for all cases.


Often times I do `ENV['CASSANITY_ENV'] || ENV['RAILS_ENV'] || ENV['RACK_ENV'] || 'development'`. Any thoughts on doing that here? Allows being explicit about cassanity env but defaults to rails or rack env for those on rails or sinatra.


thing is working ftrace requires mcount support etc which i don't think works completely.


In case you haven't done it yet, just please check if contiki has a preferred styles for lines like these.


It would probably be better here to recover a panic.


Good catch, @chasehd.  I squashed the commits with a re-worked `generate()`:
https://github.com/ulsdevteam/EzidDOI/commit/2b22e8f4948ebdd52edbc0c8022a2f3bc262729f#diff-322073fb30a56f03637125545de16100R47


Added VCF input to the guided variant calling, now variations across different samples/callers may be compared through FT
fixed some bugs


`print-lisp.c` seems broken on MSVC:
```
windump.exe -vntr lisp_ipv6.pcap
reading from file lisp_ipv6.pcap, link-type EN10MB (Ethernet)
IP (tos 0x0, ttl 64, id 1, offset 0, flags [none], proto UDP (17), length 168)
    192.168.0.105.4342 > 127.0.0.1.4342: LISP-Map-Register, flags [I-xTR-ID-Present, M-Want-Map-Notify],
    2 record(s), Authentication unknown-type,
    Authentication-Data: 0x0000:  0407 7995 4537 1906 836c d1d6 0000 05a0
    Authentication-Data: 0x0010:  0150 1000 0000 0002 2001 0db8 85a3 0000
...
```

Seems to be a problem related to structure packing; `UNALIGNED` does nothing on MSVC.


Unless node-pre-gyp is executed with Electron instead of Node, detecting Electron there doesn't make much sense, the same goes for node-webkit too.


isn't there a way to do this with an interrupt?


Note that this file is no longer used in master and instead we have a plugin:

https://github.com/grails/grails-core/blob/master/grails-gradle-plugin/src/main/groovy/org/grails/gradle/plugin/publishing/GrailsCentralPublishGradlePlugin.groovy#L228


I think it should be a parameter ($enable_epel ?).
Many companies do not allow servers to go out on the internet so they mirror everything locally.
Others might actually allow it and it's indeed enabled on some other modules.
Lastly there will be someone who is accessing yum behind a proxy.
The module should not be opinionated, but give the user the possibility to do things as they please.


It's just a common apk


é nois


nao remove, mas pega o titulo da pagina dentro do assert


I'm not sure this is a good approach. `html_entity_decode` reverses `esc_html` so this function escapes then unescapes and also provides a filter so that anyone can change the value after the escaping (should be late escaped instead). None of the built in functions can handle escaping CSS. As mentioned here: http://wordpress.stackexchange.com/questions/53970/sanitize-user-entered-css looking at how Jetpack does it is probably the best option.


tem!


@norcross ^


1. I'm not sure what a "synonym" for a source would be. Is this a synonym for words in the title? Or is this something else?

2. More generally, let's say you're creating an animal database. You enter your synonyms in coma-delimited format. So:

```
cat: feline, kitty
lion: feline, king
cougar: mountain cat, feline
dog: puppy, pup
```

The way you have it set up, you could not filter by 'feline', since the value of the string field is always a literal two words. You would be better served by making this a multi-valued Solr field and putting in each of the synonyms separately.

3. The question of how to store it in Django: You could get around having to have a second table if you guarantee that it will always be comma-delimited words, and then split the string on the comma for indexing as multivalued in Solr.


You need to remove " or" here otherwise it becomes ambiguous.


@fmoliveira adiciona ele no grupo do whatsapp


I'm not able to find the file utils.py in my configuration. (CentOS) 64 bit


boa!


We should fix this in `publishing-api`, if the item is not valid it should return something more specific.


I think we should just use "or is `<var>node</var>`" here. It's a pointer comparison and we don't want confusion if `isSameNode()` makes a comeback.


Check if u can use one of Bootstrap helper classes. Link here http://getbootstrap.com/css/#helper-classes


Probably "Archived" is better because the tag will still exist in collections-publisher after this action.


format : options.schema.options.dateFormat
why? Dates havn't the allways the same format?

Should keep the maxDate.
I Don't understand the curValue section


No inline styling. Use http://getbootstrap.com/css/#helper-classes


Probably no need to include the single quotes in the format strings.


no inline styling use http://getbootstrap.com/css/#helper-classes


I assumed you guys were using relative paths because the tool didn't work at all with absolute paths. The examples in the README use relative paths. But I'm never working from the same directory, so it seems of limited value to me.


paltry commit - I prefer 420px


rename method -  isInputWithoutSelectionPropertiesInMozilla


Incorrect assignment yes.

I like to have unit postfix in variables that have a unit, not to mix them as here. I'm okay to remove them though as the CF coding style doesn't use them.


Why this change?


what value is 204800 in Kbyte/second?


Mazo mal todo, xD


Love the tailored messages :)


It would be easier to read if decidegrees were used everywhere. Then we just need to convert to degrees for the trigonometry functions.


:ok:


If you start with working on comments for documentation, I would suggest to use a standard documentation syntax as described https://www.stack.nl/~dimitri/doxygen/manual/docblocks.html
Preferrable would be either /// comment or javadoc-like multiline comment /**


@smarterclayton this go routine is practically useless :-)


relative paths in config files always seem better as then the file can be shared (to some extent), but if it's a command line arg then indeed it does not matter much.


This commit is being reviewed as a part of review issue #63.


不错不错，加油吧。

Kevin Guan <notifications@github.com>于2015年10月6日周二 下午4:54写道：

> @andytimes <https://github.com/andytimes>
> 最近忙的没看见的回复。当然就是自己写一个Blog咯～这可是我自己拿Python写出来的，没有用模板什么的～
>
> P.S. VPS不是我的，找别人借的。所以域名是http://supvesonico.moe/的子域名。
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/racaljk/hosts/commit/54cf361ccb5835a22674d597f48dd2bd188d9a61#commitcomment-13609983>
> .
>


public char [] URLify(char [] s, int len) {
		int new_len = len;
		for (int i=0; i<len; i++) {
			if (s[i] == ' ') new_len += 2;
		}

		int inx = new_len - 1;

		for (int i = len-1; i>=0; i--) { //change string from end
			if (s[i] != ' ') s[inx--] = s[i];
			else {
				s[inx--] = '0';
				s[inx--] = '2';
				s[inx--] = '%';
			}
		}
		return s;
	}


Use `each_with_object` instead of `reduce`.<br>Do not use semicolons to terminate expressions.


Do not suppress exceptions.


Useless `private` access modifier.


Align the parameters of a method call if they span more than one line.


Use `described_class` instead of `TogglIntegration::Team`


I don't want to add any more CLI parameters than necessary.

I tested 100% also and it wasn't that bad. Let's see in next version.


@gonzaloiv Troll!!!!


well sure, but then python2 doesn't support stuff like `file=sys.stderr` and so on...

anyways, PR welcome i am sure.


I'm not sure we need this to be here. If we already add a X1000 specific ethernet driver, and these values are fixed on the datasheet, eth.c could be handling it directly. Moreover, I don't see we populating anything on pci_addr at this platform level, so we could avoid one extra parameter being passed down the stack ;).


...that 'synonyms' fields is not really about 'synonyms' but about
'variants' (I know... not my fault... I know what a 'synonym' is...). So
in a title you can have 'favorite', and one would like to make it come
up for a 'favourite' search, that's all. If they really were synonyms, I
agree with you.
As I wrote, this (false)'synonym' field has the same function as
https://github.com/bassculture/BassCulture/blob/master/bassculture/models/tune.py#L21-L22,
i.e. a simple variant spelling. I don't need/want to filter by variants
(at least for now)

Thanks a lot for your explanation anyway, that's very useful. Let me
know if I'm missing something.

On 06/10/2015 14:35, Andrew Hankinson wrote:
>
> 1.
>
>     I'm not sure what a "synonym" for a source would be. Is this a
>     synonym for words in the title? Or is this something else?
>
> 2.
>
>     More generally, let's say you're creating an animal database. You
>     enter your synonyms in coma-delimited format. So:
>
> |cat: feline, kitty lion: feline, king cougar: mountain cat, feline
> dog: puppy, pup |
>
> The way you have it set up, you could not filter by 'feline', since
> the value of the string field is always a literal two words. You would
> be better served by making this a multi-valued Solr field and putting
> in each of the synonyms separately.
>
>  1. The question of how to store it in Django: You could get around
>     having to have a second table if you guarantee that it will always
>     be comma-delimited words, and then split the string on the comma
>     for indexing as multivalued in Solr.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bassculture/BassCulture/commit/c3efe9941fd48e55661684e078408374972f309d#commitcomment-13614912>.
>


we do journal...


You did "SHELL=/bin/sh" in another commit.
So /bin/sh executes this. It seems that /bin/sh may check the syntax first. The line 47 uses bash-specific syntax.
I guess following:
The substance of /bin/sh will be /bin/bash. Howerver, when /bin/bash is executed as /bin/sh, /bin/bash may check whether the given script uses bash-specific syntax or not.


好的。。。那我先把你这边merge了。。。那个Liberaries不用升级到23么？←_←。。。FABbehavior你那边是什么问题？

- 发送自我的Sony Xperia™智能手机

---- Qi Tang编写 ----

@troyliu0105
1. 夜间模式现在差文章详情页（webview 加载的）没改，查了半天也没改成功，有空你帮忙看下。（抱歉，代码写得跟屎一样，见谅）
2. 关于 FABProgressCircleBehavior，貌似显示上还是有问题，所以我先去掉了，你也可以看一下。
3. 接下来，我这没什么了，就看你有没有什么要跟新的，明天还有一天，8 号一早升到 0.7 release。

---
Reply to this email directly or view it on GitHub:
https://github.com/tangqi92/BuildingBlocks/commit/d3e731d253592de576ec801c1e9714dbb37e9afb#commitcomment-13614785


Lo de ArubaCloud cuál es? Aruba Cloud 1 euro al mes?


I filed a PR, see #16.


how can i get this token info? i have no idea... send me a sample http call in hipchat?

just rollback to connexion version 0.11.1, our application works again, so i believe this hardcoded scope caused the error


Yeah, that accidentally got duplicated. Do you want me to fix it or will you?


nope, it's compilation warning


Setting filenameRelative and sourceFileName to null is not working. I get the following error:

```
[14:42:56] Error on translate for infrastructure/editable-viewmodel at file:///D:/Repos/tmp/icc/App/App/app/src/infrastructure/editable-viewmodel.js
        Error: file:///D:/Repos/tmp/icc/App/App/app/src/infrastructure/editable-viewmodel.js: Invalid mapping: {"generated":{"line":4,"column":17},"source":"","original":{"line":8,"column":6},"name":null}
    at SourceMapGenerator_validateMapping [as _validateMapping] (file:///D:/Repos/tmp/icc/App/App/app/jspm_packages/npm/babel-core@5.8.25/browser.js:63070:15)
    at SourceMapGenerator_addMapping [as addMapping] (file:///D:/Repos/tmp/icc/App/App/app/jspm_packages/npm/babel-core@5.8.25/browser.js:62900:14)
    at SourceMap.mark (file:///D:/Repos/tmp/icc/App/App/app/jspm_packages/npm/babel-core@5.8.25/browser.js:9937:9)
    at CodeGenerator.print (file:///D:/Repos/tmp/icc/App/App/app/jspm_packages/npm/babel-core@5.8.25/browser.js:8777:14)
    at NodePrinter.plain (file:///D:/Repos/tmp/icc/App/App/app/jspm_packages/npm/babel-core@5.8.25/browser.js:9492:27)
    at CodeGenerator.VariableDeclarator (file:///D:/Repos/tmp/icc/App/App/app/jspm_packages/npm/babel-core@5.8.25/browser.js:8222:9)
    at CodeGenerator.print (file:///D:/Repos/tmp/icc/App/App/app/jspm_packages/npm/babel-core@5.8.25/browser.js:8779:20)
    at NodePrinter.plain (file:///D:/Repos/tmp/icc/App/App/app/jspm_packages/npm/babel-core@5.8.25/browser.js:9492:27)
    at CodeGenerator.printJoin (file:///D:/Repos/tmp/icc/App/App/app/jspm_packages/npm/babel-core@5.8.25/browser.js:8826:13)
    at NodePrinter.join (file:///D:/Repos/tmp/icc/App/App/app/jspm_packages/npm/babel-core@5.8.25/browser.js:9511:27)
    at NodePrinter.list (file:///D:/Repos/tmp/icc/App/App/app/jspm_packages/npm/babel-core@5.8.25/browser.js:9526:17)
```


Int promotion should handle it, result shoult fit. But I'm okay to use 32bit for stack variables. Don't know why I didn't do that.

Don't we lose precision if servoMid - servoMin is small?


xtables-persistence?


Ideally, there would be something like this as part of Contiki core and we would just hook up to platform code. Before we get there, I think we should move this to somewhere in platform/galileo/ instead.

It is a platform decision to add a process specific to handle ethernet as well as deciding which ethernet driver provided by cpu/x86/ to use. It fits better there, in my opinion.

Perhaps we could even merge this with eth_conf.[c,h]  ?!


It should be "fail on build errors"


removed


Hola, igual estamos troleando al pobre alberto, pero si os interesa, sin registraros ni nada rellenad este formulario, a mi tardaron unos dias en aceptarme. En "que aspectos del servicio quieres probar" les  puse que meter un odoo, un servidor web, un servicio de reseller de hosting y wapadas varias.
https://www.arubacloud.es/prueba-gratuitamente.aspx


Also [here](https://github.com/NicoHood/HID/wiki/Consumer-API)(Consumer-API): [link](https://github.com/NicoHood/HID/blob/master/examples/HID_Basic/HID_Consumer/HID_Consumer.ino) is broken


I'm running dash just to find these POSIX compatibility issues.
In my opinion we should fix/improve this kind of issues, but we can simply hard code the bash requirement. In the later case, then we don't need the "$BASH_VERSION" check in each individual run script and we can instead enforce this check in `misc/units`.
What do you think we should do?


Sorry to sneak this in here but I think it's cleaner


not used, you use lambda.


I'm still against that sleep in library code.


**Index Functions - Critical**

`self.booksArray` was declared as an `Array` in line _4_, and will never be **falsy**, with the provided functions that alter it's value.


Done they are Consolidated


In fact, it could be just a `Counter`.


http://mywiki.wooledge.org/Bashism
Interesting solution for the "Process Substitution" _bashism_.


@v-vlshch : certainly, we can and should have this discussion openly in the community. No need to open a new issue, we can comment on the one that's already open. Feel free to report your findings, or let me know if you prefer me to initiate the conversation.

Since at the moment we are not convinced that Windows is at fault / we came to a different conclusion with others on the issue, I think we should hold off on opening the PR. You could however post an updated commit in this repo (fixing the style issues) and reference it from your comments, if it helps the conversation.


removed the above INTEGRATED_AR


Maybe this is a stupid question, but why don't you just ask the hardware if SG is enabled? That should make things a lot easier than getting this from the devicetree.


Oh, wait. It's broken on TDM-gcc too!. Seems the `UNALIGNED` should be applied to all members of those network structs. I.e.:
```
typedef struct map_register_hdr {
	uint8_t type_and_flag UNALIGNED;
	uint8_t reserved    UNALIGNED;
```
(Similar to a gcc `<coff.h>` header here).

So I think it's high time to do this differently. E.g.
```
#include <pack_on.h>

typedef struct sensitive_network_type {
  uint8_t foo;
  uint16_t bar;
} sensitive_network_type;

#include <pack_off.h>
```
Refs:
 * http://www.watt-32.net/watt-doc/pack__on_8h_source.html
 * http://www.watt-32.net/watt-doc/pack__off_8h_source.html


what makes this run only after mongo has started up and been registered in etcd?


can we change this to just `minify`?


will be fixed as https://github.com/checkstyle/checkstyle/issues/2273


don't need to update the version, I'll do that :)


removed INTEGRATED_AR


*page


It should return true, and this program return true.
And after I check the question, the upper letters should all be regarded as lower letters, I did wrong about this. Here's the modification version:
		for (int i=0; i<len; i++) { //put each character of s into hashmap
			tmp = s.charAt(i);
			if (tmp < 'A' || tmp > 'z' || (tmp < 'a' && tmp > 'Z')) continue; //filter
			tmp = Character.toLowerCase(tmp);     // modified
			if (mp.containsKey(tmp)) mp.put(tmp, mp.get(tmp) + 1);
			else mp.put(tmp, 1);
		}
I'm sorry maybe I still don't get what you want to say. I would appreciate if you reply to this comment.


@gut, yep, I missed that, thanks!


Do you mean that I should add some new line between codes?


làm cái tự động đánh mã số đi


Problem with the filename **2105**-09-10-wpecommerce.md


This is system config. It should be `--system`


これなんでこうすると直るのでしょうか?
というか私の環境では直ってないです。


renamae 'isInputWithoutSelectionPropertiesInMozilla'


LGTM


Checkmarx scan results 06 Oct 2015 15:48 UTC +02:00

High
Medium
Low
Info

New
6
33
399
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

f2f26a42fcb8f71329dbabcdf3b1c99e2211e157	@dortal


Someone's a little excited.....


Does it make sense to have user/pass and token?


Why was the `switch/case` above replaced by the old `if` method? The former looks more robust to me as it provides overview and implements all possible cases. What was then the motivation to switch back?


2015.10.06 소스 개선


This commit is being reviewed as a part of review issue #63.


Is it possible to use the same workaround on a custom rss feed ? (rss feed generate from kickass)


Checkmarx scan results 06 Oct 2015 15:51 UTC +02:00

High
Medium
Low
Info

New
6
33
399
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

0deb47d4599928e0615ba250d5c5eec61069afc3	@amitchen22